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
Was validating the changes made from #8250 and encountered a new issue.
The code submitted as a reproduction in the prior issue now works, so I've increased the complexity by one notch to reproduce a different failure. @sydney-runkle tagging you simply because you were so responsive on the prior issue and Im assuming this will have a similar fix :)
Importantly, the print output agrees between the two methods if you remove the computed function, or move it after y_is_none
Initial Checks
Description
Was validating the changes made from #8250 and encountered a new issue.
The code submitted as a reproduction in the prior issue now works, so I've increased the complexity by one notch to reproduce a different failure. @sydney-runkle tagging you simply because you were so responsive on the prior issue and Im assuming this will have a similar fix :)
Importantly, the print output agrees between the two methods if you remove the
computed
function, or move it aftery_is_none
I believe the issue is here: https://github.com/pydantic/pydantic-core/pull/1098/files#diff-9d4853412aaf93703485a5ed7c50c373184ec151d1d480a109e52430fedf9ca8R78
Where the code
returns
after the first None computed field, instead of simply moving onto the next field to check if it is None.Example Code
Python, Pydantic & OS Version
The text was updated successfully, but these errors were encountered: