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

ENH: Add property to discard logging training to Comet ML #49

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jhlegarreta
Copy link
Contributor

Add training configuration flag and the corresponding experiment class property to discard logging the training process to Comet ML.

It may be useful for users that do not have a Comet ML account or are not willing to log the experiment.

Fixes #46.

@jhlegarreta
Copy link
Contributor Author

Have not tested.

@github-actions github-actions bot added area:config Items affecting the configuration module area:learning Items affecting the learning module type:enhancement New feature or request labels Feb 14, 2023
@jhlegarreta jhlegarreta force-pushed the AddFlagToDiscardLoggingToComentML branch from 6f67d5a to 6cd53ae Compare February 14, 2023 15:07
@github-actions github-actions bot removed the type:enhancement New feature or request label Feb 14, 2023
@jhlegarreta
Copy link
Contributor Author

Not sure why pytdot is failing. Maybe there is an issue with the latest version, and we need to pin it to a given version.

Add training configuration flag and the corresponding experiment class
property to discard logging the training process to `Comet ML`.

It may be useful for users that do not have a `Comet ML` account or are
not willing to log the experiment.
@jhlegarreta jhlegarreta force-pushed the AddFlagToDiscardLoggingToComentML branch from 6cd53ae to 6b61ff4 Compare March 3, 2023 18:38
@github-actions github-actions bot added the type:enhancement New feature or request label Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:config Items affecting the configuration module area:learning Items affecting the learning module type:enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a condition in ae_train for the usage of COMET_ML
1 participant