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

JEDI project skeleton #1

Conversation

softwareengineerprogrammer

Project skeleton for NREL#169

@aimiktena Merge this PR into your fork, and then you will be unblocked for next steps to implement! (We can discuss what those steps are offline.)

malcolm-dsider and others added 30 commits April 29, 2024 12:54
Deleting accidentally added files
Deleting accidentally added files
Deleting accidentally added files
Deleting accidentally added files
Deleting accidentally added files
Deleting accidentally added files
…he apparent reason the clases in question weren't included is that they have no unique parameters, and so this change doesn't actually affect docs generation, but it's still good to have in case parameters are added in the future.
…rrent outputs. Fervo results will continue to evolve.
…ams now created. Also addressed issue where multiple processes were writing to the output file at the same time and corrupting parts of it using File Locking. The result is slower, but the results are better.
…t-parameters-enhancement

GeophiresInputParameters enhancement + FCR sensitivity test
added pylocker needed for MC file locking
Calling plt.close() to free memory
@softwareengineerprogrammer
Copy link
Author

Disregard, @aimiktena needs to sync their fork with main first. Once done, I will file a separate PR.

@aimiktena
Copy link
Owner

Hi @softwareengineerprogrammer,
I've just updated my branch to match the original repository as requested. Looking forward to the next steps!
Thank you.

@softwareengineerprogrammer
Copy link
Author

@aimiktena great - see #2

@softwareengineerprogrammer softwareengineerprogrammer deleted the jedi-geothermal branch May 14, 2024 21:11
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.

3 participants