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
@aazuspan Do you have an interest in contributing a fix? We are probably stuck with the type arg name at least until the next major version release, but we could probably work around the issue another way (like move the type checking outside of log_artifact() or checking the type with path.__class__).
When you pass an invalid path type to
Live.log_artifact
, it's supposed to raise anInvalidDataTypeError
:dvclive/src/dvclive/live.py
Lines 493 to 494 in 47c87b3
Instead, it raises
TypeError: str is not callable
because the builtintype
is shadowed by the parameter:dvclive/src/dvclive/live.py
Line 484 in 47c87b3
It's a pretty minor issue, but I thought I'd point it out since it could cause some confusion. Thanks!
The text was updated successfully, but these errors were encountered: