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

Fix Weights and Biases Logging and paths #2

Merged

hlml merged 4 commits into hlml:main from SauravMaheshkar:main

Apr 16, 2022

Contributor

SauravMaheshkar commented Feb 9, 2022

Hey 👋, I noticed a few errors in the codebase, this PR aims to solve some of them.

Create a minimal .gitignore file, for python and wandb.
Fix wandb.init() statements for the case when no group_vars are provided.
Most paths were of the form /home/* which might lead to permission errors on systems with non-root access, I simply substituted them with ./

Saurav Maheshkar added 4 commits

February 9, 2022 05:04


          Create gitignore


          Fix dependency conflicts

3b5ce47


          Fix wandb init


          Fix paths

4bd88b5

Contributor Author

SauravMaheshkar commented Mar 21, 2022

Request for review @hlml

hlml merged commit aeb8597 into hlml:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment