Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REopt API call fails with callback error #1830

Closed
cpaulgilman opened this issue Aug 2, 2024 · 4 comments · Fixed by #1836 or #1904
Closed

REopt API call fails with callback error #1830

cpaulgilman opened this issue Aug 2, 2024 · 4 comments · Fixed by #1836 or #1904
Assignees
Labels

Comments

@cpaulgilman
Copy link
Collaborator

cpaulgilman commented Aug 2, 2024

Describe the bug

REopt API call fails.

To Reproduce

  1. Create a default PV Battery / Commercial case.
  2. On Battery Cell and System page, click Get size and dispatch.
  3. See error.
Could not evaluate callback function:call_reopt->on_change
[71]: error in call to 'reopt_size_battery()': access violation: expected hash table, but found null[12]: eval error in statement list

Operating System and Version

  • macOS Sonoma 14.6
  • SAM 2023.12.17 r2 release candidate]
@cpaulgilman cpaulgilman added the bug label Aug 2, 2024
@brtietz brtietz added this to the 2023 Release Patch 2 milestone Aug 5, 2024
@sjanzou sjanzou linked a pull request Aug 8, 2024 that will close this issue
sjanzou added a commit that referenced this issue Aug 9, 2024
@cpaulgilman
Copy link
Collaborator Author

As of the latest Patch branch (8/12/2024), we are seeing this error on macOS, but not Windows or Linux.

@brtietz
Copy link
Collaborator

brtietz commented Sep 20, 2024

Closing since we merged PRs related to this for the patch. @sjanzou and @cpaulgilman please re-open if I've forgotten something that needs to be resolved on our end for this issue.

@brtietz brtietz closed this as completed Sep 20, 2024
@sjanzou
Copy link
Collaborator

sjanzou commented Sep 22, 2024

Closing since we merged PRs related to this for the patch. @sjanzou and @cpaulgilman please re-open if I've forgotten something that needs to be resolved on our end for this issue.

I thinks everything works fine with postman but, for curl, the issue remains and the pull request does have the REopt call for loads if the original call fails.

Finally, the REopt team has been made aware of the problem and SAM will work with any non-API breaking fixes that are donw by the REopt team for curl calls.

@cpaulgilman
Copy link
Collaborator Author

Still getting this callback error in SAM built from develop 10/30/24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment