Skip to content

Feature/ara dataset specs#200

Open
caglarkucuk wants to merge 5 commits intometno:mainfrom
caglarkucuk:feature/ara-dataset-specs
Open

Feature/ara dataset specs#200
caglarkucuk wants to merge 5 commits intometno:mainfrom
caglarkucuk:feature/ara-dataset-specs

Conversation

@caglarkucuk
Copy link
Copy Markdown

Extended identical variable names and added CRS for ARA domain (Closes #133 )

Comment thread bris/units.py Outdated
"celsius": ["C", "degC", "°C"],
"km/h": ["kmh"],
"kg/m^2": ["mm", "Kg/m^2", "Kg/m2", "kg/m2"],
"kg/m^2": ["mm", "Kg/m^2", "Kg/m2", "kg/m2", "mm/6h", "mm/3h", "mm/1h"], # verif package should be handling the temporal accum window
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I understand this comment. Can you elaborate?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Input file to verif package should correspond to the temporal accumulation window, but the comment is indeed confusing and unnecessary.
Will remove it

@tnipen
Copy link
Copy Markdown
Member

tnipen commented Oct 21, 2025

Also, can you run

ruff format --config pyproject.toml in the root of the repo to run linting?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compatibility with ARA dataset

2 participants