Skip to content

Conversation

@lambaaryan011
Copy link

Pull Request

Description

This pull request adds functionality to fetch Physical Notification (PN) data for wind farms using the Elexon API. The script dynamically queries BM Unit data from the API and filters for wind farms based on the bm_unit column. It also retrieves PN data for the selected wind farm over a specified time range.

Changes:
⇾ Added a script (examples/wind_farm_pn.py) to fetch PN data for wind farms.

⇾ Integrated Elexon API (balancing_physical_all_get and balancing_physical_get methods).

⇾ Implemented filtering logic to identify wind farms dynamically from the API response.

Fixes #60

How Has This Been Tested?

The script has been tested locally by:

⇾ Fetching BM Unit data from the Elexon API.

⇾ Filtering BM Units dynamically for wind farms.

⇾ Retrieving PN data for a selected wind farm over a 24-hour period.

  • Yes

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

  • 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

Signed-off-by: aryan lamba <aryanlamba54@gmail.com>
@lambaaryan011
Copy link
Author

Hi @peterdudfield, i complete the issue so have a look if there is any problem so let me know

@peterdudfield
Copy link
Contributor

COuld you add a screen shot of the output when running the script

@lambaaryan011
Copy link
Author

Screenshot 2025-03-20 at 5 49 22 PM

@peterdudfield, may be this is helpful

@peterdudfield
Copy link
Contributor

Do you think its correct that no wind farms were found?

@lambaaryan011
Copy link
Author

Screenshot 2025-03-21 at 3 38 08 PM Screenshot 2025-03-21 at 3 38 24 PM

@peterdudfield I think know this provide the correct information that issue demand. If something is wrong, so let me know.

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.

Wind PN example

2 participants