euro-cordex / cc-plugin-cc6

Compliance checker plugin to test compliance with CORDEX-CMIP6 archive specifications
Apache License 2.0
0 stars 0 forks source link

Splitting general and CORDEX-Checks, adding further checks #5

Closed sol1105 closed 4 months ago

sol1105 commented 4 months ago

In further PRs:

larsbuntemeyer commented 4 months ago

I can do the the domain coordinates checks!

sol1105 commented 4 months ago

@larsbuntemeyer I think it is in a state that you could have a look. Would be great if you provided some general feedback, also for the "roadmap" sketched above.

I am yet uncertain whether dependent checks should generally throw an error when they cannot be conducted or simply should not run (i.e. the frequency cannot be identified, so most of the time checks will either not run or throw errors). I mostly opted for "not running".

larsbuntemeyer commented 4 months ago

Thanks! I'll have a look asap!

larsbuntemeyer commented 4 months ago

Would be great if you provided some general feedback, also for the "roadmap" sketched above.

I think it is straightforward to check the coordinate using the domain_id attribute and comparing this using cx.domain and using my former checks. But let's keep that for another PR...

larsbuntemeyer commented 4 months ago

I think, this could be merged, if you like. This is lots of new stuff, we can then start from a common code base to tackle all of the rest.

sol1105 commented 4 months ago

Sounds good, thanks