Skip to content

New explicit archetype API for logging barcharts#3400

Merged
jleibs merged 13 commits intomainfrom
jleibs/barchart_archetype
Sep 22, 2023
Merged

New explicit archetype API for logging barcharts#3400
jleibs merged 13 commits intomainfrom
jleibs/barchart_archetype

Conversation

@jleibs
Copy link
Copy Markdown
Contributor

@jleibs jleibs commented Sep 21, 2023

What

Introduce a barchart archetype as a semantic helper instead of just logging 1-d Tensors.

Update the view to use it, and add a couple small examples.

Resolves: #3327

Checklist

@jleibs jleibs marked this pull request as ready for review September 21, 2023 21:43
@jleibs jleibs added codegen/idl sdk-python Python logging API sdk-rust Rust logging API labels Sep 21, 2023
@Wumpf Wumpf self-requested a review September 22, 2023 06:38
Copy link
Copy Markdown
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

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

neat!

@jleibs jleibs changed the title Make barchart it's own archetype New explicit archetype API for logging barcharts Sep 22, 2023
@jleibs jleibs merged commit e674332 into main Sep 22, 2023
@jleibs jleibs deleted the jleibs/barchart_archetype branch September 22, 2023 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BarChart should have its own archetype

2 participants