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

Instrument GSD lacks units #1015

Closed
jlaura opened this issue Feb 24, 2021 · 3 comments · Fixed by #1016
Closed

Instrument GSD lacks units #1015

jlaura opened this issue Feb 24, 2021 · 3 comments · Fixed by #1016
Assignees
Milestone

Comments

@jlaura
Copy link

jlaura commented Feb 24, 2021

This is tangentially related to #1014. The ground sample distance (gsd) entry in unitless. I suspect for Earth focused observations one assumes that the unit is meters? For many planetary missions (Moon, Mars, etc.) we could infer the unit from the mission name. Unfortunately, this is not universally true as the highly elliptical orbits of some outer planets result in observation sets where the GSD (or mean gsd over the observation) can vary between tens of kilometers to tens of meters.

I am wondering if it would make sense to either explicitly define the GSD as being meters (and if I missed this I apologize!) or to make the GSD an object and include the unit of the numeric.

@cholmes
Copy link
Contributor

cholmes commented Feb 25, 2021

Hrm, that's definitely an oversight if we don't define that it's in meters. I'll check, and even if we do it sounds like we need to make it more clear.

In general we try to not use objects so much, but I could see a gsd_units field, where units is defined. That one too I think probably makes sense to at least start in a 'planetary extension' (feel free to make a better name than that), and I think we could have the extension be able to redefine the units (and could put a mention in the GSD field of the main spec that the default is meters, but that extensions can over ride that or something).

@m-mohr
Copy link
Collaborator

m-mohr commented Feb 25, 2021

It's defined in meters:

gsd is the nominal Ground Sample Distance for the data, as measured in meters on the ground.

It's just not in the table, it's in the long description below. I agree that it is a good idea to add the unit to the table.

@m-mohr m-mohr mentioned this issue Feb 25, 2021
4 tasks
@m-mohr
Copy link
Collaborator

m-mohr commented Feb 25, 2021

See PR #1016

@cholmes cholmes linked a pull request Feb 25, 2021 that will close this issue
4 tasks
@cholmes cholmes added this to the 1.0.0-RC.1 milestone Feb 25, 2021
@m-mohr m-mohr closed this as completed Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants