-
Notifications
You must be signed in to change notification settings - Fork 70
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
Provide a list of points to /results #398
Comments
We may also consider similarly for forecast data when addressing #356. |
@dhblum would you mind create the branch issue398_ListOfPointsResultsAPI so I can make a pull request? All the unit tests passed on bestest_air and bestest_hydronic. However, there are still a couple of things to check before the pull request is completely ready:
|
Thanks @EttoreZ. Branch created here https://github.com/ibpsa/project1-boptest/tree/issue398_ListOfPointsResultsAPI. Make the PR and unit tests will check bullets 1 and 2. 3 shouldn't be too hard to add if I have to, and I can review comments/docs. |
Closed by #480. |
This issue is to suggest making the ability to provide a list of points to the /results API call, instead of requiring one at a time. The return would be the same object, just with additional arrays of point data.
While I acknowledge the potential memory issues of calling for large amounts of data (e.g. #240), this would reduce the need for excessive network requests (say in a long for-loop), which could otherwise bottleneck the system. And, the user would have control over the amount of data returned at one time if memory becomes too much an issue.
The text was updated successfully, but these errors were encountered: