Skip to content

Conversation

booxter
Copy link
Contributor

@booxter booxter commented Jun 9, 2025

It's always True.

Signed-off-by: Ihar Hrachyshka [email protected]

It's always True.

Signed-off-by: Ihar Hrachyshka <[email protected]>
@mergify mergify bot added ci-failure and removed ci-failure labels Jun 9, 2025
@booxter booxter marked this pull request as ready for review June 9, 2025 23:08
Copy link

E2E (NVIDIA L40S x4) (python 3.11) workflow launched on this PR: View run

Copy link

e2e workflow succeeded on this PR: View run, congrats!

@cdoern
Copy link
Contributor

cdoern commented Jun 10, 2025

@booxter do we instead want: #605 ? I think I got rid of this in there too

@booxter booxter marked this pull request as draft June 10, 2025 17:50
@booxter
Copy link
Contributor Author

booxter commented Jun 10, 2025

Sure! I will make it a draft and close if/when the refactor merges with the change included.

@booxter
Copy link
Contributor Author

booxter commented Jun 18, 2025

The change is included in Charlie's refactoring patch, so no need for a separate PR to handle the same.

@booxter booxter closed this Jun 18, 2025
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