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

Added util to print gif #164

Merged
merged 3 commits into from
Jul 8, 2021
Merged

Added util to print gif #164

merged 3 commits into from
Jul 8, 2021

Conversation

jonpsy
Copy link
Member

@jonpsy jonpsy commented Jul 1, 2021 •
edited
Loading

Yet to test its working via a simple cpp file.

jonpsy added 2 commits July 6, 2021 12:04
@jonpsy
=> Change fps to 2
ca593a9 
=> Set title
=> Use default color
@jonpsy
fix numargs
240d604
@jonpsy
Copy link
Member Author

jonpsy commented Jul 6, 2021

@zoq Good news! laptops back, off with codespaces! The gif is generating as expected!!

@jonpsy
Copy link
Member Author

jonpsy commented Jul 6, 2021 •
edited
Loading

fronts

👓 👓 @zoq @say4n , Notice the range the solutions cover. NSGA2 is within 0 -> 0.35 , whereas MOEAD goes as far as 0.8, that too much faster. That should bring things to perspective on why MOEAD is exponentially better than NSGA-II.

@jonpsy
Copy link
Member Author

jonpsy commented Jul 6, 2021

Produced from the xeus-notebook itself.

@zoq
Copy link
Member

zoq commented Jul 6, 2021

Nice, I guess the remaining work for this one is to integrate the plot into the existing notebook?

@jonpsy
Copy link
Member Author

jonpsy commented Jul 6, 2021 •
edited
Loading

Already done. In the Portfolio PR

zoq
zoq approved these changes Jul 6, 2021
View reviewed changes
Copy link
Member

@zoq zoq left a comment

Choose a reason for hiding this comment

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

Looks good to me, makes the plot even more interesting.

mlpack-bot[bot]
mlpack-bot bot approved these changes Jul 7, 2021
View reviewed changes
Copy link

@mlpack-bot mlpack-bot bot left a comment

Choose a reason for hiding this comment

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

Second approval provided automatically after 24 hours. 👍

@zoq zoq merged commit f04b427 into mlpack:master Jul 8, 2021
@jonpsy jonpsy deleted the gif branch July 8, 2021 01:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mlpack-bot mlpack-bot[bot] mlpack-bot[bot] approved these changes

@zoq zoq zoq approved these changes

Assignees
No one assigned
Labels
t: added feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jonpsy @zoq