AequilibraE / qaequilibrae

Free QGIS add-on for transportation modeling
https://www.aequilibrae.com/qgis/latest/
Other
75 stars 24 forks source link

Unable to create project with "Create Project from layers" function #181

Closed Art-Ev closed 7 months ago

Art-Ev commented 2 years ago

Describe the bug

Error when creating project from layer: \AequilibraE\project_procedures\creates_transponet_dialog.py", line 310, in run_thread self.worker_thread.ProgressValue.connect(self.progress_value_from_thread) AttributeError: 'CreatesTranspoNetProcedure' object has no attribute 'ProgressValue'

To Reproduce

Creating project from layer. For example from this network (also tested on another network with the same result) : NetworkExample.zip

Desktop (please complete the following information):

Art-Ev commented 2 years ago

Same error with matrix import from OpenLayers: \AequilibraE\matrix_procedures\load_matrix_dialog.py", line 149, in run_thread self.worker_thread.ProgressValue.connect(self.progress_value_from_thread) AttributeError: 'LoadMatrix' object has no attribute 'ProgressValue'

(Using this project : St_Varent.zip )

pedrocamargo commented 2 years ago

I would not expect to have many of these, as I am surprised there are any!!

I replaced the worker thread class throughout the software, but have clearly failed to do the stitching (and testing) properly. Wil fix it soon.

pedrocamargo commented 2 years ago

A new version fixing this issue has been uploaded

Art-Ev commented 2 years ago

Bearer of bad news is back @pedrocamargo :

pedrocamargo commented 2 years ago

@Art-Ev , The crashing should be fixed.

The second issue is also crashing (but since it is on a thread, you would need to open the little message bar on the right lower corner image

This is not actually a bug, but a case where the input data has no projection system, and therefore AequilibraE does not know how to convert the data. I have included a check for that to give the user an error message before the importing process begins.

These two issues have been updated and a new version has been uploaded.

The last issue is a little more complicated, so I am investigating. This will probably come with a new feature of "upgrading old aequilibrae models", as link_types, modes, list of matrices and other things should also be carried through.

Art-Ev commented 2 years ago

Will continue to investigate 2nd and 3rd issues

pedrocamargo commented 2 years ago

Good insight. I will take a look at this procedure this weekend. I have recently developed some similar stuff at Argonne but using a lot more SQL, which I will try to transfer here. It should simplify the code substantially

Art-Ev commented 1 year ago

Hi ! Does somebody find something (maybe just a direction) to allow project creation from layers which was working bedore ? (keeping ID's from provided layers) @pedrocamargo, @r-akemii ?

r-akemii commented 11 months ago

Hey @Art-Ev, I ran an example with links with _linkid that doesn't start in 1, and the project created seems to be working fine, keeping their original ID. Do you have any data that didn't work previously to provide me so I can better investigate this issue?

r-akemii commented 9 months ago

@Art-Ev I did some modifications in Create projects from layers that might have solved the issue you reported. Could you please check and see if it is working fine now?

r-akemii commented 7 months ago

There was indeed a bug in Create functions from layers that was solved in #291.