Skip to content

Sixparsolve error reporting#827

Merged
cpaulgilman merged 4 commits into
patchfrom
sixparsolve-error-reporting
May 30, 2022
Merged

Sixparsolve error reporting#827
cpaulgilman merged 4 commits into
patchfrom
sixparsolve-error-reporting

Conversation

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Update #802 to fix uninitialized variable error.

To test:

  1. Create a Detailed PV / No Financing case.
  2. On the module page, switch to CEC Performance Model with User Entered Specifications.
  3. Change short circuit current to 25 Amps
  4. Click Calculate and plot: "Sanity check failed" error with details should appear.
  5. Click Simulate: "Could not solve for normalized coefficients" exec fail message should appear.

@cpaulgilman cpaulgilman added this to the 2021.12.02 Patch 2 milestone May 29, 2022
@cpaulgilman cpaulgilman requested a review from sjanzou May 29, 2022 20:18
@cpaulgilman cpaulgilman self-assigned this May 29, 2022

@sjanzou sjanzou 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.

Performing test in pull request works as described! Great update
image
image

@cpaulgilman cpaulgilman merged commit c1e49d0 into patch May 30, 2022
@cpaulgilman cpaulgilman deleted the sixparsolve-error-reporting branch May 30, 2022 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants