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

Geocoding confidence values have changed between versions #2019

Closed RDmitchell closed 3 years ago

RDmitchell commented 4 years ago

Expected Behavior

I would have expected the Geocoding Confidence values to be the same between 2.6.0 (on the production server) and 2.6.2 (on the dev1 server)

Maybe something about how those values are created has been updated, but I wanted to record the difference while I am noticing it, and then we can figure out if this is an issue or not.

Actual Behavior

The geocoding confidence for the same data is different between the two releases.

2.6.2 The Geocoding Confidence value is "Low - check address (Ambiguous)" image

2.6.0 The Geocoding Confidence value for the same imported file is "**Missing address components (N/A)" image

Steps to Reproduce

Import the file 1 - example-data-ESPM.xlsx into each version of the program (see #2018 for that file)

Instance Information

Instance: dev1 SHA: 6dad9f7

Instance: seedv2 (production) SHA: df21680 Org: LBNL Test 200

adrian-lara commented 4 years ago

I wouldn't consider this an issue - just a difference in versions.

To clarify, the two messages represent two different cases.

"Low - check address (Ambiguous)" refers to an attempt at geocoding by sending address components to MapQuest. The term "Ambiguous" hints that MapQuest found multiple "valid" addresses/geolocations and SEED couldn't determine which of those were the one you wanted. Although 2264 has fake data, this can happen if a user only provided something that was too vague like "100 Market Street California", which can exist in multiple different cities.

"Missing address components (N/A)" refers to the case where no attempt was even made to send the address components to MapQuest.

adrian-lara commented 4 years ago

@RDmitchell Wanted to confirm with you before closing this - do you think the above information is enough to consider this a non-issue?

RDmitchell commented 4 years ago

@adrian-lara -- yes this is fine. I am going to assign it to myself and tag it as "documentation"

adrian-lara commented 3 years ago

@RDmitchell is this OK to close?

RDmitchell commented 3 years ago

sure, closing