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

Qmos - Latitude Constructor Error #4754

Closed
lwellerastro opened this issue Jan 19, 2022 · 4 comments · Fixed by #5130
Closed

Qmos - Latitude Constructor Error #4754

lwellerastro opened this issue Jan 19, 2022 · 4 comments · Fixed by #5130
Assignees
Labels
bug Something isn't working Products Issues which are impacting the products group
Milestone

Comments

@lwellerastro
Copy link
Contributor

lwellerastro commented Jan 19, 2022

ISIS version(s) affected:5.0.2

Description
The following error is encountered when loading a map file asking for Planetographic LatitudeType and grid modifications the first time grid options is chosen (it is super specific and described below).
qmosError

How to reproduce
Data and map files files available under my work area /Isis3Tests/Qmos/GridMapCombo

  • launch qmos
  • open cubelist cubs.lis
  • quick load map (lightening bolt icon) - choose moon_spole_ographic.map
  • click grid icon (if not there right click on window somewhere and start turning on tools)
  • select grid options - error appears, ok but will come back with every grid option select

Make the problem go away

  • quick load map moon_spole_ocentric.map
  • select grid options - works
  • quick load map moon_spole_ographic.map
  • select grid options - works this time

Possible Solution
Current work around is to use to ocentric latitude type. In fact, once an ocentric map file is loaded (before or after the ographic error message), loading an ographic based map file works without error when trying to adjust the grid.

@lwellerastro lwellerastro added the Products Issues which are impacting the products group label Jan 19, 2022
@jessemapel jessemapel added the bug Something isn't working label Jan 21, 2022
@jessemapel
Copy link
Contributor

I think this is solvable by passing the mapping group from the projection to Latitude constructors in MosaicGridTool::onProjectionChaned.

https://github.com/USGS-Astrogeology/ISIS3/blob/dev/isis/src/qisis/objs/MosaicSceneWidget/MosaicGridTool.cpp#L810

Similar to how they are handled in MosaicGridTool::setLatExtents

Related to #3714

@jessemapel
Copy link
Contributor

@AustinSanders This is a good small ticket for the support sprint. Biggest overhead will be doing the GUI testing.

@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 Jan 11, 2023
@blandoplanet
Copy link

This still needs addressed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Products Issues which are impacting the products group
Projects
None yet
5 participants