Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken layers auto-find function freezes QGIS #37757

Closed
elpaso opened this issue Jul 11, 2020 · 5 comments · Fixed by #37765
Closed

Broken layers auto-find function freezes QGIS #37757

elpaso opened this issue Jul 11, 2020 · 5 comments · Fixed by #37765
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!

Comments

@elpaso
Copy link
Contributor

elpaso commented Jul 11, 2020

The (relatively) new feature "Auto-find" in the broken layer dialog freezes QGIS completely in some cases.

I've been able to reproduce this in a few cases, try opening the attached project and click on "Auto-Find" in the broken layers dialog.

MS_itopen.zip

@elpaso elpaso added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label Jul 11, 2020
@elpaso
Copy link
Contributor Author

elpaso commented Jul 11, 2020

CC @roya0045

@roya0045
Copy link
Contributor

Its not a freeze per say, its just searching layer, I have tried to give feedback by showing a progress bar but it seems that the process doesn't allow the bar to pop up sometimes. I'm wondering if it could be done in a background thread.

But yeah, it demands a lot of ressource if the possible paths are deep to explore.

Suggestions are welcomed.

@elpaso
Copy link
Contributor Author

elpaso commented Jul 11, 2020

Its not a freeze per say,

Well, I waited ~ 10 minutes with QGIS completely unresponsive ... I'd call it a freeze/hang.

Anyway, this is a serious bug, are you sure there is no infinite loop somewhere?

its just searching layer,

Where exactly? In the whole filesystem?

I have tried to give feedback by showing a progress bar but it seems that the process doesn't allow the bar to pop up sometimes.

The process must be user-interruptible.

I'm wondering if it could be done in a background thread.

I think it's the right choice.

@roya0045
Copy link
Contributor

@elpaso the only issue that I have with background thread is users trying to use a project when the sources haven't all been found.

Yes the layers are searched in an outward spiral pattern until the search directory reaches a point where its too close to the root.

@roya0045
Copy link
Contributor

I'll try what failed previously as I used the progress dialog and process all events. I'll see if I can use a background thread instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants