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

Github issue 152: https://github.com/Unidata/netcdf-c/issues/152 #164

Merged
merged 2 commits into from
Nov 30, 2015

Conversation

DennisHeimbigner
Copy link
Collaborator

User request to have all orphaned DAP2 attributes kept as netcdf
global attributes. This is primarily a change in the oc code
nplus testcase dataset changes.

Result may be inconsistent with netcdf-Java output.

User request to have all orphaned DAP2 attributes kept as netcdf
global attributes. This is primarily a change in the oc code
nplus testcase dataset changes.

Result may be inconsistent with netcdf-Java output.
@WardF
Copy link
Member

WardF commented Nov 30, 2015

Is the fact that this is inconsistent with netcdf-java a showstopper? Copying @lesserwhirls , @JohnLCaron for their input.

@DennisHeimbigner
Copy link
Collaborator Author

I don't know. The effect is to make attributes appear
that previously did not exist. If someone should rely on
these attributes and then expect them for java, there might be
a problem. I will take a quick look at the java code to see
what happens to such attributes. In any case, I doubt that
this is a show-stopper.
=Dennis

Ward Fisher wrote:

Is the fact that this is inconsistent with netcdf-java a showstopper? Copying @lesserwhirls , @JohnLCaron for their input.


Reply to this email directly or view it on GitHub:
#164 (comment)

@DennisHeimbigner
Copy link
Collaborator Author

I checked the java code and it turns out that it already handles these
kinds of attributes, so in fact, this change to the netcdf-c will make
the c and java more consistent, not less.

@WardF
Copy link
Member

WardF commented Nov 30, 2015

Fantastic :). Merging shortly.

@WardF WardF merged commit b4459e0 into master Nov 30, 2015
DennisHeimbigner added a commit that referenced this pull request Mar 11, 2017
Github issue #152
requested that "orphaned" DAS attributes be included in the netcdf
metadata as global variables. The term orphaned here meant that
they were not connected to any variable in the DDS.
This was done in pull request #164

However, some servers (e.g. Thredds) include attributes for variables not
specified in a constraint expression, but which exist in the full DDS.
So I was adding these to the set of global attributes, but in retrospect
this should not have been done: they should have been elided.

Solution: modify oc2 code to be more distriminatory about
which orphaned attributes to include.
@WardF WardF deleted the gh152 branch September 8, 2021 16:39
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