-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
feat(ivy): support WrappedValue in pipes (FW-726) #27409
Conversation
You can preview 90e027b at https://pr27409-90e027b.ngbuilds.io/. |
You can preview 69285f1 at https://pr27409-69285f1.ngbuilds.io/. |
You can preview 0cb4774 at https://pr27409-0cb4774.ngbuilds.io/. |
You can preview adc992b at https://pr27409-adc992b.ngbuilds.io/. |
You can preview ab0eb6c at https://pr27409-ab0eb6c.ngbuilds.io/. |
You can preview 72c2835 at https://pr27409-72c2835.ngbuilds.io/. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This PR implements the support of WrappedValue in pipes.
It uses the same trick as in the ViewEngine: the result of the pipe transformation is unwrapped immediately in the pipe instruction. And, if this result is wrapped, the previous value is wrapped to force change detection.
As a result, the unwrap code is only executed for pipe output, and the WrappedValue code is imported only if a pipe is used.
This solves 21 failures in the router packages when running the tests with Ivy.