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.

Sign up for GitHub

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

Jump to bottom

Support for discrete actions #810

Merged
merged 3 commits into from
Mar 6, 2025
Merged

Support for discrete actions #810

merged 3 commits into from
Mar 6, 2025

Conversation

sandhawalia
Copy link
Contributor

@sandhawalia sandhawalia commented Mar 4, 2025 •
edited
Loading

[visualization] Add discrete actions to table action.discrete

Would go alongside action.continuous
image

@imstevenpmwork imstevenpmwork added enhancement Suggestions for new features or improvements visualization Issues about visual output, graphs, or data visualization labels Mar 4, 2025
@sandhawalia
Copy link
Contributor Author

@mishig25

@mishig25 mishig25 self-requested a review March 5, 2025 09:45
@mishig25
Copy link
Collaborator

mishig25 commented Mar 5, 2025 •
edited
Loading

LGTM!

image

@mishig25
Copy link
Collaborator

mishig25 commented Mar 5, 2025 •
edited
Loading

@aliberts @michel-aractingi could you merge the PR?

@aliberts aliberts merged commit 10706ed into huggingface:main Mar 6, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aliberts aliberts aliberts approved these changes

@mishig25 mishig25 Awaiting requested review from mishig25

Assignees
No one assigned
Labels
enhancement Suggestions for new features or improvements visualization Issues about visual output, graphs, or data visualization
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sandhawalia @mishig25 @aliberts @imstevenpmwork