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

Addition to powergrid model api to insert measurements #1589

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

tdtalbot
Copy link
Contributor

Description

Added methods to the API to automatically insert measurements
To test this, send a request to the topic goss.gridappsd.process.request.data.powergridmodel

with the json:
{
"requestType": "INSERT_ALL_MEASUREMENTS",
"resultFormat": "JSON"
}

or:
{
"requestType": "DROP_ALL_MEASUREMENTS",
"resultFormat": "JSON"
}

We should probably have a conversation about how to set up some automated tests for this. At this time I only allow users to insert or drop measurements for all model through the api, but could add model specific calls as well if it is wanted.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • [ X] 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

@poorva1209 poorva1209 removed their request for review February 3, 2022 19:00
@poorva1209 poorva1209 removed their assignment Feb 3, 2022
@poorva1209 poorva1209 requested review from poorva1209 and removed request for temcdrm, tonya1 and poorva1209 February 3, 2022 19:01
@poorva1209 poorva1209 assigned tdtalbot and unassigned temcdrm and tonya1 Feb 3, 2022
@poorva1209 poorva1209 requested review from temcdrm and tonya1 February 3, 2022 20:28
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.

4 participants