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

UMBRELLA: Overhaul climatology.py and test_climatology.py #313

Closed
5 of 6 tasks
hCraker opened this issue Dec 7, 2022 · 2 comments
Closed
5 of 6 tasks

UMBRELLA: Overhaul climatology.py and test_climatology.py #313

hCraker opened this issue Dec 7, 2022 · 2 comments
Assignees
Labels
umbrella Encompasses multiple sub-issues

Comments

@hCraker
Copy link
Contributor

hCraker commented Dec 7, 2022

Various issues dealing with climatology.py and its test file have been opened. This is an umbrella issue to help keep track of the status of these files.

@hCraker hCraker added high priority umbrella Encompasses multiple sub-issues labels Dec 7, 2022
@hCraker hCraker self-assigned this Dec 7, 2022
@anissa111 anissa111 assigned jukent and unassigned hCraker May 15, 2023
@jukent jukent mentioned this issue May 16, 2023
17 tasks
@jukent
Copy link
Contributor

jukent commented Jul 26, 2023

@jukent
Copy link
Contributor

jukent commented Aug 3, 2023

After much discussion we decided that numpy support will not be necessary in this instance. Since the input requires two variables where one is just the index of the other and outputs two variables where again one is just an index (the new time coordinates are necessary as well as the climatological values), these are not operations that should be done in numpy.

@jukent jukent closed this as completed Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
umbrella Encompasses multiple sub-issues
Projects
None yet
Development

No branches or pull requests

2 participants