-
Notifications
You must be signed in to change notification settings - Fork 1
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
Issues discovered by verifier #29
Comments
Tami mentioned a few telecons ago that this dataset is stored in netCDF,
and they had to stick a bunch of data in the master, which is 63 MB:
http://mag.gmu.edu/git-data/cdaweb-hapi-metadata/cache/bw/AIM/AIM_CIPS_SCI_3A/
It is on my list of "skips" that we should look into at the end.
…On Mon, Jun 19, 2023 at 5:28 PM Jeremy Faden ***@***.***> wrote:
The validator does a more exhaustive test, and finds some new things. This
ticket is to enumerate.
- https://cdaweb.gsfc.nasa.gov/hapi/info?id=AIM_CIPS_SCI_3A results in
400.
—
Reply to this email directly, view it on GitHub
<#29>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAUQ57RHWWRMMH255YYBCY3XMDABFANCNFSM6AAAAAAZMNNCHQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
I like the idea of a skips list. If one of every hundred data sets fail it makes the server look bad, even though 99% of the server is working properly. It's good to have a skips list so that only the working 99% is presented, and the remaining 1% is where to focus the development. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The validator does a more exhaustive test, and finds some new things. This ticket is to enumerate.
"Expect fill value for a double parameter to not have a two or more non-zero decimal places" because the integer fill value is written as 4.294967295E9 and transmitted as doubles for Nand's server. This should be fixed when integers are used.
The text was updated successfully, but these errors were encountered: