Skip to content

Skip serializing FusionKernelRuntime if it is not compiled#1875

Closed
rdspring1 wants to merge 4 commits intomainfrom
skip_serde_nc_fusions
Closed

Skip serializing FusionKernelRuntime if it is not compiled#1875
rdspring1 wants to merge 4 commits intomainfrom
skip_serde_nc_fusions

Conversation

@rdspring1
Copy link
Collaborator

This PR avoids the following error when the FusionKernelRuntime is not compiled in serde test suite.

RuntimeError: compiled_kernel_ != nullptr && (!compiled_kernel->cubin.empty() || !compiled_kernel->ptx.empty())

@rdspring1
Copy link
Collaborator Author

!build

@rdspring1 rdspring1 marked this pull request as ready for review March 6, 2024 06:21
@rdspring1 rdspring1 requested a review from jjsjann123 March 6, 2024 06:21
@jjsjann123
Copy link
Collaborator

sorry somehow forgot about this one.

Copy link
Collaborator

@jjsjann123 jjsjann123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QQ: was this the case for a no-op fusion or something?

Is there an existing test failure that this PR patches, otherwise, can we add a python test as well.

@rdspring1
Copy link
Collaborator Author

!build

Copy link
Collaborator

@jjsjann123 jjsjann123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stamping just to keep CI green.

@jjsjann123
Copy link
Collaborator

!build

@rdspring1
Copy link
Collaborator Author

Replace with #2061

@rdspring1 rdspring1 closed this Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants