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
DM-41219: Fix errors in dp02 and dp02 schemas #145
Conversation
9cf4189
to
0b57e27
Compare
0b57e27
to
bbcda74
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please correct the units ( arcsec2
-> arcsec**2
) as noted in the detailed comments in one place (but it appears multiple times). This is what the VOUnit standard requires.
We need to see if we can figure out how to make the automated checks more strict.
yml/dp02_dc2.yaml
Outdated
@@ -8092,37 +8092,37 @@ tables: | |||
datatype: double | |||
mysql:datatype: DOUBLE | |||
description: Elliptical Gaussian adaptive moments | |||
fits:tunit: asec^2 | |||
fits:tunit: arcsec2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
VOUnit requires arcsec**2
here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be fixed now.
yml/dp02_dc2.yaml
Outdated
- name: ixyPSF | ||
"@id": "#DiaSource.ixyPSF" | ||
datatype: double | ||
mysql:datatype: DOUBLE | ||
description: Adaptive moments of the PSF model at the object position | ||
fits:tunit: asec^2 | ||
fits:tunit: arcsec2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missed one here
ce79156
to
b83663c
Compare
The 'stat.error' word needs to be primary and not secondary according to UCD1+ syntax.
This conforms to the astropy v5.3.3 syntax for unit specification.
This follows the standard in the VOAUnits recommendation.
b83663c
to
3431dee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now, thank you!
No description provided.