8000 Support for Jetson Nano (aarch64 architecture) · Issue #982 · pythonnet/pythonnet · GitHub
[go: up one dir, main page]

Skip to content

Support for Jetson Nano (aarch64 architecture) #982

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

Closed
MPMongrel opened this issue Nov 4, 2019 · 3 comments · Fixed by #986
Closed

Support for Jetson Nano (aarch64 architecture) #982

MPMongrel opened this issue Nov 4, 2019 · 3 comments · Fixed by #986

Comments

@MPMongrel
Copy link

Environment

  • Pythonnet version: 2.4.0
  • Python version: 3.6 (3.7, any)
  • Operating System: Linux aarch64

Details

  • Python.Runtime crashes when used on a Jetson Nano.
  • Request: Enable support or provide hints for a workaround.

Exception:

Unhandled exception: System.NotImplementedException: No support for aarch64
   at Python.Runtime.TypeManager.NativeCode.get_Active()
   at Python.Runtime.TypeManager.InitializeNativeCodePage()
   at Python.Runtime.TypeManager.InitializeSlots(IntPtr type, Type impl)
   at Python.Runtime.TypeManager.CreateMetaType(Type impl)
   at Python.Runtime.MetaType.Initialize()
   at Python.Runtime.Runtime.Initialize(Boolean initSigs)
   at Python.Runtime.PythonEngine.Initialize(IEnumerable`1 args, Boolean setSysArgv, Boolean initSigs)
   at Python.Runtime.PythonEngine.Initialize(Boolean setSysArgv, Boolean initSigs)
   at Python.Runtime.PythonEngine.Initialize()

Thanks!

@filmor
Copy link
Member
filmor commented Nov 4, 2019

Add the missing architecture here https://github.com/pythonnet/pythonnet/blob/master/src/runtime/runtime.cs#L146 and in the respective enum and you should be good to go. PR is welcome.

@Cronan
Copy link
Contributor
Cronan commented Nov 13, 2019

@MPMongrel please let us know if this resolved your issue

@MPMongrel
Copy link
Author

@Cronan & @filmor Apologies for not coming back to this earlier. I did make very similar changes locally already, but forgot about creating a PR. I now removed my own changes and applied the ones from #986 and verified that they still work.
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
0