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

Price and inflexible device sensors data to connectivity table on asset status page #1102

Merged
merged 4 commits into from
Jun 24, 2024

Conversation

nrozanov
Copy link
Contributor

@nrozanov nrozanov commented Jun 20, 2024

Description

Start adding consumption and production price sensors + inflexible device sensors linked to an asset to asset status page

Look & Feel

On asset status page there is a new column "Relation to asset" that shows if an asset owns this sensor, or this sensor is linked to asset via consumption / production_price_sensor_id or inflexible_device_sensors

How to test

Open status page for some asset without linked price / inflexible device sensors.
Verify that "Data connectivity" table contains only this asset sensors + their relation to an asset is "included device".
Update an asset - e.g set some of its sensors as a consumption price sensor. Check that this sensor ownership on a status page now equals to "ownership; consumption price".
Update an asset - add some other asset sensor as its e.g inflexible device sensor. Check that this sensor appears on asset status page with "inflexible device" ownership

Further Improvements

Related Items


  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on code under GPL or other license that is incompatible with FlexMeasures

…et status page

Signed-off-by: Nikolai <nrozanov@iponweb.net>
@nrozanov nrozanov requested a review from nhoening June 20, 2024 15:25
@nrozanov nrozanov linked an issue Jun 20, 2024 that may be closed by this pull request
Signed-off-by: Nikolai <nrozanov@iponweb.net>
Copy link
Contributor

@nhoening nhoening left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is good work and we're almost there :)

flexmeasures/data/models/generic_assets.py Show resolved Hide resolved
flexmeasures/data/models/generic_assets.py Show resolved Hide resolved
flexmeasures/data/services/sensors.py Outdated Show resolved Hide resolved
Nikolai added 2 commits June 24, 2024 12:58
Signed-off-by: Nikolai <nrozanov@iponweb.net>
Signed-off-by: Nikolai <nrozanov@iponweb.net>
Copy link
Contributor

@nhoening nhoening left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@nhoening nhoening merged commit 74dbc18 into main Jun 24, 2024
9 checks passed
@nhoening nhoening deleted the 1080-flex-context-sensors-to-status branch June 24, 2024 11:02
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.

Add flex-context sensors to status page
2 participants