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

Use Julia new foreign thread support #26

Closed
cdsousa opened this issue Jul 31, 2022 · 3 comments
Closed

Use Julia new foreign thread support #26

cdsousa opened this issue Jul 31, 2022 · 3 comments
Assignees
Labels
acknowledged the team is aware of this issue feature request New feature or request low priority this should be fixed but it may take time

Comments

@cdsousa
Copy link

cdsousa commented Jul 31, 2022

Possibly this is not new news for you, but Julia is about to support foreign threads calling into it: JuliaLang/julia#45447.
This will probably be very helpful and simplify jluna.

@Clemapfel Clemapfel self-assigned this Jul 31, 2022
@Clemapfel Clemapfel added feature request New feature or request low priority this should be fixed but it may take time acknowledged the team is aware of this issue labels Jul 31, 2022
@Clemapfel
Copy link
Owner

I was aware of this PR but I'm not sure if I want to bump up the version requirement for Julia to something this new, I'd like it to stay at 1.7.

If this will be part of the 1.8 stable release then I'll definitely consider reworking the threadpool interface

@cdsousa
Copy link
Author

cdsousa commented Jul 31, 2022

This will be only in 1.9.
I agree you don't need to add this as soon as possible, I opened this issue more to raise awareness that such a new feature may be useful for this package in the future. Feel free to close this for now if you prefer 😉

@Clemapfel
Copy link
Owner

Considering the amount of jank I had to implement to get the jluna threadpool to work, it was appropriate to point out - so thank you! Hopefully they also fix @threadcall which is currently unusable because of how the C-API work.

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 feature request New feature or request low priority this should be fixed but it may take time
Projects
None yet
Development

No branches or pull requests

2 participants