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

Add ⭐️Weights & Biases⭐️ Logging #22

Closed
wants to merge 2 commits into from

Conversation

SauravMaheshkar
Copy link

This PR aims to add basic Weights and Biases Metric Logging by appending to the existing codebase with minimal changes while supporting Checkpoint uploads as Weights and Biases Artifacts.

Wherever needed, I have used the existing Weights and Biases integrations viz. LightGBM and XGBoost.

I have validated the performance of all the proposed runs by running 150+ runs, which can be viewed on this project page and in detail in an accompanying blog post.

@SauravMaheshkar
Copy link
Author

Request for Review @Yura52

@Yura52
Copy link
Collaborator

Yura52 commented Feb 25, 2022

@SauravMaheshkar sorry for the slow reply.

Thank you for the report and the pull request!
If I understand correctly, the report contains results for all the models on the California Housing dataset.
Nice to see that they are close to what we report in the paper.

As for the pull request, as of now, we would like to keep the implementation of the paper unchanged and only consider patches that are some kind of fixes. However, we can add a link to the pull request (or to the public fork, if available) in README.

@Yura52
Copy link
Collaborator

Yura52 commented Mar 12, 2022

Feel free to reopen if needed, or add comments and continue the discussion!

@Yura52 Yura52 closed this Mar 12, 2022
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.

3 participants