Add loading wind power and India PV Power

[jacobbieker]

Pull Request

Description

Adds loading wind power, such as for India. Also adds a test for loading India PV power

Fixes #

How Has This Been Tested?

Unit Tests

• [x] Yes

Checklist:

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

[codecov[bot]]

Codecov Report

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

Comparison is base (e86aabf) 77.51% compared to head (0972bb7) 77.99%.

Files	Patch %	Lines
ocf_datapipes/load/wind/utils.py	90.90%	1 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #260 +/- ## ========================================== + Coverage 77.51% 77.99% +0.47% ========================================== Files 118 120 +2 Lines 5364 5457 +93 ========================================== + Hits 4158 4256 +98 + Misses 1206 1201 -5 ```

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

[jacobbieker]

Thanks! Yeah, for the failing test, in discussion here https://github.com/openclimatefix/india-forecast-app/issues/2 as to what to do about it, as its from nowcasting_datamodel's check of PV data

[peterdudfield]

Could you keep the validator there for PV, but not for Wind?

[jacobbieker]

The validator isn't running for Wind, but is for PV. We can add it back for PV, but we probably should put the definition of the names in datapipes, not in datamodel. As of right now, the validator validates passiv and pvoutput.org from nowcasting_datamodels, and adds SV here. So I can re-add it, but would like to remove the dependency on nowcasting_datamodel

[peterdudfield]

The validator isn't running for Wind, but is for PV. We can add it back for PV, but we probably should put the definition of the names in datapipes, not in datamodel. As of right now, the validator validates passiv and pvoutput.org from nowcasting_datamodels, and adds SV here. So I can re-add it, but would like to remove the dependency on nowcasting_datamodel

I agree, yea nice