Add demos - #38
Merged
Merged
Conversation
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Add docs/demos/co-registration.md and demo-co-registration.sh covering the tiepoint -> pointing-solution -> mosaic chain, with verified runs on MSL and M20 NavCam data and an explicit statement that no change-detection program exists in the image. Correct the marsmap/marsmos rows in docs/architecture/components.md and add the missing marsortho row, per the in-image .pdf parameter files. Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-authored-by: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Add test-product-pipeline.sh, which builds a synthetic stereo pair and CAHV camera models inside the image (no calibration, no downloaded data) and runs marscorr -> marsxyz -> marsmesh plus marsmos and marsmap, asserting on product content: tiepoint and valid-point counts, OBJ vertex/face counts, finite vertex coordinates and reconstructed range against the fixture's known 20 m truth, and mosaic dimensions plus pixel statistics. Wire it into the build/publish workflow as its own step, and stop Test 6 from closing 'list' output early, which deadlocks the wrapper. Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
tig translates paths in arguments but not inside a list file, and only $HOME is mounted at its host path, so inputs elsewhere need a /host prefix. Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
…e a vertex floor in the mesh coordinate check Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
…c-demo * Add in-situ 360-degree panorama mosaic demo Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * Document remaining workspace outputs and overlap-count variability Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * Preserve frame order and avoid name collisions in the marsrad stage Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * Trim comments Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * Refuse to run when a frame lives in the workspace being cleaned Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * Warn when the display stretch produces nothing Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: jason.han <jason.han@jpl.nasa.gov>
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: jason.han <jason.han@jpl.nasa.gov>
…duct-tests * CI: assert TIG produces terrain products Add test-product-pipeline.sh, which builds a synthetic stereo pair and CAHV camera models inside the image (no calibration, no downloaded data) and runs marscorr -> marsxyz -> marsmesh plus marsmos and marsmap, asserting on product content: tiepoint and valid-point counts, OBJ vertex/face counts, finite vertex coordinates and reconstructed range against the fixture's known 20 m truth, and mosaic dimensions plus pixel statistics. Wire it into the build/publish workflow as its own step, and stop Test 6 from closing 'list' output early, which deadlocks the wrapper. Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * Do not pipe VICAR wrapper output into an early-closing reader; require a vertex floor in the mesh coordinate check Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
…oodness) demo-surface-characteristics.sh derives surface normals, slope/heading/northerly-tilt/solar, roughness and instrument-placement goodness from an XYZ point cloud and converts each product to PNG. docs/demos/surface-characteristics.md documents the workflow, parameter defaults, physical units and the verified/unverified split.
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-authored-by: HuiJun <jason.han@jpl.nasa.gov>
…ends Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
…teristics * Add worked surface-characteristics demo (slope, roughness, normals, goodness) demo-surface-characteristics.sh derives surface normals, slope/heading/northerly-tilt/solar, roughness and instrument-placement goodness from an XYZ point cloud and converts each product to PNG. docs/demos/surface-characteristics.md documents the workflow, parameter defaults, physical units and the verified/unverified split. * Correct three measured figures in the doc; clean stale products for every instrument * README: reconcile scope list, demos section and project tree with the new demo * README: the panorama demo now on develop also ships a worked example --------- Co-authored-by: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
…-doc * docs: co-registration and change-monitoring workflow Add docs/demos/co-registration.md and demo-co-registration.sh covering the tiepoint -> pointing-solution -> mosaic chain, with verified runs on MSL and M20 NavCam data and an explicit statement that no change-detection program exists in the image. Correct the marsmap/marsmos rows in docs/architecture/components.md and add the missing marsortho row, per the in-image .pdf parameter files. Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * fix: keep final list entry when the image list has no trailing newline Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * docs: reflect run-to-run variability and the marsmap hwloc workaround Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * fix: write container-visible paths into the frame list tig translates paths in arguments but not inside a list file, and only $HOME is mounted at its host path, so inputs elsewhere need a /host prefix. Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * docs: loosen the MSL example's run-to-run figures to magnitudes Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * fix: resolve symlinks in input paths and fail loudly if marschkovl abends Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * fix: check calibration after argument parsing so --help works without it Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * fix: fail the run when marsautotie or marsnav abends Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> * docs: print the marsmap hwloc workaround in the demo's next-step hint Co-Authored-By: jason.han <jason.han@jpl.nasa.gov> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: jason.han <jason.han@jpl.nasa.gov>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Demos to show capabilities mentioned in the advertisements.