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

add(capacity-parser) - Indian zones #6479

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

ghost
Copy link

@ghost ghost commented Feb 13, 2024

Issue

Indian capacity can be extracted from the CEA API
Renewable capacity still needs to be added manually from MNRE (it is only available in pdf)

Description

Add parser for conventional capacity -> CEA
Add function for REN capacity: the pdf can be converted to excel and then the data is extracted from the excel using the get_renewable_capacity function

Preview

Double check

  • I have tested my parser changes locally with poetry run test_parser "zone_key"
  • I have run pnpx prettier@2 --write . and poetry run format in the top level directory to format my changes.

@github-actions github-actions bot added python Pull requests that update Python code zone config Pull request or issue for zone configurations capacity parser labels Feb 13, 2024
@ghost ghost changed the title cea capacity parser + capacity update add(capacity-parser) - Indian zones Feb 13, 2024
@ghost ghost requested a review from unitrium February 14, 2024 09:05
@ghost ghost marked this pull request as ready for review February 14, 2024 09:05
@ghost ghost requested a review from VIKTORVAV99 February 20, 2024 14:39
Copy link
Member

@VIKTORVAV99 VIKTORVAV99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
capacity parser python Pull requests that update Python code zone config Pull request or issue for zone configurations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant