openclimatefix / nowcasting_dataset

Prepare batches of data for training machine learning solar electricity nowcasting data
https://nowcasting-dataset.readthedocs.io/en/stable/
MIT License
25 stars 6 forks source link

Issue/628 satellite order #640

Closed peterdudfield closed 2 years ago

peterdudfield commented 2 years ago

Pull Request

Description

make sure Satellite batch object has the dimesions in a certain order

Fixes #628

How Has This Been Tested?

-normal unittests -added a test

Checklist:

codecov-commenter commented 2 years ago

Codecov Report

Merging #640 (aac1225) into main (e68285d) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #640   +/-   ##
=======================================
  Coverage   93.32%   93.33%           
=======================================
  Files          45       45           
  Lines        3041     3044    +3     
=======================================
+ Hits         2838     2841    +3     
  Misses        203      203           
Impacted Files Coverage Δ
..._dataset/data_sources/satellite/satellite_model.py 100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more