Ouranosinc / xscen

A climate change scenario-building analysis framework.
https://xscen.readthedocs.io/
Apache License 2.0
15 stars 2 forks source link

Fix build_path for invariants #239

Closed aulemahal closed 1 year ago

aulemahal commented 1 year ago

Reverse logic to allow missing dates on FX data.

Pull Request Checklist:

What kind of change does this PR introduce?

In _schema_dates (which generates the "DATES" string), the fast-path for invariant data happend before the optional checks. Invariant data should not have date_start and date_end, so the current code will always fail for xrfreq='fx'.

Does this PR introduce a breaking change?

No.

Other information:

aulemahal commented 1 year ago

Didn't even wait for a successful CI. :cowboy_hat_face: