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

Thread adoption #45

Closed
Taaitaaiger opened this issue Feb 19, 2023 · 2 comments
Closed

Thread adoption #45

Taaitaaiger opened this issue Feb 19, 2023 · 2 comments
Assignees
Labels
acknowledged the team is aware of this issue critique constructive criticism duplicate issue already exists

Comments

@Taaitaaiger
Copy link

Hi!

I've got to admit I'm not a user of JLuna, but rather someone who's working on a similar library. Considering you've put a lot of effort in mutithreading support, I think you might also be interested in thread adoption. This is a feature that will be available in Julia 1.9 that can be used to create a pool of threads that can call into Julia.

Feel free to ask any questions you have, or close this issue if you're not interested :)

@Clemapfel Clemapfel self-assigned this Feb 20, 2023
@Clemapfel Clemapfel added duplicate issue already exists critique constructive criticism acknowledged the team is aware of this issue labels Feb 20, 2023
@Clemapfel
Copy link
Owner

Clemapfel commented Feb 20, 2023

Thank you, it is nice to find a fellow independent-julia-wrapper-library creator.

I was aware of this feature the moment the PR for it was opened on the Julia GitHub, just a few days ago (#42) I was informed how the crash is related to thread local storage, which is interesting. I might update the docs with that context.

I'll be deprecating the jluna multi-threading modules once 1.9 becomes stable, however, that will be many months in the future, I'm sure.

Thank you for thinking of informing me, I feel like it's possible to feel the undercurrent of frustration in how I wrote the multi-threading docs, mostly because I spend 3 days trying to reverse engineer the crash thinking it was a bug. I'm actually still kinda upset about that but what's done is done.

I'll be closing this issue as a duplicate of #26 and #42

@Taaitaaiger
Copy link
Author

Taaitaaiger commented Feb 20, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
acknowledged the team is aware of this issue critique constructive criticism duplicate issue already exists
Projects
None yet
Development

No branches or pull requests

2 participants