Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

johnjasa

johnjasa pushed 5 commits to feature/clean_up_type_dec bayc/HOPP-1
  • update the api url for the nsrdb 517e156
  • correct naming of the test pypi workflow 644bd37
  • Merge pull request #381 from bayc/feature/update_NSRDB_API_URL Update the API URL for the NSRDB e893987
  • Merge pull request #384 from RHammond2/fix/action-naming Bug Fix: Correct the display name of the Test PyPI Action 15b63a5
  • Merge branch 'develop' into feature/clean_up_type_dec 5684a28

View on GitHub

johnjasa created a comment on a pull request on NREL/GreenHEART
Files changed shows an example modified but not imports? I know this is a draft PR, maybe you have more changes coming!

View on GitHub

johnjasa created a review on a pull request on NREL/HOPP

View on GitHub

johnjasa pushed 9 commits to develop NREL/GreenHEART
  • Merge pull request #9 from NREL/develop v0.1 Release 7d5d19f
  • update floris input files to v4 format 2e2a5a9
  • update floris input files to v4 format fc97979
  • Merge branch 'bugfix/fix_CI' of https://github.com/bayc/GreenHEART into bugfix/fix_CI d1591e6
  • update floris "reinitialize" method to "set" method for floris v4 072a4d9
  • fix resource paths for greensteel tests fb2923d
  • bump the version to v0.1.1 4912b22
  • Merge pull request #17 from bayc/bugfix/fix_CI Fix the CI for the new GreenHEART repo 52773c4
  • Merge pull request #21 from NREL/main Backmerging main into develop a2d41eb

View on GitHub

johnjasa closed a pull request on NREL/GreenHEART
Backmerging main into develop
As Chris noted in #17, we'll need to backmerge this into develop. This PR does that.
johnjasa opened a pull request on NREL/GreenHEART
Backmerging main into develop
As Chris noted in #17, we'll need to backmerge this into develop. This PR does that.
johnjasa closed an issue on NREL/GreenHEART
Fix the CI tests
The tests are failing after splitting out the GreenHEART repo from within the HOPP repo and need to be fixed.
johnjasa pushed 7 commits to main NREL/GreenHEART
  • update floris input files to v4 format 2e2a5a9
  • update floris input files to v4 format fc97979
  • Merge branch 'bugfix/fix_CI' of https://github.com/bayc/GreenHEART into bugfix/fix_CI d1591e6
  • update floris "reinitialize" method to "set" method for floris v4 072a4d9
  • fix resource paths for greensteel tests fb2923d
  • bump the version to v0.1.1 4912b22
  • Merge pull request #17 from bayc/bugfix/fix_CI Fix the CI for the new GreenHEART repo 52773c4

View on GitHub

johnjasa closed a pull request on NREL/GreenHEART
Fix the CI for the new GreenHEART repo
## Fix the CI for the new GreenHEART repo <!-- Be sure to title your pull request so that readers can scan through the list of PR's and understand what this one involves. It should be a few well...
johnjasa created a review on a pull request on NREL/GreenHEART
Thanks for these changes, looks good to me as well!

View on GitHub

johnjasa pushed 5 commits to feature/remove_unused_dependencies bayc/HOPP-1
  • update the api url for the nsrdb 517e156
  • correct naming of the test pypi workflow 644bd37
  • Merge pull request #381 from bayc/feature/update_NSRDB_API_URL Update the API URL for the NSRDB e893987
  • Merge pull request #384 from RHammond2/fix/action-naming Bug Fix: Correct the display name of the Test PyPI Action 15b63a5
  • Merge branch 'develop' into feature/remove_unused_dependencies 29ea2d6

View on GitHub

johnjasa pushed 2 commits to develop NREL/HOPP
  • correct naming of the test pypi workflow 644bd37
  • Merge pull request #384 from RHammond2/fix/action-naming Bug Fix: Correct the display name of the Test PyPI Action 15b63a5

View on GitHub

johnjasa closed a pull request on NREL/HOPP
Bug Fix: Correct the display name of the Test PyPI Action
This PR is a minor fix to the Test PyPI uploader GitHub Action to ensure the displayed name in the Actions tab shows as expected. Previously the PyPI and Test PyPI display name were mistakenly the ...
johnjasa pushed 2 commits to develop NREL/HOPP
  • update the api url for the nsrdb 517e156
  • Merge pull request #381 from bayc/feature/update_NSRDB_API_URL Update the API URL for the NSRDB e893987

View on GitHub

johnjasa closed a pull request on NREL/HOPP
Update the API URL for the NSRDB
Update the API URL for the NSRDB <!-- Be sure to title your pull request so that readers can scan through the list of PR's and understand what this one involves. It should be a few well selected...
johnjasa created a review on a pull request on NREL/HOPP

View on GitHub

johnjasa created a review comment on a pull request on NREL/HOPP
Very nice, love this addition! Slick solution

View on GitHub

johnjasa created a review comment on a pull request on NREL/HOPP
I guess this might already answer my prior question re: the excel sheets included in this PR?

View on GitHub

johnjasa created a review comment on a pull request on NREL/HOPP
I'm assuming these files are meaningfully different than those other two? If not, we should get rid of one set.

View on GitHub

johnjasa created a review comment on a pull request on NREL/HOPP
This is a large enough file that I'm slightly hesitant about just adding it in. Would it make sense to instead ask users to download this and place it here if they want to use it? Or maybe have it ...

View on GitHub

johnjasa created a review on a pull request on NREL/HOPP
Partial review, I'll dig in more later on! Great work, love to see this coming in.

View on GitHub

johnjasa created a review on a pull request on NREL/HOPP
Partial review, I'll dig in more later on! Great work, love to see this coming in.

View on GitHub

johnjasa pushed 11 commits to hopp_lca_pr dakotaramos/HOPP
  • make battery heuristic general so other generation tech besides wind and pv can be used 56a1fc0
  • remove outdated dir 15e6800
  • Merge pull request #378 from kbrunik/wavvy remove duplicate outdated dir 3c86578
  • Merge branch 'develop' into feature/generalize-dispatch 4099b33
  • add test for load following heuristic with wave 15d7d28
  • Merge remote-tracking branch 'myfork/feature/generalize-dispatch' into feature/generalize-dispatch b57173e
  • simplify battery_heuristic() gen sum 22d25d8
  • initialize tot_gen as numpy array e58bc22
  • remove extra '=' 410f97b
  • Merge pull request #377 from jaredthomas68/feature/generalize-dispatch make battery heuristic general so other gener... 08e3175
  • Merge branch 'develop' into hopp_lca_pr 5e0b99b

View on GitHub

johnjasa opened an issue on NREL/GreenHEART
Add a developer guide to the docs
Once #15 is achieved, we should add a dev guide to make it clear how we want developers to add code, docs, and tests. We can draw from existing dev guides, like FLORIS: https://nrel.github.io/flor...
johnjasa created a comment on a pull request on NREL/HOPP
I might be missing it, but are there meaningful changes to the warning messages here? just looks like formatting changes

View on GitHub

johnjasa created a comment on an issue on NREL/GreenHEART
This came up in a meeting where @RHammond2 originally noting they'd want to update some parts; not sure which! Will assign Rob.

View on GitHub

johnjasa created a comment on an issue on NREL/HOPP
Yes, it'd be updating [this file](https://github.com/NREL/HOPP/blob/main/RELEASE.md) with v2.0, v3.0, etc change details.

View on GitHub

johnjasa pushed 1 commit to add_soec johnjasa/HOPP
  • Removed tools that weren't updated for different electrolyzer types yet a8d2df4

View on GitHub

johnjasa pushed 2 commits to add_soec johnjasa/HOPP
  • Minor note addition and cleanup 21bcc5f
  • Added more generic electrolyzer tools 733a669

View on GitHub

johnjasa pushed 3 commits to feature/generalize-dispatch jaredthomas68/HOPP
  • remove outdated dir 15e6800
  • Merge pull request #378 from kbrunik/wavvy remove duplicate outdated dir 3c86578
  • Merge branch 'develop' into feature/generalize-dispatch 4099b33

View on GitHub

Load more