Skip to content

Adding example CDL file swath_AMSU_Brightness_Temp.cdl #29

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

Merged
merged 2 commits into from
Apr 19, 2017

Conversation

JimBiardCics
Copy link
Contributor

This file describes a set of swath observations. In this case the swath is acquired as a satellite instrument acquires measurements by scanning across the surface of the earth perpendicular to the direction of satellite motion. The independent dimensions in this instance are scan number and pixel in scan. These are not represented as true coordinate variables in this file, but this is not always the case. The result of the observations is a 2D image for each measurement quantity acquired, but this image does not lie cleanly on a lat/lon or projected x/y coordinate grid, so the geospatial coordinate variables are also 2D. There are a number of quality flag variables that provide information about the measurements.

@marqh
Copy link
Member

marqh commented Apr 19, 2017

Hi @JimBiardCics

i have updated the testing of these CDL files, and fixed an associated test failure.

Each file creates its own test case now. I have merged a number of CDL files

However, for this file, the test is failing, see:
https://travis-ci.org/binary-array-ld/bald/builds/210942538#L394

the ncgen command is failing. please could you have a look locally and try to ncgen this CDL file and see if there is a simple mistake in the syntax which would fix this?

thank you
mark

@JimBiardCics
Copy link
Contributor Author

@marqh I found the problem. It was the _FillValue attribute on the string time variable. Thoroughly non-standard, and although the CDL was generated from an actual file, ncgen refused to accept it. I don't know if I am free to click the "Merge pull request" button. Would you rather?

@marqh
Copy link
Member

marqh commented Apr 19, 2017

thank you @JimBiardCics i'll hit merge

@marqh marqh merged commit b02ef2b into binary-array-ld:master Apr 19, 2017
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 this pull request may close these issues.

2 participants