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

Mapping to a field with units, that doesn't display the units, causes mapping to "hang" #1675

Closed RDmitchell closed 4 years ago

RDmitchell commented 6 years ago

Expected Behavior

Import a file, map the fields, proceed to the mapping review screen

Actual Behavior

When a field that should have units associated with it does not, the mapping screen seems to hang at progress = 0% after clicking the Map Your Data button in the main mapping screen.

Steps to Reproduce

Org: LBNL Test 200 Cycle: 2018 Compliance Cycle File: 1 - example-data-ESPM-Alerts-UBID.xlsx

So maybe the program needs to do some error checking on the mapping screen to tell the user when there are fields that need units that don't have them (??)

FYI, this doesn't seem to be an issue on dev1 - the program proceeds with the mapping even if the units are not defined for Gross Floor Area.

Instance Information

instance: LBNL staging SHA: 1a8cd10 (2.3.3 release)

RDmitchell commented 6 years ago

I'm setting this to Priority = 2 since there is a work around.

RDmitchell commented 6 years ago

can some error checking be added to the mapping screen to alert the user if they have mapped a field to a field name that should have units but doesn't?

lainsworth8801 commented 4 years ago

@RDmitchell hi Robin can you please confirm if this is still an issue? I wasn't able to recreate it on dev1. I'm on 2.7.1 and the units seem to be imported fine with no hang. Thanks!

RDmitchell commented 4 years ago

@lainsworth8801 -- I can't reproduce this error either, so am closing