Skip to content

Initial version of the ncldDump application. #11

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 8 commits into from
Feb 24, 2017

Conversation

JimBiardCics
Copy link
Contributor

Mark,

Here's the initial version. You can read some details in the ReadMe.txt file.

Jim

@marqh
Copy link
Member

marqh commented Nov 1, 2016

Hi @JimBiardCics

an initial thought, while i go through the code in more detail

the name nclddump makes a bit wary

in netCDF world ncdump makes CDL, which is parseable via ncgen to make a file

So, do you think that the output from this tool will be parseable by ncgen?

If not, perhaps it is worth rethinking the name, to make it clear that this is not an ncgen parseable output

what do you think?

mark

@JimBiardCics
Copy link
Contributor Author

Mark,

I'm good either way. It is parseable with a couple of minor tweeks. I've
got some other things I want to do as well. What I posted is definitely
a rough first draft!

Grace and peace,

Jim

On 11/1/16 9:13 AM, marqh wrote:

Hi @JimBiardCics https://github.com/JimBiardCics

an initial thought, while i go through the code in more detail

the name |nclddump| makes a bit wary

in netCDF world ncdump makes CDL, which is parseable via ncgen to make
a file

So, do you think that the output from this tool will be parseable by
ncgen?

If not, perhaps it is worth rethinking the name, to make it clear that
this is not an ncgen parseable output

what do you think?

mark


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ5DJlr42xa8rr6MGCfs00Q82pdohukpks5q5zrugaJpZM4KjxTU.

CICS-NC http://www.cicsnc.org/ Visit us on
Facebook http://www.facebook.com/cicsnc Jim Biard
Research Scholar
Cooperative Institute for Climate and Satellites NC http://cicsnc.org/
North Carolina State University http://ncsu.edu/
NOAA National Centers for Environmental Information http://ncdc.noaa.gov/
/formerly NOAA’s National Climatic Data Center/
151 Patton Ave, Asheville, NC 28801
e: jbiard@cicsnc.org mailto:jbiard@cicsnc.org
o: +1 828 271 4900

/Connect with us on Facebook for climate
https://www.facebook.com/NOAANCEIclimate and ocean and geophysics
https://www.facebook.com/NOAANCEIoceangeo information, and follow us
on Twitter at @NOAANCEIclimate https://twitter.com/NOAANCEIclimate and
@NOAANCEIocngeo https://twitter.com/NOAANCEIocngeo. /

@JimBiardCics
Copy link
Contributor Author

Mark,
The output can now be cut and pasted into a text editor and it produces valid CDL that will regenerate the structure of the original netCDF file when passed to ncgen.

@jyucsiro
Copy link
Contributor

jyucsiro commented Feb 1, 2017

@JimBiardCics JimBiardCics merged commit aaccefe into binary-array-ld:master Feb 24, 2017
@JimBiardCics
Copy link
Contributor Author

I had some modifications from just before AGU that I hadn't commited.

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.

3 participants