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

[Feature]: Improved documentation for WANDB_SWEEP_ID env variable #4736

Open
mklasby opened this issue Jan 6, 2023 · 6 comments
Open

[Feature]: Improved documentation for WANDB_SWEEP_ID env variable #4736

mklasby opened this issue Jan 6, 2023 · 6 comments
Labels
c:doc Component: Docs c:sweeps Component: Sweeps ty:feature_request type of the issue is a feature request

Comments

@mklasby
Copy link

mklasby commented Jan 6, 2023

Description

Documentation for enviornment variables does not include information on WANDB_SWEEP_ID. This can lead to a problem when calling wandb.init() with id, project, or entity parameters if this enviornment variable has been set. This can result in crashes or unexpected behaviour, particularlly when trying to a resume a run that was part of a sweep but crashed/ failed prematurely.

Suggested Solution

Add information regarding WANDB_SWEEP_ID to the online documentation

Alternatives

Do not ignore wandb.init() parameters when script is executed directly by user rather than sweep agent initalized with wandb agent. Provide warnings to user if env variable is set and wandb agent is not used to start run.

Additional Context

No response

@mklasby mklasby added the ty:feature_request type of the issue is a feature request label Jan 6, 2023
@luisbergua
Copy link
Contributor

Hi @mklasby, thanks for suggesting this! I'll share this feedback with our team.

@kptkin kptkin added c:doc Component: Docs c:sweeps Component: Sweeps labels Mar 8, 2023
@rajveer43
Copy link

Is this issue open for work?

@luisbergua
Copy link
Contributor

Hey @rajveer43, are you asking about submitting a PR to add this information to our docs?

@rajveer43
Copy link

Hey @rajveer43, are you asking about submitting a PR to add this information to our docs?

yes!

@luisbergua
Copy link
Contributor

Hi @rajveer43, thank you for your patience! After checking this internally, our team would like to connect with you so I have just sent an email introducing the team, will follow up there!

@rajveer43
Copy link

thank you. I have replied to that email!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:doc Component: Docs c:sweeps Component: Sweeps ty:feature_request type of the issue is a feature request
Projects
None yet
Development

No branches or pull requests

4 participants