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

[fix] Plotting Events Components #1420

Closed

Conversation

SimonWittner
Copy link
Collaborator

🔬 Background

🔮 Key changes

  • Pytest

📋 Review Checklist

  • I have performed a self-review of my own code.
  • I have commented my code, added docstrings and data types to function definitions.
  • I have added pytests to check whether my feature / fix works.

Please make sure to follow our best practices in the Contributing guidelines.

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #1420 (ef6cd44) into main (56d9910) will increase coverage by 0.04%.
Report is 1 commits behind head on main.
The diff coverage is 88.46%.

@@            Coverage Diff             @@
##             main    #1420      +/-   ##
==========================================
+ Coverage   89.88%   89.92%   +0.04%     
==========================================
  Files          38       38              
  Lines        5071     5073       +2     
==========================================
+ Hits         4558     4562       +4     
+ Misses        513      511       -2     
Files Changed Coverage Δ
neuralprophet/plot_utils.py 88.80% <88.46%> (+0.83%) ⬆️

@SimonWittner SimonWittner added the status: in development Pull requests which are in development label Sep 27, 2023
@SimonWittner
Copy link
Collaborator Author

Fixed with #1434

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: in development Pull requests which are in development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant