Merged
Conversation
Codecov Report
@@ Coverage Diff @@
## main #890 +/- ##
==========================================
- Coverage 69.50% 69.49% -0.01%
==========================================
Files 578 578
Lines 64859 64859
Branches 12064 12064
==========================================
- Hits 45080 45077 -3
- Misses 17508 17512 +4
+ Partials 2271 2270 -1
Continue to review full report at Codecov.
|
bpaul4
added a commit
to MAZamarripa/idaes-pse
that referenced
this pull request
Jul 18, 2022
* Sphinx Documentation Build Issue (IDAES#890) * switch lang flag to 'en' * Flag formatting * Update path reference * updating power plant capcost and QGESS tests to run as expected * bring over MEA_CCS updates and tests * running black on costing/ directory * move custom dicts to new file * fix some spacing causing doc build errors * Replace method calls with UnitModelCostingBlocks * Running black on the entire repo (IDAES#894) * Running black on the entire repo * Add pinned requirement for Black stable version * Add CI check to ensure code is Black-formatted Co-authored-by: Ludovico Bianchi <[email protected]> * Fix PySMO tests (IDAES#903) * Minor change to trigger tests * Added abs to tolerance check * Run black * Remove string comparison * Tighten tolerance * Fixed kriging test by removing variable dependence * bring in report method updates * bring in more MEA CCS updates * Add check on total TPC * final formatting fixes * Update report method Adding cost of electricity and cost of capture to report method * Black reformatting * Update report method Fixing units of cost of electricity Co-authored-by: Keith Beattie <[email protected]> Co-authored-by: Ludovico Bianchi <[email protected]> Co-authored-by: OOAmusat <[email protected]> Co-authored-by: Daison2102 <[email protected]>
andrewlee94
pushed a commit
that referenced
this pull request
Oct 14, 2022
* updating power plant costing to new costing framework * updating power plant capcost and QGESS tests to run as expected * bring over MEA_CCS updates and tests * running black on costing/ directory * move custom dicts to new file * fix some spacing causing doc build errors * Replace method calls with UnitModelCostingBlocks * bring in report method updates * bring in more MEA CCS updates * Add check on total TPC * final formatting fixes * Update report method Adding cost of electricity and cost of capture to report method * Black reformatting * Update report method Fixing units of cost of electricity * Fixing test issues, bringing in MEA_CCS code (#1) * Sphinx Documentation Build Issue (#890) * switch lang flag to 'en' * Flag formatting * Update path reference * updating power plant capcost and QGESS tests to run as expected * bring over MEA_CCS updates and tests * running black on costing/ directory * move custom dicts to new file * fix some spacing causing doc build errors * Replace method calls with UnitModelCostingBlocks * Running black on the entire repo (#894) * Running black on the entire repo * Add pinned requirement for Black stable version * Add CI check to ensure code is Black-formatted Co-authored-by: Ludovico Bianchi <[email protected]> * Fix PySMO tests (#903) * Minor change to trigger tests * Added abs to tolerance check * Run black * Remove string comparison * Tighten tolerance * Fixed kriging test by removing variable dependence * bring in report method updates * bring in more MEA CCS updates * Add check on total TPC * final formatting fixes * Update report method Adding cost of electricity and cost of capture to report method * Black reformatting * Update report method Fixing units of cost of electricity Co-authored-by: Keith Beattie <[email protected]> Co-authored-by: Ludovico Bianchi <[email protected]> Co-authored-by: OOAmusat <[email protected]> Co-authored-by: Daison2102 <[email protected]> * costing updates * costing updates * improving generic variable O&M costs calculation * Updating currency and basis unit handling, other minor fixes * Zipping together costing dictionaries * Improvements from review comments * Updating tests for new changes * Updating author tags * Removing QGESS tests, updating NGCC tests, adding capcost tests, param unit fixes * Sort author list * General improvements and cost variable unit corrections * Removing deprecated default arguments * Addressing most review comments * Addressing more review comments * Testing and coverage improvements * Resolving pylint and directory reference issues * Separating power plant currency units * cleaning up comments * Final fixes from review comments * Add some checks to json regeneration test Co-authored-by: Miguel Zamarripa-Perez <[email protected]> Co-authored-by: Daison2102 <[email protected]> Co-authored-by: Keith Beattie <[email protected]> Co-authored-by: Ludovico Bianchi <[email protected]> Co-authored-by: OOAmusat <[email protected]>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes
142 WARNING: Invalid configuration value found: 'language = None'. Update your configuration to a valid langauge code. Falling back to 'en' (English).... long list of deprecated import path warnings (see Local Documentation Build Warnings #883 ) ...
872 /home/runner/work/idaes-pse/idaes-pse/docs/tutorials/tutorials_examples.rst:3: WARNING: unknown document: howto/indexNonewas removed as a supported flag for thelanguageentry. This update switches the flag to 'en' (the new default) to remove the first warning and fixes the path reference in the second warning.Summary/Motivation:
Changes proposed in this PR:
Legal Acknowledgement
By contributing to this software project, I agree to the following terms and conditions for my contribution: