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

hep_tfds support in raytune #82

Merged
merged 21 commits into from
Sep 30, 2021
Merged

Conversation

erwulff
Copy link
Collaborator

@erwulff erwulff commented Sep 16, 2021

Add support for hep_tfds datasets in raytune command. Also add additional analysis plots to the raytune-analysis command.

@erwulff erwulff changed the title Feat raytune analysis hep_tfds support in raytune Sep 16, 2021
@erwulff erwulff marked this pull request as ready for review September 16, 2021 14:33
@jpata
Copy link
Owner

jpata commented Sep 30, 2021

Thanks for this!

@jpata jpata merged commit 7ddbd34 into jpata:master Sep 30, 2021
jpata pushed a commit that referenced this pull request Sep 15, 2023
* feat: Add more plots to raytune-analysis command

* fix: add install of seaborn to github tests

* feat: hep_tfds support for raytune

* fix: Update parameter name in raytune config

* fix: Timeout error running Ray Tune on HPC systems

* up: update raytune parameters

* fix: cleaner sbatch scripts for raytune

* fix: bug in raytune sbatch script for flatiron

* chore: add check for specification of raytune local_dir

* feat: add plotting script for exported comet.ml json

* chore: Add raytune default_metric to config file

* feat: add more hyperparameters to tune in raytune

* fix: remove extra tab

* feat: time raytune command and log time

* feat: log best parameters after raytune run

* fix: raytune dropout parameter

The raytune dropout parameter now sets both the
combined_graph_layer dropout and the output decoding dropout.
The combined_graph_layer dropout will be set to dropout/2.

Former-commit-id: 7ddbd34
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