recent changes to propertynames spits out key error

JuliaPy / PyCall.jl

Package to call Python functions from the Julia language

MIT License

1.47k stars 187 forks source link

recent changes to propertynames spits out key error #893

Closed ppalmes closed 2 weeks ago

ppalmes commented 3 years ago

this happens when you call propertynames in sklearn to check for fit or transform vs predict. once you call one them, it throws a key error exception because while propertynames say fit or transform/predict is available, actually calling one of them breaks. getmember is a more stable call instead of dir which list methods which may not be callable.

ppalmes commented 3 years ago

https://github.com/IBM/Lale.jl/runs/2329281498?check_suite_focus=true

stevengj commented 3 years ago

cc @sethaxen since #879 is at issue here.

ppalmes commented 3 years ago

yes. i have to copy the old code which uses getmember so that my fit/transform call to sklearn works. i’m not sure why python returns key error. it seems the dir command has bogus return of fit/transform such that calling python object returns error because it doesn’t support it.

sethaxen commented 3 years ago

Could you provide a minimal example of a Python class for which dir fails but getmember works? See for example the test class added in #879 for which dir works but getmember fails.