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

Generate files for regional scaling factors for urban and rural areas #344

Merged
merged 8 commits into from
Feb 10, 2023

Conversation

rouille
Copy link
Collaborator

@rouille rouille commented Feb 9, 2023

Pull Request doc

Purpose

Remove outdated regional scaling factors files and generate new ones using code implemented in #333, #335, #337 and #342. Also the fuel efficiency file has been updated

What the code is doing

Use if __name__ == '__main__' code block in the generate_scaling_factors to be able to generate the regional scaling factors files for urban and rural areas for all 32 years (64 files) in the top level environment. Also update the generate_BEV_vehicle_profiles module to use state abbreviation in both urban and rural and update tests following changes in fuel efficiency file.

Testing

Existing unit tests

Where to look

Very little code. Just using existing code in `if name == 'main' code block to write files on disk

Usage Example/Visuals

N/A

Time estimate

10min

@rouille rouille self-assigned this Feb 9, 2023
Copy link
Collaborator

@BainanXia BainanXia left a comment

Choose a reason for hiding this comment

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

Very nice. Thanks.

Copy link
Collaborator

@dmuldrew dmuldrew left a comment

Choose a reason for hiding this comment

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

🚢

Copy link

@danlivengood danlivengood left a comment

Choose a reason for hiding this comment

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

Looks good to me! (Adding my approval in case Ben's waiting for that to merge)

@rouille rouille merged commit f9eb59c into transportation_electrification Feb 10, 2023
@rouille rouille deleted the ben/files branch February 10, 2023 00:26
rouille added a commit that referenced this pull request Feb 24, 2023
Generate regional scaling factors files for urban and rural areas
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.

4 participants