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.

Sign up for GitHub

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

Jump to bottom

Parse date to just year #4

Open
ellesmith88 opened this issue Aug 14, 2020 · 3 comments
Open

Parse date to just year #4

ellesmith88 opened this issue Aug 14, 2020 · 3 comments

Comments

@ellesmith88
Copy link
Contributor

Description

In daops and clisops currently only the year from the provided date is used. The TimeParameter class at the moment only provides the year in the dict and tuple because of this. Passing the full date and time through causes an error.
@ellesmith88
Copy link
Contributor Author

The error with the full date is ValueError: zero-size array to reduction operation minimum which has no identity

@agstephens
Copy link
Contributor

@ellesmith88 : please isolate a test in clisops to work out what happens when you call the clisops.core.subset.subset_* functions with different date formats.

@Zeitsperre
Copy link
Contributor

@cehbrecht Is this still the case in CLISOPS?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@agstephens @Zeitsperre @ellesmith88