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

Threadedclass and Electron #54

Open
sensslen opened this issue Jul 27, 2021 · 2 comments
Open

Threadedclass and Electron #54

sensslen opened this issue Jul 27, 2021 · 2 comments

Comments

@sensslen
Copy link

I'm currently trying to implement an electron application using threadedClass. When starting a new threaded class I do see a lot of errors thrown. So far I think to have it tracked down to be an issue with the threading context (epectron does not allow worker threads to access electron internals). Do I understand right, that the result of threaded functions is passed on the worker thread?

Please excuse my strange wording. I'm kind of new to node development and my multithreading experience comes from languages that do have better support for multithreading (e.g. C#).

@sensslen
Copy link
Author

Reading through #20 I guess the problems I see are related. Is there any chance for a merge of #20 or #34 any time soon?

@Julusian
Copy link
Collaborator

Sorry for not noticing this sooner.

Unfortunately neither of those PRs change how the library operates. #20 is simply some new tests showing a flaw in the architecture, and #34 is the start of an attempt to make the library be more helpful in informing about the limitations without attempting to improve on them.
As such both PRs do not have a high priority from me to finish as functionally nothing will change other than being clearer when you as a user try to do something which won't work.

As for the original question, I do not have enough understanding of how this library works to answer that

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

No branches or pull requests

2 participants