Performance refactor merge ubids - Githubissues

SEED-platform / seed

Standard Energy Efficiency Data (SEED) Platform™ is a web-based application that helps organizations easily manage data on the energy performance of large groups of buildings.

Other

111 stars 54 forks source link

Performance refactor merge ubids #4704

Closed perryr16 closed 4 months ago

perryr16 commented 5 months ago

Any background context you want to provide?

What's this PR do?

Refactors a query within states_to_views to query against all states within the cycle, while the original queried against all property states regardless of cycle. Depending on the size of the db this could be a very large number of states
Improves ubid merging and early exits if possible

benchmarking for 2 files with 500 matching records: develop: 80s this branch: 70s

How should this be manually tested?

What are the relevant tickets?

Screenshots (if appropriate)

axelstudios commented 4 months ago

Incorporated into https://github.com/SEED-platform/seed/pull/4718