Skip to content

Fix monthly load and energy plots#1394

Merged
cpaulgilman merged 2 commits into
patchfrom
report-energy-graphs
May 23, 2023
Merged

Fix monthly load and energy plots#1394
cpaulgilman merged 2 commits into
patchfrom
report-energy-graphs

Conversation

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Description

Fix monthly energy graphs on Page 2 of PV report template.

  • Show data in kWh rather than trying to scale to avoid issue with exponential values

SAM file for testing: pdf-report-large-load-test.zip

Fixes #1351

Type of change

  • 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

- Show data in kWh rather than trying to scale to avoid issue with exponential values
@cpaulgilman cpaulgilman added this to the 2022.11.21 Patch 2 milestone Apr 21, 2023
@cpaulgilman cpaulgilman requested a review from mjprilliman April 21, 2023 22:42
@cpaulgilman cpaulgilman self-assigned this Apr 21, 2023
@cpaulgilman cpaulgilman merged commit d940029 into patch May 23, 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
@cpaulgilman cpaulgilman deleted the report-energy-graphs branch October 3, 2023 16:48
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.

PDF report for BTM graph scales load data incorrectly

2 participants