Skip to content

Conversation

@jakubchlapek
Copy link
Contributor

@jakubchlapek jakubchlapek commented Oct 17, 2025

Checklist before merging this PR:

  • Mentioned all issues that this PR fixes or addresses.
  • Summarized the updates of this PR under Summary.
  • Added an entry under Unreleased in the Changelog.

Summary

Changed Timeseries.plot() implementation to no longer rely on xarray under the hood while keeping same functionality.

Copy link
Collaborator

@dennisbader dennisbader left a comment

Choose a reason for hiding this comment

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

Beautiful, thanks a lot @jakubchlapek 😍 🚀

jakubchlapek and others added 2 commits October 20, 2025 09:00
Co-authored-by: Dennis Bader <[email protected]>
Co-authored-by: Dennis Bader <[email protected]>
@dennisbader dennisbader moved this to In review in darts Oct 21, 2025
@codecov
Copy link

codecov bot commented Oct 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.45%. Comparing base (9b61374) to head (4e98c5f).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2932      +/-   ##
==========================================
+ Coverage   95.40%   95.45%   +0.05%     
==========================================
  Files         146      146              
  Lines       15701    15703       +2     
==========================================
+ Hits        14979    14989      +10     
+ Misses        722      714       -8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dennisbader
Copy link
Collaborator

I added some unit tests for TimeSeries.plot(), will merge after all tests have passed.

@dennisbader dennisbader merged commit e106771 into unit8co:master Oct 31, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in darts Oct 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants