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

Fixed code errors #1371

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

Fixed code errors #1371

wants to merge 6 commits into from

Conversation

Eeonum
Copy link

@Eeonum Eeonum commented Apr 9, 2024

diagPrices changed to diag_prices in rollcalendars_from_arcticprices_to_csv.py and multiple_and_adjusted_from_csv_to_arctic.py

@tgibson11
Copy link
Contributor

This pull request includes several other changes from your fork that you probably didn't intend to include. You'll need remove them before this can be merged.

Also, are you sure this should be merged into master rather than develop? The general idea is that most changes will be done in develop first, and develop will eventually be merged into master when it is considered to be reasonably stable.

@Eeonum
Copy link
Author

Eeonum commented Apr 9, 2024

It should be part of the master's as this is final. I tried to fix the redundancies, but can you remove adding the CSV files on your end if it didn't work? The rest should be added to the master.

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.

None yet

2 participants