-
Notifications
You must be signed in to change notification settings - Fork 2
Initial ACDD Tests #65
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
base: main
Are you sure you want to change the base?
Conversation
An initial set of ACDD tests. Adapted from ioos/compliance_checker.
|
@abkfenris, thank you very much and we apologize for overlooking this. We are extremely busy at the moment, which is why |
|
@forman Glad to hear that We're still interested in implementing tests for ACDD and other conventions, but didn't want to go too far down the wrong track or sink a lot of effort into something that is no longer maintained. |
|
It is in active use in our company (for sure a few others), just maintenance slowed down recently. We'll be back! |
|
Great! When I went asking around the community many folks hadn't heard about it but were very interested. |
forman
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That looks already very good. Great work! We are happy to take this on board.
To move forward, could you please also
- Describe in
CHANGES.mdwhat will we added - Adapt the rule documentation generator mkruleref.py to also convert the new rules into markdown docs.
- Run the converter to update docs/rule-ref.md
Note, I wont be able to support you until the 14.01.2026, therefore pls contact my colleague @clarasb.
pyproject.toml
Outdated
|
|
||
| [project.optional-dependencies] | ||
| acdd = [ | ||
| "isodate>=0.7.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm also fine to make this a common project dependency.
|
Great, I'll work on getting those ready by the time you're back, and I may have some additional ACDD tests adapted as well. |
|
...and please add yourself to the list of authors in |
Adds documentation for ACDD tests, isodate to the dependencies, and updates the changes.
I wanted to share an early draft PR before implementing too many rules to make sure I'm on the right track.
The IOOS compliance checker has implemented their own tests for ACDD, CF, IOOS, and others, but have ended up with a much more complex and hard to adapt library in the process. There is interest in using XRLint to help simplify things, and make it easier to share work across the ecosystem. Along those lines an idea is to move tests/rules upstream if there is interest rather than keeping their own implementation.
I've converted many of the more general ACDD global attribute tests over here.