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

Revert "Remove superfluous argument" #681

Closed
wants to merge 1 commit into from

Conversation

glubsy
Copy link
Contributor

@glubsy glubsy commented Jun 30, 2020

This reverts commit bcb2650.

@arsenetar
Copy link
Owner

@glubsy, I see you mentioned this fixes a issue with the scanner getting stuck at "Verified X/X Matches" dialog window over at #677. How were you able to come across the situation, I was not able to get into that state when I checked this before.

@glubsy
Copy link
Contributor Author

glubsy commented Jun 30, 2020

To reproduce the issue, I did a scan in "picture mode" and enabled "can mix file kinds" while still having the two test folders in the tested path as mentioned in #676.

To note: this bug also makes python hang and not quite terminate properly, needs a SIGKILL in order to get rid of the process.
Perhaps this needs further examination and a better fix, but as it is this revert solves the problem so far.

@arsenetar
Copy link
Owner

Did some looking into this with a few different scans again, I am not seeing any issue with the scan hanging for me in Picture mode with that option enabled. I have tried a few folder sets with and without this change I am not seeing any differences in behavior.

Additionally looking around a bit it seems _create_file is only called when importing from the saved results xml file so this should not be causing issues during a scan.

I am testing this on windows 10 right now with 64 bit python, but not seeing a reason why this would behave differently between OSes.

@glubsy
Copy link
Contributor Author

glubsy commented Jun 30, 2020

Right, it seems the option "match files of different dimensions" is actually what causes the issue for me. Perhaps even when both options are enabled at the same time. Can you reproduce this?

Edit: this is very strange, I cannot reproduce this consistently. It seems almost random.

@glubsy
Copy link
Contributor Author

glubsy commented Jun 30, 2020

Yeah it seems his revert doesn't actually fix anything. The problem is somewhere else, and most likely not related to these commits.

@glubsy glubsy closed this Jun 30, 2020
@arsenetar
Copy link
Owner

@glubsy, I'll have to play around with the "different dimensions" option turned on to see if I can get it to happen. Do you mind filling an issue for this so we can keep track of the issue?

@glubsy
Copy link
Contributor Author

glubsy commented Jul 1, 2020

Yes I'll have to investigate this further to figure out how to accurately reproduce the issue before filing a bug report.

@glubsy
Copy link
Contributor Author

glubsy commented Jul 25, 2020

See #693 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants