-
Notifications
You must be signed in to change notification settings - Fork 86
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
feat: general refactoring, closing todos #1328
Conversation
Hello @Lopa10ko! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-10-17 09:37:07 UTC |
Code in this pull request still contains PEP8 errors, please write the Comment last updated at |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1328 +/- ##
==========================================
- Coverage 80.14% 80.10% -0.04%
==========================================
Files 146 146
Lines 10285 10296 +11
==========================================
+ Hits 8243 8248 +5
- Misses 2042 2048 +6 ☔ View full report in Codecov by Sentry. |
moved to #1343 |
This is a 🔨 code refactoring.
Summary
OutputData
(804ef43)ar
to a minimal set for fast train forecasting (f15a7a1)nodes: List[PipelineNode]
property to fix unresolved references (5c27ac8)