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

IntSlider() #96

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

IntSlider() #96

wants to merge 23 commits into from

Conversation

romainfrancois
Copy link
Contributor

No description provided.

@romainfrancois
Copy link
Contributor Author

ping @SylvainCorlay much still needs to happen, but this is a first draft IntSlider() widget being created (along with its 3 comms), and interacted with (with update msg from the front end, and echo_update as a reply).

Screen.Recording.2024-09-11.at.13.40.17.mov

@romainfrancois
Copy link
Contributor Author

played around some more with a second widget: Button() to try out some interaction.

Screen.Recording.2024-09-11.at.18.37.41.mov

There probably is still a lot to do in terms of refactoring etc ... but the game was to get something that does something. Currently with the interface:

# slider: 
$on_update(handler)  # takes a functio that is called back with state when the comm receives an update msg
$update(...)                  # send an update message to the front end with the state from ...

# button
$on_click(handler)      # handler is called when the button is clicked

@romainfrancois romainfrancois marked this pull request as ready for review September 12, 2024 07:28
} else {
p_interpreter->publish_stream("stdout", output);
// std::cerr << output << std::endl;
// std::cerr << output ;
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to keep the std::cerr portion?

@SylvainCorlay
Copy link
Member

@romainfrancois btw, would you be able to re-open these PRs from folks rather than the main repository?

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