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

[DOCS] Updated docstring for num2date #467

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ukmo-ccbunney
Copy link
Contributor

🚀 Pull Request

The underlying cftime package will now preferentially return an integer rather than a float if if encoded datetime(s) can be expressed exactly as an integer.

Description

Docstring for num2date updated to reflect this underlying change in behaviour.
Extra example added showing outputs of type integer where encoded allows.

Fixes: #202

Change in cfunits mean it will now preferentially return an integer
rather than a float if if encoded datetime(s) can be expressed exactly using ints.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In review
Status: No status
Development

Successfully merging this pull request may close these issues.

Wrong dtype of return in date2num
2 participants