search

dpinney / omf

The Open Modeling Framework for smart grid cost-benefit analysis.
https://omf.coop
GNU General Public License v2.0
112 stars 60 forks source link

Phase ID error: IndexError: index 3 is out of bounds for axis 0 with size 3 #410

Closed Grubes2023 closed 1 month ago

Grubes2023 commented 3 months ago

Howdy,

I am trying to use your phase_id model; however, I am getting an error each time. I have examined the the example sandria file but see no real differences between mine and the example. I am not a coding expert, so I cannot personally troubleshoot beyond the obvious.

Any help is greatly appreciated

PhaseFormat_for_help.csv Screenshot 2024-07-11 165647

Brandon-TriCounty commented 2 months ago

Any progress on this??? I saw that there was a couple commits for this but the website version does not seem to reflect that. And I couldn't get windows to install a local copy. Pip did not like the requirements file

dpinney commented 2 months ago

Yep. Changes had to be made in an upstream package. We now have to integrate them and update prod.

Brandon-TriCounty commented 2 months ago

I just checked the website version and it seems to be updated. The code worked for that example; however, I got another error when running a different dataset... Screenshot 2024-08-27 164047

After a quick lookthrough the code, I think the problem is that we have transformer voltages that are not supported in the code. I believe the relevant code is the ConvertToPerUnit_Voltage function in the PhaseIdent_Utils.py file. Could you add support for the following voltages : 208, 277, 480

Thanks

dpinney commented 2 months ago

Sure thing, I just sent a pull request for this change and we should have it integrated next week up on OMF.coop. Thank you for the detailed change request.

dpinney commented 1 month ago

The change with support for additional voltage levels is live now on OMF.coop.