-
Notifications
You must be signed in to change notification settings - Fork 0
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
Parameter object in observations profile collection and data response #22
Comments
@Teddy-1000 Sounds good. But as with title, I think the intention here is to keep it short. Include a rule for 50 characters max? And that example from E-SOH may be from CoverageJSON, not collection? As labels in EDR collection does not currently support multi-language. |
This issue is regarding the observation profile response, so this would indeed be CovJSON. |
Ah, I misunderstood, sorry. I don't think its necessary to limit description no. |
I can expand this issue to also include the collection Parameter, since they are very closely connceted? |
I think it makes sense to handle both in one PR yes, since they anyway need to be in sync. Should probably exist as separate requirments in the profile documenth though. CF names seems good enough for me. We need to tackle exceptions maybe, but we can start with mandating CF names. I have no experience with CF cell methods, so not sure there. |
We've been discussing Parameter metadata for RODEO WP5 (Climate) as well. This is our current proposal:
This proposal is with the following ideas in mind:
The
This queries for air temperature, between 1 and 2 meter. It may return one or more actual parameters, that fit these properties. Another discussion point has been the the Parameter ID. We have chosen to give this no further meaning. In theory, it should only link |
@PaulVanSchayck Great, thank you! Could you also add an example for how this will look in the responding CoverageJSON document? |
You would hope they are equal, wouldn't you. Unfortunately, the specs (EDR vs CovJSON) do differ somewhat from each other. We've not implemented this proposal yet in CovJSON, so I'm not 100% sure if I've missed anything.
The most obvious difference is the i18n stuff which is present in CovJSON but not EDR. Also |
Many parts of the Parameter object are quite straight forward. Most difficult to define I think is the parameter-id, since this might be used by clients.
I do suggest we make a parameter label mandatory and description optional.
This is a current example of how E-SOH format of a parameter. It does not have a parameter label yet. First is from a data query, seconds is from a collection response.
Any feed back or discussion can be done here, or else I will get to creating a PR for this.
The text was updated successfully, but these errors were encountered: