Skip to content

feat: delay kernel rejection until all kernels have run - #63

Merged
hargoniX merged 1 commit into
masterfrom
hbv/delayed_kernel_rejection
Aug 5, 2026
Merged

feat: delay kernel rejection until all kernels have run#63
hargoniX merged 1 commit into
masterfrom
hbv/delayed_kernel_rejection

Conversation

@hargoniX

@hargoniX hargoniX commented Aug 5, 2026

Copy link
Copy Markdown
Member

No description provided.

@jcreedcmu jcreedcmu left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yes, I think this is good for now, assuming I understand correctly from @robsimmons that comparator-live will simply show all of stdout when any kernel fails. In the future when one kernel fails and another succeeds, it will be easy to conjure up a simple link that clearly shows that happening.

@hargoniX
hargoniX force-pushed the hbv/delayed_kernel_rejection branch from 98c3c66 to a1e39a6 Compare August 5, 2026 13:33
@hargoniX
hargoniX force-pushed the hbv/delayed_kernel_rejection branch from a1e39a6 to 5d21b89 Compare August 5, 2026 13:33
@jcreedcmu

jcreedcmu commented Aug 5, 2026

Copy link
Copy Markdown

although I am confused by "Lean default kernel rejects the solution / Your solution is okay!" in the erstwhile CI failure but I think you just pushed a fix

@hargoniX
hargoniX merged commit 5f2b56b into master Aug 5, 2026
2 checks passed
@hargoniX
hargoniX deleted the hbv/delayed_kernel_rejection branch August 5, 2026 13:49
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