Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add unit test for OSIRIS-REx SamCam #4280

Closed
jessemapel opened this issue Feb 9, 2021 · 2 comments · Fixed by #4378
Closed

Add unit test for OSIRIS-REx SamCam #4280

jessemapel opened this issue Feb 9, 2021 · 2 comments · Fixed by #4378
Assignees
Labels
enhancement New feature or request Missions Issues which are a priority for missions

Comments

@jessemapel
Copy link
Contributor

Description

Now that the OSIRIS-REx mission has gone to Bennu and collected a sample, we should add a unit test for SamCam.

This is a continuation of work discussed under #4221

Example

This can match the existing PolyCam and MapCam instruments from #4221 and #4060

@KrisBecker Can you point us to a good SamCam image for this?

@jessemapel jessemapel added enhancement New feature or request Missions Issues which are a priority for missions labels Feb 9, 2021
@KrisBecker
Copy link
Contributor

Try this one: https://sbnarchive.psi.edu/pds4/orex/orex.ocams/data_calibrated/sample_collection/20200414T224222S022_sam_radL2pan5.fits

@KrisBecker
Copy link
Contributor

Important note: The SamCam orientation (TRANS/ITRANS) and focal length from IAK v10 is updated. v10 may work OK for the SamCam in ISIS4. I can also provide some caminfo/phocube output for confirmation if needed.

@amystamile-usgs amystamile-usgs self-assigned this Mar 22, 2021
@amystamile-usgs amystamile-usgs linked a pull request Mar 25, 2021 that will close this issue
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Missions Issues which are a priority for missions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants