Skip to content
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

Stateful repl menu #21429

Closed
wants to merge 7 commits into from
Closed

Stateful repl menu #21429

wants to merge 7 commits into from

Conversation

rgbkrk
Copy link
Member

@rgbkrk rgbkrk commented Dec 2, 2024

image

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 2, 2024
@rgbkrk rgbkrk force-pushed the stateful-repl-menu branch 3 times, most recently from 61328d8 to be20928 Compare December 10, 2024 21:04
now the repl menu is keeping state around, but not watching that the editor changed (?)

There's a disconnect between selected kernel and the kernel that's running in the editor.
@rgbkrk rgbkrk force-pushed the stateful-repl-menu branch from be20928 to 80c2f9d Compare December 10, 2024 23:00
@rgbkrk
Copy link
Member Author

rgbkrk commented Dec 10, 2024

Dusting this off again... The repl menu is keeping state around, but not seeing that the editor changed. There's a disconnect between selected kernel and the kernel that's running in the editor.

@iamnbutler
Copy link
Member

Hey @rgbkrk, what do we need to get this over the line? Can I help or do you need some help from @osiewicz?

@rgbkrk
Copy link
Member Author

rgbkrk commented Jan 21, 2025

Might be worth just getting inspired by this branch rather than building directly on top of it. Happy to pair sometime.

@maxdeviant
Copy link
Member

Going to close this until we're ready to pick it back up.

@maxdeviant maxdeviant closed this Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants