Skip to content

WEAVE: Accumulator documentation#2464

Merged
dbrian57 merged 4 commits intomainfrom
weave/accumulator
Apr 21, 2026
Merged

WEAVE: Accumulator documentation#2464
dbrian57 merged 4 commits intomainfrom
weave/accumulator

Conversation

@dbrian57
Copy link
Copy Markdown
Contributor

@dbrian57 dbrian57 commented Apr 15, 2026

Description

Resolves WBDOCS-409. Adds a new section to the generator function docs that teaches users how to use the accumulator parameter with weave.op().

@dbrian57 dbrian57 requested a review from a team as a code owner April 15, 2026 19:34
@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Apr 15, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
wandb 🟢 Ready View Preview Apr 15, 2026, 7:37 PM

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 15, 2026

📚 Mintlify Preview Links

🔗 View Full Preview

📝 Changed (1 total)

📄 Pages (1)

File Preview
weave/guides/tracking/trace-generator-func.mdx Trace Generator Func

🤖 Generated automatically when Mintlify deployment succeeds
📍 Deployment: d16c78a at 2026-04-21 14:26:18 UTC

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 15, 2026

🔗 Link Checker Results

All links are valid!

No broken links were detected.

Checked against: https://wb-21fd5541-weave-accumulator.mintlify.app

Copy link
Copy Markdown
Contributor

@anastasiaguspan anastasiaguspan left a comment

Choose a reason for hiding this comment

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

capitalized Ops/Calls throughout - including the older content :) I must have missed this file. lots of 'calls' on this page, that aren't Calls - do double check me. but i think this is one of those pages where the caps Calls helps clarify over function calls.

Comment thread weave/guides/tracking/trace-generator-func.mdx Outdated
Comment thread weave/guides/tracking/trace-generator-func.mdx Outdated
Comment thread weave/guides/tracking/trace-generator-func.mdx Outdated
Comment thread weave/guides/tracking/trace-generator-func.mdx Outdated

For more on decorating functions and methods with `@weave.op`, see [Create calls](/weave/guides/tracking/create-call).

## Use the accumulator parameter
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

To make this header something people would be looking for, perhaps describing the function would be better? like:
Accumulate yielded values into a single trace

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I totally meant to change this, but that is a very good header and I'm gonna use it.

dbrian57 and others added 2 commits April 21, 2026 10:22
Co-authored-by: Anastasia Guspan <happyguspan@gmail.com>
@dbrian57 dbrian57 merged commit 05749da into main Apr 21, 2026
9 checks passed
@dbrian57 dbrian57 deleted the weave/accumulator branch April 21, 2026 14:47
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.

2 participants