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

Input type fix for export_rate_beyond_net_metering_limit #371

Merged
merged 8 commits into from
Apr 10, 2024

Conversation

adfarth
Copy link
Collaborator

@adfarth adfarth commented Apr 5, 2024

  • Added export_rate_beyond_net_metering_limit to list of inputs to be converted to type Real, to avoid MethodError if type is vector of Any.
  • This was previously causing an error with posts sent from the API with this input

@adfarth adfarth requested a review from rathod-b April 5, 2024 18:21
@adfarth adfarth linked an issue Apr 5, 2024 that may be closed by this pull request
@adfarth adfarth marked this pull request as ready for review April 5, 2024 20:56
@rathod-b
Copy link
Collaborator

rathod-b commented Apr 9, 2024

Did the highstests (linux) just time out in 360 minutes?

@hdunham hdunham merged commit 22fdbb9 into develop Apr 10, 2024
7 of 10 checks passed
@hdunham hdunham mentioned this pull request Apr 10, 2024
indu-manogaran pushed a commit that referenced this pull request Sep 16, 2024
Input type fix for export_rate_beyond_net_metering_limit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix validation of export_rate_beyond_net_metering_limit
6 participants