pytorch-forecasting icon indicating copy to clipboard operation
pytorch-forecasting copied to clipboard

Fixed RuntimeError on variable validation batch size

Open Luke-Chesley opened this issue 1 year ago • 1 comments

Description

Changed dimension in concat_sequences to cat along dim = 0. This fixes bug mentioned in linked issue.

Checklist

  • [ ] Linked issues (if existing)
  • [ ] Amended changelog for large changes (and added myself there as contributor)
  • [ ] Added/modified tests
  • [ ] Used pre-commit hooks when committing to ensure that code is compliant with hooks. Install hooks with pre-commit install. To run hooks independent of commit, execute pre-commit run --all-files

Luke-Chesley avatar Feb 08 '24 15:02 Luke-Chesley

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b3fcf86) 90.19% compared to head (67ed8e5) 90.11%. Report is 8 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1511      +/-   ##
==========================================
- Coverage   90.19%   90.11%   -0.09%     
==========================================
  Files          30       30              
  Lines        4724     4724              
==========================================
- Hits         4261     4257       -4     
- Misses        463      467       +4     
Flag Coverage Δ
cpu 90.11% <100.00%> (-0.09%) :arrow_down:
pytest 90.11% <100.00%> (-0.09%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Feb 10 '24 14:02 codecov-commenter