search

FLO-2DSoftware / qgis-flo-2d-plugin

A plugin for pre-processing/post-processing FLO-2D models
8 stars 8 forks source link

cell array error for levee breach #1571

Closed FLO-2DKaren closed 1 month ago

FLO-2DKaren commented 1 month ago

Rio Grande Short.zip

This is an imported project. I wanted to set up a levee breach but I got an error about the cell array. So I deleted the old levee and rebuilt it using a short section. Can you click the breach button and see if you get the error?

rpachaly commented 1 month ago

What button exactly was triggering the error?

FLO-2DKaren commented 1 month ago

Levee Breach button.

rpachaly commented 1 month ago

It worked fine on my side.

FLO-2DKaren commented 1 month ago

This button works? It still isn't working for me. image

FLO-2DKaren commented 1 month ago

2024-10-10T08:59:43 WARNING Traceback (most recent call last): File "C:\Users/Karen/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\flo2d\gui\levee_and_breach_editor_widget.py", line 139, in show_levees dlg_individual_levees = IndividualLeveesDialog(self.iface, self.lyrs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "C:\Users/Karen/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\flo2d\gui\dlg_breach.py", line 763, in init if cellIDNumpyArray is None: ^^^^^^^^^^^^^^^^ NameError: name 'cellIDNumpyArray' is not defined

rpachaly commented 1 month ago

Hi Karen,

Yes, it is working fine here. Can you make a short video so I can see what is happening?

FLO-2DKaren commented 1 month ago

https://github.com/user-attachments/assets/3a2466b0-2c28-4ed8-8a4a-90a32e66439b

rpachaly commented 1 month ago

The project you sent to me is just a portion of the levee. Can you send this one that you're using? I don't see the error here.,

FLO-2DKaren commented 1 month ago

Yes. I rebuild a short section to test it but it still wasn't working on my computer. I'm going to restart my computer. Rio Grande Short orig.zip

rpachaly commented 1 month ago

I got the error here! I was using an old plugin. Let me fix it.

FLO-2DKaren commented 1 month ago

Yay!!!

rpachaly commented 1 month ago

I was using an old qgis profile thinking that my plugin was updated. My bad

rpachaly commented 1 month ago

This was caused by a removed import. It is a little weird, because it was a global variable. Done and fixed on https://github.com/FLO-2DSoftware/qgis-flo-2d-plugin/pull/1572

FLO-2DKaren commented 1 month ago

We wouldn't have seen it in testing because we don't set up levee failure that way on the self help kit.

FLO-2DKaren commented 1 month ago

Tested on multiple projects.