Skip to content
This repository was archived by the owner on Jun 2, 2025. It is now read-only.

add if statement #295

Merged
merged 4 commits into from
Mar 26, 2024
Merged

add if statement #295

merged 4 commits into from
Mar 26, 2024

Conversation

peterdudfield
Copy link
Contributor

Pull Request

Description

Fix tests

Fixes #

How Has This Been Tested?

CI tests

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@peterdudfield peterdudfield requested review from Sukh-P and dfulu and removed request for jacobbieker March 26, 2024 17:13
Copy link

codecov bot commented Mar 26, 2024

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 76.51%. Comparing base (e85b5a1) to head (282677f).
Report is 19 commits behind head on main.

Files Patch % Lines
ocf_datapipes/training/windnet.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #295      +/-   ##
==========================================
- Coverage   76.52%   76.51%   -0.02%     
==========================================
  Files         124      125       +1     
  Lines        5875     5909      +34     
==========================================
+ Hits         4496     4521      +25     
- Misses       1379     1388       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@peterdudfield peterdudfield merged commit 5d9b1e3 into main Mar 26, 2024
@peterdudfield peterdudfield deleted the issue/fix-tests branch March 26, 2024 17:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants