Add support for setting WandB Entity / Comet Workspace via experiment config#3627
Open
jannick-schroeer wants to merge 7 commits intonerfstudio-project:mainfrom
Open
Add support for setting WandB Entity / Comet Workspace via experiment config#3627jannick-schroeer wants to merge 7 commits intonerfstudio-project:mainfrom
jannick-schroeer wants to merge 7 commits intonerfstudio-project:mainfrom
Conversation
Collaborator
|
Is this argument useful for other writers, like Comet etc? If not, could we rename it to |
Author
|
It seems comet has something similar to entities called Should I also migrate to the new comet api or would you prefer this to be a separate PR? |
ee88ef3 to
a51eb26
Compare
f3f9ea9 to
9d98a82
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds support for specifying the Weights & Biases (WandB) entity (team or organization) and the Comet workspace directly in the Nerfstudio experiment config.
Why?
Currently, users must either:
This can be limiting when working across multiple teams or accounts. With this change, users can now explicitly set the WandB entity / Comet Workspace in the config through the
workspace_name, making it easier to log runs under different teams or organizations without needing to modify global environment settings.Notes
As this is my first PR to Nerfstudio, so I’m happy to take any suggestions or feedback!