-
Notifications
You must be signed in to change notification settings - Fork 46
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'develop' into dev-refactor-develop
This was a bit tricky given that we removed many ECO files for the v2.0.0 release.
- Loading branch information
Showing
126 changed files
with
34,655 additions
and
180,537 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,50 @@ | ||
# Contributing | ||
|
||
We welcome contributions in the form of bug reports, bug fixes, improvements to the documentation, ideas for enhancements (or the enhancements themselves!). | ||
|
||
You can find a [list of current issues](https://github.com/NREL/HOPP/issues) in the project's GitHub repo. Feel free to tackle any existing bugs or enhancement ideas by submitting a [pull request](https://github.com/NREL/HOPP/pulls). | ||
|
||
## Bug Reports | ||
|
||
* Please include a short (but detailed) Python snippet or explanation for reproducing the problem. Attach or include a link to any input files that will be needed to reproduce the error. | ||
* Explain the behavior you expected, and how what you got differed. | ||
|
||
## Pull Requests | ||
|
||
* Please reference relevant GitHub issues in your commit message using `GH123` or `#123`. | ||
* Changes should be [PEP8](http://www.python.org/dev/peps/pep-0008/) compatible. | ||
* Keep style fixes to a separate commit to make your pull request more readable. | ||
* Docstrings are required and should follow the [Google style](https://www.sphinx-doc.org/en/master/usage/extensions/example_google.html). | ||
* When you start working on a pull request, start by creating a new branch pointing at the latest commit on [main](https://github.com/NREL/HOPP). | ||
* The HOPP copyright policy is detailed in the [`LICENSE`](https://github.com/NREL/HOPP/blob/main/LICENSE). | ||
|
||
## Documentation | ||
|
||
All newly introduced code should be documented in Google format as described in the previous section. To generate the docs: | ||
|
||
``` | ||
cd docs | ||
make html | ||
``` | ||
|
||
Then open the docs in your browser: | ||
|
||
``` | ||
open _build/html/index.html | ||
``` | ||
|
||
## Tests | ||
|
||
The test suite can be run using `pytest tests/hopp`. Individual test files can be run by specifying them: | ||
|
||
``` | ||
pytest tests/hopp/test_hybrid.py | ||
``` | ||
|
||
and individual tests can be run within those files | ||
|
||
``` | ||
pytest tests/hopp/test_hybrid.py::test_hybrid_wind_only | ||
``` | ||
|
||
When you push to your fork, or open a PR, your tests will be run against the [Continuous Integration (CI)](https://github.com/NREL/HOPP/actions) suite. This will start a build that runs all tests on your branch against multiple Python versions, and will also test documentation builds. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
.. _BatteryStorage: | ||
|
||
|
||
Battery Storage | ||
================================ | ||
|
||
Battery Storage class based on PySAM's BatteryStateful Model | ||
|
||
|
||
.. autoclass:: hopp.simulation.technologies.battery.battery.Battery | ||
:members: | ||
|
||
.. autoclass:: hopp.simulation.technologies.battery.battery.BatteryConfig | ||
:members: | ||
|
||
.. autoclass:: hopp.simulation.technologies.battery.battery.BatteryOutputs | ||
:members: |
17 changes: 17 additions & 0 deletions
17
docs/hopp/simulation/technologies/battery/battery_stateless.rst
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
.. _BatteryStorageStateless: | ||
|
||
|
||
Battery Storage (Stateless) | ||
================================ | ||
|
||
Battery Storage class with no system model for tracking the state of the battery. | ||
|
||
|
||
.. autoclass:: hopp.simulation.technologies.battery.battery_stateless.BatteryStateless | ||
:members: | ||
|
||
.. autoclass:: hopp.simulation.technologies.battery.battery_stateless.BatteryStatelessConfig | ||
:members: | ||
|
||
.. autoclass:: hopp.simulation.technologies.battery.battery_stateless.BatteryStatelessOutputs | ||
:members: |
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
.. _CSPPlant: | ||
|
||
|
||
Concentrating Solar Power (CSP) plant | ||
===================================== | ||
|
||
Abstract base class for CSP generation technologies that contains shared methods. | ||
|
||
.. autoclass:: hopp.simulation.technologies.csp.csp_plant.CspPlant | ||
:members: | ||
|
||
.. autoclass:: hopp.simulation.technologies.csp.csp_plant.CspConfig | ||
:members: | ||
|
||
.. autoclass:: hopp.simulation.technologies.csp.csp_plant.CspOutputs | ||
:members: | ||
:undoc-members: |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
.. _TroughSource: | ||
|
||
|
||
Parabolic Trough Plant | ||
============================= | ||
|
||
Parabolic trough concentrating solar power class based on SSC's Parabolic trough - physical model | ||
|
||
.. autoclass:: hopp.simulation.technologies.csp.trough_plant.TroughPlant | ||
:members: | ||
:undoc-members: | ||
|
||
.. autoclass:: hopp.simulation.technologies.csp.trough_plant.TroughConfig | ||
:members: | ||
:undoc-members: |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.