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.

Sign up for GitHub

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

Jump to bottom

460 multiple results #486

Merged
merged 20 commits into from
Nov 30, 2023
Merged

460 multiple results #486

merged 20 commits into from
Nov 30, 2023

Conversation

cdeline
Copy link
Contributor

@cdeline cdeline commented Nov 9, 2023

trackerdict[index][Results] updated to not over-write if you run analysis1axis multiple times. Fixes #460.

  • Code changes are covered by tests
  • Code changes have been evaluated for compatibility/integration with GUI
  • New functions added to __init__.py
  • manualapi.rst is up to date, along with other sphinx docs pages
  • Example notebooks are rerun and differences in results scrutinized
  • Updated changelog

@cdeline
Copy link
Contributor Author

cdeline commented Nov 9, 2023

Fixes #405 too.

shirubana and others added 18 commits November 16, 2023 07:25
@shirubana
jupyterbook
21cb1c9
@shirubana
cleanup HPC Scripts
4ae5f3a
@cdeline
new tests for addPiles
346aec1
@cdeline
minor pytest edit
69800b4
@cdeline
fix test_spectra.py
6e7fe0a
@cdeline
fix pandas 2.0 errors in development
67f1891
@cdeline
add NSRDBWeatherData pytest
0c0365b
@cdeline
Update whatsnew/pending.rst and manualapi.rst
d69b5f3
@cdeline
Merge pull request #494 from NREL/493_pytests
967e848 
493 pytests
@cdeline
Only re-initialize trackerdict[index][Results] if it's empty
6b0f9a5
@cdeline
add customname to keys of trackerdict['Results']
f636a1d
@cdeline
Merge branch '460_multipleResults' of https://github.com/NREL/bifacia…
bdee8a8 
…l_radiance into 460_multipleResults
@cdeline
Add 'columns' and 'methods' introspection to RadObj, MetObj and Groun…
d218698 
…dObj
@cdeline
Update pytests. Give GroundObj a __repr__.
f2b28ea
@cdeline
Merge pull request #495 from NREL/addColumns
2cb5032 
Add 'columns' and 'methods' introspection to RadObj, MetObj and GrounObj
@cdeline
Only re-initialize trackerdict[index][Results] if it's empty
8e22bd2
@cdeline
add customname to keys of trackerdict['Results']
6a670d9
@cdeline
Merge branch '460_multipleResults' of https://github.com/NREL/bifacia…
1dab269 
…l_radiance into 460_multipleResults
@cdeline cdeline merged commit 378da18 into development Nov 30, 2023
2 of 6 checks passed
@cdeline cdeline deleted the 460_multipleResults branch November 30, 2023 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Fixed in development branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cdeline @shirubana