autowrap icon indicating copy to clipboard operation
autowrap copied to clipboard

Published 20 hours ago • verified publisher icon OpenMS

Allow nogil for most types of methods

Open jpfeuffer opened this issue 7 months ago • 3 comments

This requires a change in conversion providers. +Some other cleanups.

jpfeuffer avatar Nov 18 '23 12:11 jpfeuffer

I will fix tests because of newer typing syntax

jpfeuffer avatar Nov 18 '23 17:11 jpfeuffer

Probably none out of the box. But it allows to use our functions in a multi threaded python context e.g. via the python threading module.

This brings some responsibilities for the user with it but you can decide which functions are safe to use.

jpfeuffer avatar Nov 21 '23 12:11 jpfeuffer

This is a breaking change. Be prepared to fix OpenMS if you merge this.

jpfeuffer avatar Mar 16 '24 21:03 jpfeuffer

Given that we already need to do some fixes for OpenMS to get it to work with the current develop version of autowrap, I'm good merging this in and planning to have the core group sit down and fix at some point in the future.

poshul avatar Apr 16 '24 09:04 poshul

what needs to be done to work with the current develop version of autowrap otherwise?

timosachsenberg avatar Apr 16 '24 09:04 timosachsenberg

and what would need to be fixed?

timosachsenberg avatar Apr 16 '24 09:04 timosachsenberg

and what would need to be fixed?

https://github.com/OpenMS/autowrap/issues/186

poshul avatar Apr 16 '24 09:04 poshul