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

A more stable SeaPearlZoo #36

Merged
merged 29 commits into from
May 17, 2022
Merged

A more stable SeaPearlZoo #36

merged 29 commits into from
May 17, 2022

Conversation

louis-gautier
Copy link
Collaborator

This PR aims at enabling to launch experiments in a more stable way.

  • It fixes some bugs which caused errors when launching TSPTW, nqueens and graphcoloring.
  • It automatically saves all the important parameters of the experiments in a params.json file for later reference.
  • It uses better hyperparameters for fast and optimal learning.
  • It is adapted to the new DefaultStateRepresentation features set by setting chosen_features.

louis-gautier and others added 24 commits April 12, 2022 11:15
@louis-gautier
Made knapsack problem functional with metrics export
ed7c8b1
@louis-gautier
Fixed MissingMethod error in tsptw experiment
ad66ec8
@louis-gautier
Adapted TSPTW to DefaultStateRepresentation
50d021f
@louis-gautier
Fixed hardcoding of the number of cities
59eba22
@louis-gautier
Merge branch 'master' into louis/experiments_fixes
9ea00cf
@gostreap
Merge tsptw.jl et tsptw_defaultstaterepresentation.jl into one file
be59294
@louis-gautier
Added FeaturizationHelper experiment for TSPTW
e7f7f91
@louis-gautier
Merge branch 'louis/experiments_fixes' of https://github.com/corail-r…
2411e12 
…esearch/SeaPearlZoo into louis/experiments_fixes
@gostreap
Fix nb_features
6f128af
@louis-gautier
Provisory adjustment to take one-hot encoding of constraint type into…
723adeb 
… consideration
@louis-gautier
Changed values to perform easy experiments on TSPTW
fcc1b5b
@louis-gautier
Merge branch 'master' into louis/experiments_fixes
bf41887
@louis-gautier
Merge branch 'master' into louis/experiments_fixes
6a834e2
@louis-gautier
Adapted agent and problem definition on graphcoloring to automatic ex…
6de3a93 
…periment storage
@louis-gautier
Adapted TSPTW with DefaultStateRepresentation to new pipeline
1e8664a
@louis-gautier
Best current agent hyperparameters for graphcoloring
9dc66f8
@louis-gautier
Better hyperparameters for TSPTW with DefaultStateRepresentation
96fee01
@louis-gautier
More expressive params.json for TSPTW
2634017
@louis-gautier
Better params.json for nqueens
3c41b7f
@louis-gautier
Fixed some bugs with TSPTW
f118dd3
@louis-gautier
Fixed some bugs with nqueens
64298ef
@louis-gautier
Better parameters for graphcoloring and nqueens
042764d
@louis-gautier
Merge remote-tracking branch 'origin/master' into louis/experiments_f…
d6a4982 
…ixes
@louis-gautier
Converting LearnedHeuristic to SimpleLearnedHeuristic
de6c19e
@louis-gautier louis-gautier marked this pull request as ready for review May 11, 2022 21:34
@louis-gautier louis-gautier changed the title Louis/experiments fixes A more stable SeaPearlZoo May 13, 2022
@louis-gautier louis-gautier merged commit 6a2da32 into master May 17, 2022
@ziadelassal ziadelassal mentioned this pull request May 17, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ziadelassal ziadelassal ziadelassal approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@louis-gautier @ziadelassal @gostreap