You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Could be. I just recognised this today as a problem, when I (again) had to do an clr.AddReference ... .GetDependentAssemblies-dance to find out which library could not be loaded.
@filmor the problem is that some third-party packages try to import some libraries and have backup logic if no success. So how would we distinguish between package -handled and -unhandled imports?
We should still raise an import error, we should just add information to the message, which assembly could not be loaded and why, basically the information that the.NET exception contains.
That's not what I'm saying, I'm suggesting keeping the ImportError and just extending the information it carries, i.e. you get ImportError: No module named 'bla', tried to import bla.dll but failed loading the dependency blubb.dll because of reasons.
Activity
den-run-ai commentedon Sep 7, 2016
related? #111
filmor commentedon Sep 7, 2016
Could be. I just recognised this today as a problem, when I (again) had to do an
clr.AddReference ... .GetDependentAssemblies-dance to find out which library could not be loaded.den-run-ai commentedon Sep 7, 2016
@filmor the problem is that some third-party packages try to import some libraries and have backup logic if no success. So how would we distinguish between package -handled and -unhandled imports?
filmor commentedon Sep 7, 2016
We should still raise an import error, we should just add information to the message, which assembly could not be loaded and why, basically the information that the.NET exception contains.
den-run-ai commentedon Sep 7, 2016
what I meant is actually cases like this and we would bug down the user with optional imports failing:
https://github.com/pydata/pandas/blob/master/pandas/computation/__init__.py
filmor commentedon Sep 7, 2016
That's not what I'm saying, I'm suggesting keeping the
ImportErrorand just extending the information it carries, i.e. you getImportError: No module named 'bla', tried to import bla.dll but failed loading the dependency blubb.dll because of reasons.den-run-ai commentedon Sep 20, 2016
@filmor agreed! we need to extend the passed information.
Addresses issue pythonnet#261 with providing more detail information …
Added unit test for fix for pythonnet#261 added .NET DLL with missing…
Addresses issue pythonnet#261 with providing more detail information …
Added unit test for fix for pythonnet#261 added .NET DLL with missing…
Addresses issue pythonnet#261 with providing more detail information …
Added unit test for fix for pythonnet#261 added .NET DLL with missing…
3 remaining items
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…
lostmsu commentedon Apr 27, 2020
@filmor the right way to do it is to enable Fusion Log on Windows or set Mono debug environment variables for Mono.
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…
do not eat all exceptions when trying to add CLR reference, provide m…