-
Notifications
You must be signed in to change notification settings - Fork 747
Infinite loop/crash in root.LoadNames() #603
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
Comments
@barrybriggs how can someone reproduce this issue? Did you look at troubleshooting and installation wikis? Does this happen without WPF (in console)? |
Thanks for the quick response. The code is very, very simple -- click on a WPF menu and the click handler calls
The 'names' that it loads are: "FXAssembly", "ThisAssembly", "AssemblyRef", "", and then it starts trying to load what seems like types "_StaticArrayInitTypeSize=6" and then 8, and so on. It does this forever. I have a feeling this may be because there is some required assembly or .py file or something like that it's looking for, but I can't find it. UPDATE: I should add in my bin\debug folder there are: clr.py, clr.pyd, Python.Runtime.dll, python3.dll and python36.dll ... missing anything? |
Do the demo apps work for you? Your example is still not reproducible. |
|
Please try "featured" branch. At least problems with "_StaticArrayInitTypeSize=6" types are eliminated. |
@dmitriyse That seems to have cured the problem. Thanks!! |
@dmitriyse what is the corresponding commit that fixes this "_StaticArrayInitTypeSize=6". I have not seen this issue before. |
This is the corresponding PR #528 |
closed by #723 |
Environment
Details
(Pythonnet newbie). Trying to embed Pythonnet in a simple WPF app. Built Python.Runtime from source setting PYTHON3 and PYTHON36 on x64. Gets to GetCLRModule in importhook.cs and gets in infinite loop in root.LoadNames() line 98, ultimately crashes.
The text was updated successfully, but these errors were encountered: