Closed drmfinlay closed 4 months ago
I've looked into this now. There are a few issues, but I think it's possible.
I've started working on this here: https://github.com/drmfinlay/natlink2/tree/use_py3_stable_abi
My repository (natlink2) doesn't have many of the more recent changes. I used an older revision (without the PyConfig
stuff) to reduce the amount of initial work, and to make building a PYD file easier.
Seems to be progressing nicely. The project cannot yet be built without a few prior changes to CMakeLists.txt.
The natConnect()
function's optional parameter is a little tricky to deal with using only the limited API. @quintijn, do you know if there is any benefit to passing False
(default) to natConnect()
? Most users seem to pass True
or 1
.
I think I am now able to answer my own question. It looks like the only upside to not enabling thread safety is that the Natlink window's "Reload Everything" menu button works.
I have been able to successfully use a Natlink PYD file built for Python 3.4 (stable ABI) with Python 3.8, 3.9 and 3.11. Dragonfly's test suite passes normally. So that's nice. No thread safety yet, however. But then, Natlink doesn't have it on by default anyway.
Presumably, this might make porting to python 11/12 easier.
It should do, yes. Natlink would only need to have one installer for Python 3.
@quintijn Does unimacro spawn any threads? Or any of the timers?
That is a too diffucult question, but: 1: I am not aware of that, 2: unimacro occasionally uses the natlinktimer.py feature that I introduced in the recent releases...
Let us discuss via zoom at some time, maybe,
Quintijn
Doug Ransom @.***> schreef op 26 oktober 2023 18:24:32 CEST:
@quintijn Does unimacro spawn any threads? Or any of the timers?
-- Reply to this email directly or view it on GitHub: https://github.com/dictation-toolbox/natlink/issues/171#issuecomment-1781448564 You are receiving this because you were mentioned.
Message ID: @.***>
Sorry, yes, it is a difficult question. I've solved the problem I was having with thread safety. Threads, timers and the natConnect()
function are working normally with my changes.
I've got it mostly working now in my natlink2 repo.
https://github.com/drmfinlay/natlink2/tree/use_py3_stable_abi
A PYD can be built and will work out of process on Python 3.3+. As for normal use as a DNS compatibility module, there is a dependency issue with python3.dll. I think this was probably solved a while ago in this repo.
I had to stop working on this in November. The main roadblock here is my limited understanding of how to change the installer logic. Perhaps someone might be able to help me with that at some point?
I wanted to mention, for consideration, an idea I had. I was thinking that, since the installer would work with essentially any 32-bit Python version 3.x, there should be an option in the installer for which version to install/use. Only one version can be used at a time, for complex reasons. Such functionality would also need to exist in the NatLink configurations utilities -- something similar to the "register" button in the old 4.x series GUI.
Let me know if you guys think this is still worth pursuing. I still think the changes would be useful, especially from a maintenance perspective.
Thanks for all these things, Dane. It is hard for me to judge the things. Also because I am still (and for a long time by now) busy getting a python3 version of Natlink working with all the things around it (like other packager, just now busy getting Unimacro things checked)
Do I understand well, that you make a special python install, just for Natlink to run on? I have other programs having such a dedicated python install. Then the Natlink python install does not have any connection with possible other python installs?
Dane, do you work with Dragon 16? Can you test if playEvents (and playString) work with you C Limited API??
Sorry for the late reply.
Yes, I normally use Natlink with a system Python installation that isn't used for much else. But that is mainly because Natlink requires 32-bit Python.
I don't work with Dragon 16. However, I can tell you that the issue with playEvents and playString would remain if Python's C limited API were to be used.
Using that API shouldn't do anything other than change the Python DLL requirement of the _natlink_core.pyd file from python310.dll to python3.dll, assuming Python 3.10 is used. That will allow using the same _natlink_core.pyd file with Python 3.8, Python 3.9, Python 3.10, and so on without recompiling.
Thinking about my proposal here, I think it is something that should have been done years ago when the C++ code was made compatible with Python 3. I don't think there's much point now.
Dane, I am rereading issues of natlink and related. Pffff, what a lot of work you did here, with no concrete result. Sorry I cannot judge all implications of this, but my great appreciations for this work, which will probably not be used then.
In general, we have two types of users,
I don't know what I want to say with this, only that some dedicated install will be sufficient.
Greetings, Quintijn
Hi Quintijn,
Thank you for your comment. What you say on the two types of Natlink users is true and sounds reasonable to me. A dedicated install is good enough, yes.
My work on this issue would have been more valuable when the C++ codebase was being ported to Python 3. In fact, I recall that one of the programmers involved in that mentioned Python's limited API in a comment somewhere. I suppose he found it too tricky to work with.
In any case, the work was quite valuable to me as a learning experience.
On Thu, 06 Jun 2024 08:53:38 -0700 Quintijn Hoogenboom @.***> wrote:
Dane, I am rereading issues of natlink and related. Pffff, what a lot of work you did here, with no concrete result. Sorry I cannot judge all implications of this, but my great appreciations for this work, which will probably not be used then.
In general, we have two types of users,
- no programmers, they don't care which python is installed, even don't know what python is.
- programmers, who will use a 64bit version of python. Installing some 32bit version for natlink will not hurt, but more important, "we" can choose which one is suitable, as it will not be used for other applications.
I don't know what I want to say with this, only that some dedicated install will be sufficient.
Greetings, Quintijn
-- Reply to this email directly or view it on GitHub: https://github.com/dictation-toolbox/natlink/issues/171#issuecomment-2152870396 You are receiving this because you modified the open/close state.
Message ID: @.***>
At the moment, Natlink must be compiled for each minor 32-bit version of Python. It has been a pain-point getting the right PYD file loaded. I wonder if it might be worthwhile to use Python's limited API, documented here and available since Python 3.2. If Natlink could be compiled this way, the same PYD file would work on Python 3.8, 3.9, 3.10 and so on.
It may not be possible to do this with Natlink, I'll need to look into the code first. If it can be done, I would be willing to work on this. I think it would simplify the project and make it easier to maintain going forward.