-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Error assert when use lazy_instance #515
Comments
Thank you for reporting! I will look at it. |
For now some information. That assert is there because I didn't know if it could happen, because it it did, the implementation does not currently consider it. Removing the assert wouldn't impact the CLI logic, but the lazy instance could misbehave if used. Also, the problem happens with tensorflow 2.10 but not with the latest version. Would be good to extend the implementation to consider this case. But I would do so only if I am able to write a unit test for it, and I haven't been able for the moment. |
@mauvilsa Hi mauvilsa! Where can I find |
Well, there isn't much to illustrate about |
🐛 Bug report
when use lazy_instance as default value, there is a assert cannot satisfy
To reproduce
trace:
when comment
# assert name not in self.__dict__
it works success-->
Expected behavior
run normaly
Environment
pip install jsonargparse[all]
):The text was updated successfully, but these errors were encountered: