-
Notifications
You must be signed in to change notification settings - Fork 6
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
[DOCS-466] Standardize docs about creating API keys and signing in #1054
base: main
Are you sure you want to change the base?
Conversation
- Update each integration to use the same headings and structure - Fix other section headings in the integrations - Use the same details in the public API guide - Use the same details in the quickstart
Deploying docs with Cloudflare Pages
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo, code example fix, and other small nits.
@@ -240,9 +250,9 @@ wandb.config.update({"model_parameters": 3500}) | |||
|
|||
For more information on how to define a config file, see [Configure Experiments with wandb.config]({{< relref "/guides/models/track/config" >}}). | |||
|
|||
### Logging To W&B | |||
### Log o W&B |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo
|
||
```python |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WandbLogger
refers to a Class that makes it easier to log checkpoints, metrics, etc. for folks using DeepChem.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I.e. So you still need the import statements.
{{% /tab %}} | ||
|
||
{{% tab header="Python" value="python" %}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The example below could be Python, but you'd need to remove the !
in the pip install statement.
Or, rename "Python" to "Python notebook", "Juptyer Notebook", "Google Colab", etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This applies to this example, and all the other markdown files in this PR.
{{< /tabpane >}} | ||
|
||
### 2. Name the project | ||
If you are using Weights and Biases for the first time you might want to check out our [**quickstart**]({{< relref "/guides/quickstart.md" >}}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Rename "Weights & Biases" to "W&B"
content/guides/integrations/yolox.md
Outdated
|
||
## Log metrics | ||
|
||
Use the `--logger wandb` command line argument to turn on logging with wandb. Optionally you can also pass all of the arguments that [wandb.init]({{< relref "/ref/python/init" >}}) would expect, just prepend `wandb-` to the start of each argument |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use wandb
if referring to Py module. Or, if referring to the company, use "W&B"
DOCS-466 Standardize docs about creating API keys and signing in
The original request asked for a dedicated page with these details, and to point the integrations at that page. That's not ideal because they need to go read that page and then come back to the integration page. Ideally that "Create an API key" page would be a sibling to the Quickstart, but I think it would actually look quite strange. See the left hand when you are viewing https://docs.wandb.ai/quickstart/ to see what I mean.
Ideally I could save this content to one or two includes which could then be included by each of the integrations, but I ran into issues with caching and with Docsy shortcodes not being evaluated within the includes. So this uses a more brute force approach for now.