Skip to content

Updates to MEWave report template#1399

Merged
mjprilliman merged 2 commits into
patchfrom
me-report-updates
May 19, 2023
Merged

Updates to MEWave report template#1399
mjprilliman merged 2 commits into
patchfrom
me-report-updates

Conversation

@mjprilliman

Copy link
Copy Markdown
Collaborator

Description

  1. For plant capacity have this round up (no decimal places)
  2. For annual energy production, have this round up (no decimal places)
  3. For the Capex and Opex plots round up to the nearest dollar (no decimal places)
  4. Add table with LCOE and FCR
    

Would be good to get this in patch 2 if possible

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • This change modifies variables in existing compute modules. Please see Checking for PySAM Incompatible API Changes.

Checklist:

If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

@mjprilliman mjprilliman added this to the 2022.11.21 Patch 2 milestone May 12, 2023
@mjprilliman mjprilliman requested a review from cpaulgilman May 12, 2023 15:55
@mjprilliman mjprilliman self-assigned this May 12, 2023

@cpaulgilman cpaulgilman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Economic Performance table is cut off at bottom of Page 2 for ME Tidal / LCOE Calculator configuration:

image

Consider adding a page and reorganizing tables:

Page 1:

  • Tidal Energy Converter / Wave Energy Converter table
  • For tidal: power curve and velocity distribution graphs

Page 2:

  • Array Design Parameters
  • System Performance

Page 3:

  • System Costs
  • Economic Performance

Page 4 (graphs):

  • Opex
  • Capex
  • LCOE

@mjprilliman

Copy link
Copy Markdown
Collaborator Author

@cpaulgilman I just pushed those suggested edits for the report so it now has four pages. My only concern is that there is not much content on page 2, but maybe that's fine.

@cpaulgilman cpaulgilman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks!

@mjprilliman mjprilliman merged commit a241fd9 into patch May 19, 2023
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

added to release notes PR and/or issue has been added to release notes for a public release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants