Skip to content

Added LR26 test for hourly points over 60 day period with flow units (kW)#1632

Open
KarnbirKhera wants to merge 1 commit into
OpenEnergyDashboard:developmentfrom
KarnbirKhera:development
Open

Added LR26 test for hourly points over 60 day period with flow units (kW)#1632
KarnbirKhera wants to merge 1 commit into
OpenEnergyDashboard:developmentfrom
KarnbirKhera:development

Conversation

@KarnbirKhera
Copy link
Copy Markdown

@KarnbirKhera KarnbirKhera commented May 28, 2026

Added LR26 test for hourly points over 60 day period with flow units as (kW)

Description

Added the LR26 test case to readingsLineMeterRangeFlow.js which verifies that the API returns the hourly aggregated min/max points specifically for when a 60 day time window is requested @ a 15 minute flow meter with kW units.

Also includes the respective .csv file with 1440 hourly data points.

Partly addresses #962

Type of change

  • Note merging this changes the database configuration.
  • This change requires a documentation update

Checklist

  • I have followed the OED pull request ideas
  • I have removed text in ( ) from the issue request
  • You acknowledge that every person contributing to this work has signed the OED Contributing License Agreement and each author is listed in the Description section.

Limitations

None that I am aware of in terms of the LR26 test as it passes all 891 tests using the provided the OED Docker container.

@huss
Copy link
Copy Markdown
Member

huss commented May 28, 2026

@KarnbirKhera Thank you for this contribution. The PR description states the CLA has been signed by all contributors. However, OED records do not have one for you. Thus, I would like you to:

  • Please sign the CLA via the link in the description or let me know that you think our records are off.
  • Add a comment that gives the names of any other people involved or state that you are the only one who did work. If others worked on this then they also need to sign the CLA.

Once this is done, this PR can be formally reviewed. Please let me know if you have any questions/thoughts.

@KarnbirKhera
Copy link
Copy Markdown
Author

Hi @huss

My apologies I thought I had submitted an CLA agreement, but it seems I may have forgotten, so I just submitted one.

This contribution was a part of our CodeDay group with @livanho

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.

2 participants