Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace H with h inside pandas' floor function #64

Conversation

0xFrama
Copy link
Contributor

@0xFrama 0xFrama commented Feb 27, 2024

Pull Request

Description

Replaced parameter H with parameter h inside pandas' floor function.

Fixes #62

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

Copy link

codecov bot commented Feb 27, 2024

Codecov Report

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

Project coverage is 80.83%. Comparing base (6b127a9) to head (771de96).
Report is 1 commits behind head on main.

Files Patch % Lines
quartz_solar_forecast/dataset/make_test_set.py 0.00% 1 Missing ⚠️
quartz_solar_forecast/eval/nwp.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   80.83%   80.83%           
=======================================
  Files          11       11           
  Lines         334      334           
=======================================
  Hits          270      270           
  Misses         64       64           

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

@peterdudfield
Copy link
Contributor

You mind checking one more - quartz_solar_forecast/eval/nwp.py:84 I think. I saw this in the warnings in Github Actions when pytest is running

@0xFrama
Copy link
Contributor Author

0xFrama commented Feb 27, 2024

This PR fixes the one you mentioned and another one in quartz_solar_forecast/dataset/make_test_set.py:119 as well.
I didn't find anything else.

@peterdudfield peterdudfield merged commit af1a7bd into openclimatefix:main Feb 27, 2024
3 of 4 checks passed
peterdudfield added a commit that referenced this pull request Mar 4, 2024
* Bump version: 1.0.9 → 1.0.10 [skip ci]

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

* Bump version: 1.0.10 → 1.0.11 [skip ci]

* Replace H with h inside pandas' floor function (#64)

Thanks @0xFrama for this

* Bump version: 1.0.11 → 1.0.12 [skip ci]

* Added Contribution Guide (#65)

* added contribution guide

* added reference to coding style

* Bump version: 1.0.12 → 1.0.13 [skip ci]

* docs: add roshnaeem as a contributor for doc (#67)

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* Test set analysis (#60)

* test set analysis

* added link to nbviewer in notebook

* deleted checkpoints

* Delete checkpoints

* delete metadata.csv

* using metadata from hugging face

* remove unused import

* Bump version: 1.0.13 → 1.0.14 [skip ci]

* docs: add bikramb98 as a contributor for code (#68)

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* fetches live Enphase data at the moment it is called (#66)

* fetches live Enphase data at the moment it is called

replace 'user_enphase_api_key' and 'user_enphase_user_id' with the actual Enphase API key and user ID. The make_pv_data function now fetches live data from the Enphase API and combines it with the existing fake PV data
This "may" not be from the time of interference, do i add make use of timestamps to enable it?

* Added Flag indicating whether to use live Enphase data or not

Now the user will have an option whether to use live enphase data or the default data

* Updated site object to accomodate is_inverter option + created /inverters/enphase.py file

* added environment variables

* Bump version: 1.0.14 → 1.0.15 [skip ci]

* added init.py in invertors directory

* fixed the import of enphase_data

---------

Co-authored-by: BumpVersion Action <bumpversion@github-actions>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Peter Dudfield <34686298+peterdudfield@users.noreply.github.com>
Co-authored-by: Francesco <30957828+0xFrama@users.noreply.github.com>
Co-authored-by: Bikram Baruah <baruahbikram05@gmail.com>
Co-authored-by: Aryan Bhosale <36108149+aryanbhosale@users.noreply.github.com>
peterdudfield added a commit that referenced this pull request Mar 4, 2024
* Bump version: 1.0.9 → 1.0.10 [skip ci]

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

* Bump version: 1.0.10 → 1.0.11 [skip ci]

* Replace H with h inside pandas' floor function (#64)

Thanks @0xFrama for this

* Bump version: 1.0.11 → 1.0.12 [skip ci]

* Added Contribution Guide (#65)

* added contribution guide

* added reference to coding style

* Bump version: 1.0.12 → 1.0.13 [skip ci]

* docs: add roshnaeem as a contributor for doc (#67)

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

---------



* Test set analysis (#60)

* test set analysis

* added link to nbviewer in notebook

* deleted checkpoints

* Delete checkpoints

* delete metadata.csv

* using metadata from hugging face

* remove unused import

* Bump version: 1.0.13 → 1.0.14 [skip ci]

* docs: add bikramb98 as a contributor for code (#68)

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

---------



* fetches live Enphase data at the moment it is called (#66)

* fetches live Enphase data at the moment it is called

replace 'user_enphase_api_key' and 'user_enphase_user_id' with the actual Enphase API key and user ID. The make_pv_data function now fetches live data from the Enphase API and combines it with the existing fake PV data
This "may" not be from the time of interference, do i add make use of timestamps to enable it?

* Added Flag indicating whether to use live Enphase data or not

Now the user will have an option whether to use live enphase data or the default data

* Updated site object to accomodate is_inverter option + created /inverters/enphase.py file

* added environment variables

* Bump version: 1.0.14 → 1.0.15 [skip ci]

* added init.py in invertors directory

* fixed the import of enphase_data

---------

Co-authored-by: Rosheen Naeem <rosheennaeem4@gmail.com>
Co-authored-by: BumpVersion Action <bumpversion@github-actions>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Francesco <30957828+0xFrama@users.noreply.github.com>
Co-authored-by: Bikram Baruah <baruahbikram05@gmail.com>
Co-authored-by: Aryan Bhosale <36108149+aryanbhosale@users.noreply.github.com>
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.

update H to h
2 participants