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

Add 'max devices' config to Energy Devices Graph #17553

Conversation

thomaskonrad
Copy link
Contributor

@thomaskonrad thomaskonrad commented Aug 11, 2023

Proposed change

I'd like to propose adding an optional max_devices config setting to the Devices Energy Graph. If the setting is present, the card will only list the top N devices instead of all devices.

The use case is that users might want to only show the N biggest energy consumers in their home without occupying too much space on the dashboard.

Screenshot 2023-08-11 at 08 53 12

Note that this change does not change the current behaviour. If the config setting is not present, it will list all devices.

Docs PR: home-assistant/home-assistant.io#29255

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

type: energy-devices-graph
max_devices: 3

Additional information

N/A

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@home-assistant

This comment was marked as resolved.

Copy link
Member

@bramkragten bramkragten left a comment

Choose a reason for hiding this comment

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

This can be merged when documentation for this feature is added.

@frenck
Copy link
Member

frenck commented Oct 9, 2023

Added a documentation PR.

@bramkragten bramkragten merged commit 4a5b67e into home-assistant:dev Oct 9, 2023
@thomaskonrad thomaskonrad deleted the energy-devices-graph-max-devices branch October 26, 2023 04:28
@github-actions github-actions bot locked and limited conversation to collaborators Oct 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants