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

Duplicate Tax Lot records are being created when importing paired ESPM data #3011

Closed RDmitchell closed 2 years ago

RDmitchell commented 2 years ago

Describe the bug Importing a Portfolio Manager file with records that pair with existing Tax Lot records causes duplicate Tax Lot records to be created.

Expected Behavior Duplicate Tax Lot records should not be created

Actual Behavior See description See this video https://drive.google.com/file/d/14CM0D5wbx1mz-L9n8L5nJxqSv8Dy3Tsk/view?usp=sharing It is in the issue folder https://drive.google.com/drive/folders/146p4jYlzYoTuHe7vzj4NIfV1xKOFHl0m?usp=sharing The files I imported are also in that folder

Steps to Reproduce

Instance Information

Instance: //ad707077bc25e403eafed1eb5cc6874d-8554735.us-east-1.elb.amazonaws.com SHA: f92d4b42d Org: LBNL 101

macintoshpie commented 2 years ago

@RDmitchell could you share your column settings for tax lots? I'm wondering if this is actually behaving as expected. It looks like by default we use Address Line 1 as matching criteria, and since the second import in your specified flow above doesn't include it (but the first one does), the tax lots are not considered matches.

The simplest fix is to update your matching criteria to just be tax lot ID (assuming it is indeed the case that you have address line 1 as a matching criteria).

RDmitchell commented 2 years ago

@macintoshpie -- I think you are right. I will rework my matching criteria and make sure everything looks good.

Glad it is user error !!!

RDmitchell commented 2 years ago

@macintoshpie -- Yes if I don't include Address as a matching field, everything is working as normal.

Sorry for the panic. Me being stupid.

Closing this issue