Skip to content

Conversation

@camirmas
Copy link
Contributor

@camirmas camirmas commented Aug 30, 2023

Feature or improvement description

  • Fix a bug with the isort pre-commit hook
  • Add an example on conversion efficiency
  • Enhanced degradation calculations
  • Added alkaline model
  • Reformatted docs to use jupyterbook
  • Added new repo structure focused on generalizing models for future work

Related issue, if one exists

Impacted areas of the software

Additional supporting information

Test results, if applicable

bayc and others added 28 commits October 18, 2023 15:53
Updating dryer loss and membrane thickness
Adds alkaline modeling capabilities and tests.
Reorganize file structure for BERT
* removed setup and added to pyproject

* Updated CI workflow

* WIP: Added CI and docs build

* Added jupyterbook docs

* Updated readme

* change CI to install all
* reformatted

* updated PEM schema and removed LCOH tests temporarily

* updated alkaline modeling schema and alkaline Urev0 calc

* update lcoh glue_code test regression values

* added hydrogen loss from degradation and input toggle to choose that option

* added LCA functions to stack.py

* added eol_eff_percent_loss as degradation input for stack

* added run_LCA functionto run_electrolyzer script

* added more tracking variables to stack

* updated all py file import paths

* updated jupyter notebook import paths

* added validators and minor fix in stack for cell type

* added skeleton for cluster and cell classes

* added generic base class

* added example skeleton for simulation config

* added file for power-curve function templates

* updated inputs for Stack in test_stack

* added import to fix tests to stack.py

* removed empty PEM cell test

* renamed test_cell to test_PEM_cell

* updated validator format

* removed files unrelated to primary feature-adds

* reformatted bert.py

* added back-in placeholder files

* removed alkaline sources in alkaline model into md file

* Typographical changes

* minor cleanups to alkaline cell and stack

* removed commented out code and added docstrings

* fixed aep datadrame in bert.py

* added more docstrings to stack.py

* added comments to pem.py for faradaic coefficients

---------

Co-authored-by: bayc <[email protected]>
Co-authored-by: John Jasa <[email protected]>
@johnjasa johnjasa merged commit 1fa349b into main Nov 7, 2025
6 checks passed
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.

7 participants