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

[EXAMPLES] Create dashboard example #5488

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

frascuchon
Copy link
Member

Description

This PR shows how to create a dataset annotation dashboard using the new feature included in #5479

Type of change

  • Improvement (change adding some improvement to an existing functionality)
  • Documentation update

How Has This Been Tested

Checklist

  • I added relevant documentation
  • I followed the style guidelines of this project
  • I did a self-review of my code
  • I made corresponding changes to the documentation
  • I confirm My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

Copy link
Contributor

@burtenshaw burtenshaw left a comment

Choose a reason for hiding this comment

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

Looks ok. In general I think the dashboard could be simpler with less details.
How does it use webhooks?

# Argilla Dashboard example

## Description
This is an example of a dashboard created with Argilla. The dashboard is a simple gradio app that
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
This is an example of a dashboard created with Argilla. The dashboard is a simple gradio app that
This is an example of a dashboard created with Gradio and Argilla. The dashboard is a Gradio app that

return fig


def create_treemap(user_annotations, total_records):
Copy link
Contributor

Choose a reason for hiding this comment

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

@davidberenstein1957 Do you think we could simplify this?

Choose a reason for hiding this comment

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

I think we can maybe just add percentages to the table? I am not 100% sure if the treemap adds a lot of value compared to that.

@@ -0,0 +1,205 @@
# app dashboard from https://huggingface.co/spaces/davanstrien/argilla-progress/blob/main/app.py

Choose a reason for hiding this comment

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

I think we could add a duplicate space button too.https://www.gradio.app/docs/gradio/duplicatebutton

@@ -0,0 +1,3 @@
gradio
argilla @ git+https://github.com/argilla-io/argilla.git@develop#subdirectory=argilla

Choose a reason for hiding this comment

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

we should not forget to update this

return fig


def create_treemap(user_annotations, total_records):

Choose a reason for hiding this comment

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

I think we can maybe just add percentages to the table? I am not 100% sure if the treemap adds a lot of value compared to that.

with gr.Blocks() as demo:
gr.Markdown("# Argilla Dataset Dashboard")

datasets_dropdown = gr.Dropdown(label="Select your dataset")

Choose a reason for hiding this comment

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

I think it would be nice to pre-select a dataset instead of entering on an empty screen.

}


def create_gauge_chart(progress):

Choose a reason for hiding this comment

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

should we add a legend to this chart? It isn't too clear what the colors red vs blue mean.

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.

3 participants