openclimatefix / PVNet

PVnet main repo
MIT License
15 stars 3 forks source link

Unsqueeze batch if batch size == 1 #137

Closed jacobbieker closed 4 months ago

jacobbieker commented 4 months ago

Pull Request

Description

Please delete the italicised instruction text! Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

Checklist:

codecov[bot] commented 4 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (0e59c2c) 56.99% compared to head (b353f52) 56.98%.

Files Patch % Lines
pvnet/models/multimodal/site_encoders/encoders.py 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #137 +/- ## ========================================== - Coverage 56.99% 56.98% -0.01% ========================================== Files 28 28 Lines 1795 1797 +2 ========================================== + Hits 1023 1024 +1 - Misses 772 773 +1 ```

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