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
107 stars 55 forks source link

Update default dates on data quality rules #4538

Closed nllong closed 7 months ago

nllong commented 7 months ago

Any background context you want to provide?

When creating a new org, the default dates for a few of the rules (e.g., sale_date, release_date, year_built) where from 2020 or 2019.

What's this PR do?

How should this be manually tested?

What are the relevant tickets?

n/a

RDmitchell commented 7 months ago

@nllong -- is there a way to have the program figure out what the current year is and make that default, so in 2025 we don't have to change it by hand?

nllong commented 7 months ago

@nllong -- is there a way to have the program figure out what the current year is and make that default, so in 2025 we don't have to change it by hand?

Yes, we can do this, but it require a bit more rework of the defaults section. I'd be happy to do it, but this fixes the issue for now.

nllong commented 7 months ago

nvm, I just updated it to be dynamic...

RDmitchell commented 7 months ago

@nllong -- thanks !!

nllong commented 7 months ago

@nllong, what about the default cycle that gets created with a new org? Screenshot 2024-02-19 at 3 47 42 PM

good idea, I can add this update too!

nllong commented 7 months ago

@nllong, what about the default cycle that gets created with a new org? Screenshot 2024-02-19 at 3 47 42 PM

good idea, I can add this update too!

Actually, this is already done... it is just the current_year -1. Which is okay, right? Or do we want this to be just the current year? It is strange because most cycles are mid year, (June 30). So we will always be off half the year. I say we leave it for now.

RDmitchell commented 7 months ago

instance: dev1 SHA: bcbec0272 Org: LBNL 633

I made a new org and looked at the dates in the DQ world, and they seem to be still maxed out at 2020, with the starting dates being 1889 ???

image

@kflemin -- seems like maybe this hasn't been deployed to dev1 yet?

axelstudios commented 6 months ago

@RDmitchell This fix is deployed to dev1 currently, and appears to working as expected image

RDmitchell commented 6 months ago

Instance: dev1 SHA: 77b9e4afa

When I make a new rule, and set the type to Date, then click the calendar, April 2024 appears image

@nllong -- Is that what this fixes? For existing rules, nothing changes

image

kflemin commented 6 months ago

@RDmitchell it was my understanding that this was for new rules. I don't think we can auto-update existing rules.

RDmitchell commented 6 months ago

@kflemin -- Ah, yes, that makes sense.

I will close the issue