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

docs(weave): Document batch delete calls #3932

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

docs(weave): Document batch delete calls #3932

wants to merge 3 commits into from

Conversation

J2-D2-3PO
Copy link
Contributor

@J2-D2-3PO J2-D2-3PO commented Mar 21, 2025

Description

  1. Documents weave client batch delete of calls as shown in merged chore(weave): weave client batch delete calls #3903
  2. Modifies all non-title headers in page to sentence case unless its proper noun (Calls)
  3. Replaces a few bolded subsection titles with H4 headers

Testing

yarn start on local

@J2-D2-3PO J2-D2-3PO requested review from a team as code owners March 21, 2025 17:30
@circle-job-mirror
Copy link

circle-job-mirror bot commented Mar 21, 2025


<Tabs groupId="client-layer">
<TabItem value="python_sdk" label="Python">
To delete large batches of Calls using the Python API, pass a list of Call IDs to `delete_calls()`.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gtarpenning I see that we reference the autogenerated client info for delete_call here (Call.Delete) https://weave-docs.wandb.ai/guides/tracking/tracing#delete-a-call

I want to add the same thing in for delete_calls, but I don't see information in the client page that I can link to, just the singular delete method https://weave-docs.wandb.ai/reference/python-sdk/weave/trace/weave.trace.weave_client

@J2-D2-3PO J2-D2-3PO requested a review from gtarpenning March 21, 2025 17:41
@J2-D2-3PO J2-D2-3PO changed the title Document batch delete calls docs(weave): Document batch delete calls Mar 21, 2025
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.

1 participant