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

feat(datasets): add span to dataset from the trace page #3230

Merged
merged 7 commits into from
May 24, 2024

Conversation

mikeldking
Copy link
Contributor

@mikeldking mikeldking commented May 17, 2024

resolves #3208
Screenshot 2024-05-23 at 4 13 39 PM

Adds the ability to add a span to a dataset.Has some follow-up UI elements that will be filed separately.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@mikeldking mikeldking changed the base branch from main to datasets May 17, 2024 20:34
@mikeldking mikeldking changed the title 3208 add span to dataset feat(datasets): add span to dataset from the trace page May 23, 2024
@mikeldking mikeldking marked this pull request as ready for review May 23, 2024 23:44
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label May 23, 2024
app/src/pages/trace/SpanToDatasetExampleDialog.tsx Outdated Show resolved Hide resolved
}) => (
<Card
title="Metadata"
subTitle="All data from the span to use during experimentation or evaluation"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
subTitle="All data from the span to use during experimentation or evaluation"
subTitle="Span attributes that can be used during experimentation or evaluation"

app/src/pages/trace/SpanToDatasetExampleDialog.tsx Outdated Show resolved Hide resolved
@axiomofjoy
Copy link
Contributor

Noticing we don't have a close button on this slideover like we do for other slideovers.

Screenshot 2024-05-24 at 1 22 32 PM

mikeldking and others added 3 commits May 24, 2024 16:52
Co-authored-by: Xander Song <axiomofjoy@gmail.com>
Co-authored-by: Xander Song <axiomofjoy@gmail.com>
Co-authored-by: Xander Song <axiomofjoy@gmail.com>
@mikeldking
Copy link
Contributor Author

works on my machine? Let me check with you later.
Screenshot 2024-05-24 at 4 57 16 PM

@mikeldking mikeldking merged commit ed34143 into datasets May 24, 2024
12 checks passed
@mikeldking mikeldking deleted the 3208-add-span-to-dataset branch May 24, 2024 22:58
mikeldking added a commit that referenced this pull request Jun 1, 2024
* feat(datasets): add span to dataset

* structer guidelines

* Add tests

* add tests

* Update app/src/pages/trace/SpanToDatasetExampleDialog.tsx

Co-authored-by: Xander Song <axiomofjoy@gmail.com>

* Update app/src/pages/trace/SpanToDatasetExampleDialog.tsx

Co-authored-by: Xander Song <axiomofjoy@gmail.com>

* Update app/src/pages/trace/SpanToDatasetExampleDialog.tsx

Co-authored-by: Xander Song <axiomofjoy@gmail.com>

---------

Co-authored-by: Xander Song <axiomofjoy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants