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

clarify /req/edr/rc-md-query-links against non EDR based collections #150

Open
tomkralidis opened this issue Jan 5, 2025 · 4 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@tomkralidis
Copy link
Contributor

Latest testing of EDR CITE beta against a local pygeoapi instance yields the following failure:

Screenshot 2025-01-05 at 10 38 28

In my case, my OGC API instance has a few collections, a few of which are EDR related, and others are not. The above failure is based on a collection that is feature based, not EDR based.

When testing the same setup against Features CITE, all related tests pass.

Should this CITE test evaluate this requirement only if the collection if found to have EDR capabilities?

cc @bpross-52n

@tomkralidis
Copy link
Contributor Author

Related issue below (i.e. The data_queries element is missing from the collection lakes . The data_queries element is missing from the collection canada-hydat-daily-mean-02hc003).

Screenshot 2025-01-05 at 11 21 56

In other words, CITE tests should only evaluate collections which are EDR capable. This will also make things easier for products to provide CITE demos supporting multiple standards using a single deployment.

@dstenger dstenger added this to CITE Jan 6, 2025
@github-project-automation github-project-automation bot moved this to To do in CITE Jan 6, 2025
@dstenger dstenger added the enhancement New feature or request label Jan 7, 2025
@dstenger
Copy link
Collaborator

dstenger commented Jan 7, 2025

Thanks for the proposal. We will discuss it in the CITE team.

@dstenger dstenger moved this from To do to In progress in CITE Mar 13, 2025
@dstenger
Copy link
Collaborator

Question regarding #150 (comment):
This seems to be a error message which contains misleading content. The error is occurring in collection icoads-sst which should be an EDR collection. Can you confirm that?
All other mentioned collections are printed but not related to the error.
We will check if the error message can be improved.

@tomkralidis
Copy link
Contributor Author

@dstenger confirm. Tests can be (now) run per below:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In progress
Development

No branches or pull requests

3 participants