Add square footage total to data summary page enhancement#5131
Open
LaurenKerrNREL wants to merge 6 commits intodevelopfrom
Open
Add square footage total to data summary page enhancement#5131LaurenKerrNREL wants to merge 6 commits intodevelopfrom
LaurenKerrNREL wants to merge 6 commits intodevelopfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request adds a "Total Square Feet" metric to the inventory summary dashboard by calculating the sum of gross floor areas from property states and displaying it in the summary grid.
- Adds backend calculation for total square footage from
gross_floor_areafield - Updates frontend summary grid to display the new metric alongside existing statistics
- Changes the column header from "Count" to "Value" to better reflect the mixed data types now displayed
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| seed/views/v3/analyses.py | Adds query to fetch gross_floor_area values and calculates total_sqft for the stats endpoint response |
| seed/static/seed/js/controllers/inventory_summary_controller.js | Updates summary grid to include Total Square Feet row and renames column header from "Count" to "Value" |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Any background context you want to provide?
Enhancement for SEED biannual statistics, current work requires a manual process to get square footage by exporting and summing the square footage column manually. This work automates that by providing the sum of square footage in the summary table.
What's this PR do?
Adds sum of square footage to the summary table
How should this be manually tested?
We used test buildings with square footage, exported the data from inventory and summed the square footage field then compared it to the square footage sum in the summary table. Both values matched. We may want to test in the same way for a live organization instance once merged to develop.
What are the relevant tickets?
None, request from Nicholas by way of conversation and teams during ideation around improvements to biannual stats process
Screenshots (if appropriate)
NA