cityenergyproject / seattle

Energy benchmarking for Seattle
ISC License
1 stars 5 forks source link

Look at how no-data situations are handled #151

Closed vinaydixit closed 2 years ago

vinaydixit commented 4 years ago

need consistent messaging, better wording and expected behavior.

ebrelsford commented 4 years ago

@seattle-benchmarking I believe this is mostly taken care of, but are there specific situations that you would like us to take a look at?

ebrelsford commented 4 years ago

Check on common times when there will not be data available:

ebrelsford commented 4 years ago

Energy star says "Energy Star Score data not available" in both popup and on scorecard.

seattle-benchmarking commented 4 years ago

@ebrelsford A few minor changes to wording for the errors if you don't mind:

https://cityenergyproject.github.io/seattle/#seattle/2018?layer=energy_star_score&sort=energy_star_score&order=desc&lat=47.60810&lng=-122.33216&zoom=14&building=18&report_active=true

https://cityenergyproject.github.io/seattle/#seattle/2015?layer=energy_star_score&sort=energy_star_score&order=desc&lat=47.54968&lng=-122.31603&zoom=14&building=215&report_active=true

seattle-benchmarking commented 4 years ago

@ebrelsford another example of missing EUI for one year in the Trend line: https://cityenergyproject.github.io/seattle/#seattle/2018?categories[0][field]=property_type&categories[0][other]=false&categories[0][values][]=Small-+and+Mid-Sized+Office&layer=energy_star_score&sort=energy_star_score&order=desc&lat=47.61167&lng=-122.32028&zoom=15&building=839&report_active=true

Perhaps we can just set 0s to null to avoid the weird line. Not sure this would work if a missing value was 2015 or 2018 though?

ebrelsford commented 4 years ago

@seattle-benchmarking I'd appreciate it if we could keep these as separate issues as some of them have different causes and bundling them here can make it difficult to track.

As for building ID 215, this isn't because of the year data, it's because it doesn't have a property type:

image

seattle-benchmarking commented 4 years ago

@ebrelsford Can you copy the new ENERGY STAR text to the EUI situations too?

We don't have an EUI for this building for this year. Contact energybenchmarking@seattle.gov if this is incorrect.

image

ebrelsford commented 4 years ago

@seattle-benchmarking Done!