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

BETTER analysis: Errors due to wrong mapping - can we improve this? #4148

Open RDmitchell opened 1 year ago

RDmitchell commented 1 year ago

Instance: dev1 SHA: 5f7d20c31 Org: LBL Test 401

In order to perform a BETTER analysis, fields must be mapped properly.

For example, BETTER wants the following fields:

The default fields in ESPM are

If the user doesn't know to change these fields, then the BETTER analysis will not run.

It gives these errors

image

The problem is that the user has to then reimport their data with different field mappings in order to get BETTER to run.

I am wondering if there is something we can do, possibly at the time of Mapping, to help the user understand how to map their fields if they want to do a BETTER analysis?

It seems like the first thing to do would be to add some help text in the "Step 1: Map Your Data" section image

In general, we need to give users more information about how to properly map their data, so I guess we should start here.

Maybe there are some more "aggressive" ways to handle this, but for now, I think we should at least add more documentation.

RDmitchell commented 1 year ago

And because the ESPM fields come in as extra data if they are not "remapped", I don't think changing their display name will help image image

RDmitchell commented 1 year ago

I verified that changing the display names didn't fix the problem, because these are extra data fields with those names image

and same with State

RDmitchell commented 1 year ago

Here is some possible text to add to the Step1 mapping tab

BETTER Analysis Mapping A BETTER analysis requires that both "City" and "State" fields be defined properly in the mapped fields. BETTER also requires that "Property Type" be defined as a field.

For example, in ENERGY STAR Portfolio Manager, those fields may be defined as

In that case, you should remap them to:

image

RDmitchell commented 1 year ago

After reimporting my data 3 times, with the proper mapping, it does appear that I am getting some BETTER results.

So we definitely need to figure out how to improve the user experience.

And reimporting the way I did wouldn't really be practical if already had a lot of imported data. In my case I had a clean slate, but that is not generally the case. Users could have a ton of data already imported with the wrong mapping that they couldn't really reimport very easily.

github-actions[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity within 60 days. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 11 months ago

This issue has been automatically marked as stale because it has not had recent activity within 60 days. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 9 months ago

This issue has been automatically marked as stale because it has not had recent activity within 60 days. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 7 months ago

This issue has been automatically marked as stale because it has not had recent activity within 60 days. It will be closed if no further activity occurs. Thank you for your contributions.