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

Oblique Detector Resolution Should Use Local Emission Angle #3600

Closed
KrisBecker opened this issue Dec 13, 2019 · 4 comments
Closed

Oblique Detector Resolution Should Use Local Emission Angle #3600

KrisBecker opened this issue Dec 13, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request Missions Issues which are a priority for missions
Milestone

Comments

@KrisBecker
Copy link
Contributor

ISIS version(s) affected: All

Note the OSIRIS-REx team has a custom version of ISIS we have been maintaining and have modified code to address issues we have encountered working with small irregular bodies.

Description

The oblique detector resolution that is being calculated in the Camera::ObliqueDetectorResolution() class uses the emission angle computed from the ellipsoid model. We changed from using the ellipsoid model to instead apply the DEM/shape model so that the local emission angle can be used.

Using the local emission angle provides a much better indication of the oblique detector resolution because if takes into account surface topography.

This also affects phocube oblique detector resolution backplane as well (in a non-backward compatible way, so a new option was added to compute the old oblique detector resolution) which I could also submit for changes. Also, as a side affect, any other computation involving the ObliqueDetectorResolution, such as ObliqueSampleResolution and ObliqueLineResolution, are also affected.

How to reproduce

Applications such as phocube, camstats, campt and caminfo can provide examples of this data value.

Possible Solution

We can provide a PR/code that has this modification.

Additional context

@jessemapel jessemapel added the Missions Issues which are a priority for missions label Dec 13, 2019
@jessemapel
Copy link
Contributor

Why this is important

The oblique detector resolution is suppose to be a more accurate measurement of the resolution. So, we should use the most accurate surface model available. For irregular bodies, there can be large variations in the surface normal, and thus the emission angle. In these situations, the ellipsoid based emission angle is insufficient.

@jessemapel jessemapel added the enhancement New feature or request label Dec 16, 2019
@ascbot
Copy link
Contributor

ascbot commented Sep 1, 2020

I am a bot that cleans up old issues that do not have activity.

This issue has not received feedback in the last six months. I am going to add the inactive label to
this issue. If this is still a pertinent issue, please add a comment or add an emoji to an existing comment.

I will post again in five months with another reminder and will close this issue on it's birthday unless it has
some activity.

@ascbot ascbot added the inactive Issue that has been inactive for at least 6 months label Sep 1, 2020
@jessemapel
Copy link
Contributor

This is still outstanding

@jessemapel jessemapel removed the inactive Issue that has been inactive for at least 6 months label Sep 9, 2020
@github-actions
Copy link

Thank you for your contribution!

Unfortunately, this issue hasn't received much attention lately, so it is labeled as 'stale.'

If no additional action is taken, this issue will be automatically closed in 180 days.

@github-actions github-actions bot added the inactive Issue that has been inactive for at least 6 months label May 26, 2021
@AustinSanders AustinSanders added this to the 6.1.0 milestone Aug 5, 2021
@acpaquette acpaquette removed the inactive Issue that has been inactive for at least 6 months label Aug 9, 2021
@tgiroux tgiroux self-assigned this Aug 16, 2021
@scsides scsides modified the milestones: 6.1.0, 7.0.1 Jan 18, 2022
@jessemapel jessemapel moved this to Needs Initial Review in ASC Software Support Feb 2, 2022
@jlaura jlaura moved this from Needs Initial Review to Todo in ASC Software Support Feb 9, 2022
@jessemapel jessemapel moved this from Todo to Under Review in ASC Software Support Feb 11, 2022
Repository owner moved this from Under Review to Done in ASC Software Support Feb 14, 2022
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

No branches or pull requests

7 participants