Skip to content

Conversation

@dennisbader
Copy link
Collaborator

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.

Fixes #2524.

Summary

  • Fixed a bug when performing probabilistic optimized historical forecasts (num_samples > 1) with a regression models where reshaping the array resulted in a wrong order of samples across components and forecasts.

@codecov
Copy link

codecov bot commented Sep 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.76%. Comparing base (41e1177) to head (3ecfc2b).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2534      +/-   ##
==========================================
- Coverage   93.77%   93.76%   -0.02%     
==========================================
  Files         139      139              
  Lines       14765    14751      -14     
==========================================
- Hits        13846    13831      -15     
- Misses        919      920       +1     

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

@dennisbader dennisbader merged commit 67d4dbd into master Sep 15, 2024
eschibli pushed a commit to eschibli/darts that referenced this pull request Oct 8, 2024
@madtoinou madtoinou mentioned this pull request Nov 8, 2024
3 tasks
@madtoinou madtoinou deleted the fix/optimized_hfc_prob_regr branch November 11, 2024 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Quantiles/samples are wrong when using enable_optimization=True (default!)

2 participants