Skip to content

Conversation

@LaurenKerrNREL
Copy link

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

@LaurenKerrNREL LaurenKerrNREL added the Enhancement Add this label if functionality was generally improved but not a full feature or maintentance. label Nov 5, 2025
@nllong nllong requested review from Copilot and haneslinger November 6, 2025 19:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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_area field
  • 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.

LaurenKerrNREL and others added 2 commits November 7, 2025 10:16
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Add this label if functionality was generally improved but not a full feature or maintentance.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants