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

Robynpy bugfix graphs #1150

Merged
merged 4 commits into from
Nov 18, 2024
Merged

Conversation

dhavalpatel624624
Copy link

@dhavalpatel624624 dhavalpatel624624 commented Nov 18, 2024

Fixes robyn py in how it selects model before calling allocator

fixed the plot all function to display the cluster plot also
fixed a minor fix to pareto_optimizer where one list was getting set as a set instead of list

Tested with tutorial1 notebook

image

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 18, 2024
Copy link

@sumalreddy17 sumalreddy17 left a comment

Choose a reason for hiding this comment

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

LGTM

@dhavalpatel624624 dhavalpatel624624 merged commit 8c46a29 into robynpy_release Nov 18, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants