Skip to content

Add dt.date to plottable types#8873

Merged
dcherian merged 13 commits intopydata:mainfrom
saschahofmann:enable-date-type-in-plots
Mar 29, 2024
Merged

Add dt.date to plottable types#8873
dcherian merged 13 commits intopydata:mainfrom
saschahofmann:enable-date-type-in-plots

Conversation

@saschahofmann
Copy link
Contributor

@saschahofmann saschahofmann commented Mar 25, 2024 •
edited
Loading

Simply adds datetime.date to plottable types in _ensure_plottable in plot/utils.pyL675 to enable the plotting of dates.

Matplotlib handles date types automatically, so I think there is no other change needed.

Do I need to add a test for this? Any pointers on where I would put it that?

@welcome
Copy link

welcome bot commented Mar 25, 2024

Thank you for opening this pull request! It may take us a few days to respond here, so thank you for being patient.
If you have questions, some answers may be found in our contributing guidelines.

@saschahofmann saschahofmann mentioned this pull request Mar 25, 2024
Closed
5 tasks
@Illviljan
Copy link
Contributor

Illviljan commented Mar 27, 2024

Could you add a test in https://github.com/pydata/xarray/blob/main/xarray/tests/test_plot.py so we don't regress?

@saschahofmann
Copy link
Contributor Author

saschahofmann commented Mar 27, 2024

Sure! Shall I just check whether it plots ? Something like this test_datetime_dimension in L612 just with a date dimension?

@saschahofmann
Copy link
Contributor Author

saschahofmann commented Mar 28, 2024

I added the test_date_dimension plot, I saw that there is a whole TestDatetimePlot class. Should I add an equivalent or even refactor it so I only need to change the self.darray? Or is it fine with a single test?

@saschahofmann
Copy link
Contributor Author

saschahofmann commented Mar 28, 2024 •
edited
Loading

Quick question: Would it be worth it to un-gitignore your .vscode folder?

dcherian
dcherian approved these changes Mar 28, 2024
View reviewed changes
Copy link
Contributor

@dcherian dcherian left a comment

Choose a reason for hiding this comment

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

Nice work. Thanks!

@dcherian dcherian added the plan to merge Final call for comments label Mar 28, 2024
@dcherian dcherian merged commit 852b7e6 into pydata:main Mar 29, 2024
@welcome
Copy link

welcome bot commented Mar 29, 2024

Congratulations on completing your first pull request! Welcome to Xarray! We are proud of you, and hope to see you again! celebration gif

dcherian added a commit to dcherian/xarray that referenced this pull request Apr 2, 2024
@dcherian
Merge branch 'main' into state-machine
4399d96 
* main: (26 commits)
  [pre-commit.ci] pre-commit autoupdate (pydata#8900)
  Bump the actions group with 1 update (pydata#8896)
  New empty whatsnew entry (pydata#8899)
  Update reference to 'Weighted quantile estimators' (pydata#8898)
  2024.03.0: Add whats-new (pydata#8891)
  Add typing to test_groupby.py (pydata#8890)
  Avoid in-place multiplication of a large value to an array with small integer dtype (pydata#8867)
  Check for aligned chunks when writing to existing variables (pydata#8459)
  Add dt.date to plottable types (pydata#8873)
  Optimize writes to existing Zarr stores. (pydata#8875)
  Allow multidimensional variable with same name as dim when constructing dataset via coords (pydata#8886)
  Don't allow overwriting indexes with region writes (pydata#8877)
  Migrate datatree.py module into xarray.core. (pydata#8789)
  warn and return bytes undecoded in case of UnicodeDecodeError in h5netcdf-backend (pydata#8874)
  groupby: Dispatch quantile to flox. (pydata#8720)
  Opt out of auto creating index variables (pydata#8711)
  Update docs on view / copies (pydata#8744)
  Handle .oindex and .vindex for the PandasMultiIndexingAdapter and PandasIndexingAdapter (pydata#8869)
  numpy 2.0 copy-keyword and trapz vs trapezoid (pydata#8865)
  upstream-dev CI: Fix interp and cumtrapz (pydata#8861)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dcherian dcherian dcherian approved these changes

Assignees

No one assigned

Labels

plan to merge Final call for comments topic-plotting

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Cannot plot datetime.date dimension

3 participants

@saschahofmann @Illviljan @dcherian