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

Does not cope with multiple instances well. #11

Closed
kerryland opened this issue Jul 3, 2020 · 1 comment
Closed

Does not cope with multiple instances well. #11

kerryland opened this issue Jul 3, 2020 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@kerryland
Copy link
Contributor

"One or errors occurred"

To reproduce:

  • Start auxmic.ui.exe, and use file01.mp4 as the "master" and file02.mp4 as the "other" file.
  • Wait for file02.mp4 to load
  • Start auxmic.ui.exe, and use file02.mp4 as the "master". Notice that it loads it more-or-less instantly.
  • Load file01.mp4 as the "other" file.

Notice the error dialog that says "One or errors occurred".

I got into this situation simply because after processing the files the first time I realised I had them the wrong way around, so just ran it again with them the "correct" way around. In truth I don't know what "master" means. Should "master" be renamed to "audio"?

I'd guess it's a problem with name clashes in the temp folder -- maybe add a unique instance identifier to the file names, or just disallow multiple instances of the app :-)

@osmanovv osmanovv added the bug Something isn't working label Jul 14, 2020
@osmanovv osmanovv self-assigned this Jul 14, 2020
osmanovv added a commit that referenced this issue Jul 14, 2020
@osmanovv
Copy link
Owner

Hi, Kerry! Thank you for your feedback again : )

I've just fixed this multiple instances running issue. If you don't want to wait until the next release, you can download fixed version from actions section - Artifacts: https://github.com/osmanovv/auxmic/actions/runs/169077570

Yes, that's temp file issue.

Could you please suggest UI improvements in #6. I also don't like current naming, "audio" not suite quite well because it's not neccesary may be an audio, it could be a video media file with better audio as well.

osmanovv added a commit that referenced this issue Aug 9, 2020
v0.8.1.115
- [x] Improved usability (#6)
- [x] Added ability to export synchronized file with `FFmpeg` (#7)
- [x] Fixed multiple instances run (#11)
- [x] Fixed progress display (while matching files) (#17)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants