Skip to content

Defines input-output maps with forecast variables ref#50

Closed
Giovanni3A wants to merge 8 commits into
devfrom
feature/forecast-var-indexed-map
Closed

Defines input-output maps with forecast variables ref#50
Giovanni3A wants to merge 8 commits into
devfrom
feature/forecast-var-indexed-map

Conversation

@Giovanni3A

Copy link
Copy Markdown
Collaborator
  • Creates variable indexed vector and matrix
  • Apply them on forecast step so predictions are indexed with variable refs
  • Use the variable index when applying predicted values in plan step

@codecov

codecov Bot commented Jan 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.52761% with 35 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.93%. Comparing base (bb71bba) to head (0a9d56e).
⚠️ Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
src/variable_indexed_structs.jl 64.78% 25 Missing ⚠️
src/ApplicationDrivenLearning.jl 42.85% 4 Missing ⚠️
src/predictive_model.jl 92.30% 4 Missing ⚠️
src/optimizers/gradient.jl 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev      #50      +/-   ##
==========================================
- Coverage   65.59%   64.93%   -0.66%     
==========================================
  Files          12       13       +1     
  Lines         558      693     +135     
==========================================
+ Hits          366      450      +84     
- Misses        192      243      +51     

☔ 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.

@Giovanni3A

Copy link
Copy Markdown
Collaborator Author

Closing this pull request because we though about a simpler way of doing this that results in lower training time.

@Giovanni3A Giovanni3A closed this Jan 28, 2026
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.

1 participant