Skip to content

Add data science demos to landing page#2067

Merged
freddyaboulton merged 2 commits into
mainfrom
2048-add-ds-demo-to-landing-page
Aug 24, 2022
Merged

Add data science demos to landing page#2067
freddyaboulton merged 2 commits into
mainfrom
2048-add-ds-demo-to-landing-page

Conversation

@freddyaboulton
Copy link
Copy Markdown
Collaborator

@freddyaboulton freddyaboulton commented Aug 23, 2022

Description

Fixes #2048

Added two demos:

Forecasting, Prophet, Tabular datasets, and Prediction Explainability are all concepts that have come up in our conversations with data scientists (both internal and external) and these demos cover those concepts.

My hope is that putting them in the front page will catch the eyes of prospective data science users. The xgboost example is a sneak peek of what we want to be able to automate for skops models.

image

image

Checklist:

  • I have performed a self-review of my own code
  • My code follows the style guidelines of this project
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@github-actions
Copy link
Copy Markdown
Contributor

All the demos for this PR have been deployed at https://huggingface.co/spaces/gradio-pr-deploys/pr-2067-all-demos

@freddyaboulton freddyaboulton marked this pull request as ready for review August 23, 2022 19:54
<div class="demo-tab hover:text-gray-800 cursor-pointer px-3 py-1"
demo="4"
onclick="load_demo(4)">
Time Series Forecasting With Prophet
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thoughts on shortening the titles so that they fit better and roughly match the length of the titles of the first 3 demos?

Suggested change
Time Series Forecasting With Prophet
Time Series Forecasting

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing! I was just trying to use eye catching terms lol

<div class="demo-tab hover:text-gray-800 cursor-pointer px-3 py-1"
demo="5"
onclick="load_demo(5)">
XGBoost Classification with Explainability
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
XGBoost Classification with Explainability
XGBoost Classification

@abidlabs
Copy link
Copy Markdown
Member

abidlabs commented Aug 23, 2022

Looks great @freddyaboulton! Suggest shortening the titles so that they all fit in one line.

Original:
image

New:
image

Also, it seems we are missing code skeletons for the last 2 demos? For the sake of consistency, I think it would be good to include the skeletons.

image

@freddyaboulton
Copy link
Copy Markdown
Collaborator Author

@abidlabs Good point re the code skeleton!

I deliberately left them out since they are blocks demos and would need more code than the other demos on the landing page. I thought it would be distracting. If we want to include the code skeleton for blocks, what's the right amount? How about the following?

with gr.Blocks() as demo:
   # your forecasting demo here
demo.launch()

@abidlabs
Copy link
Copy Markdown
Member

abidlabs commented Aug 23, 2022

I see, that's a good point. I think that's one option, although the code snippet doesn't really add much.

What about leaving out the code snippet then but adding some text that says, "see full code here" and then linking to https://huggingface.co/spaces/gradio/timeseries-forecasting-with-prophet/tree/main? Is it possible to do that in a way that doesn't distract from the demo?

@abidlabs
Copy link
Copy Markdown
Member

abidlabs commented Aug 23, 2022

Oh nvm you already do that in the demos! Haha that's cool. I'll make some suggestions about the Spaces via PRs on the demos themselves but this LGTM :)

@freddyaboulton freddyaboulton force-pushed the 2048-add-ds-demo-to-landing-page branch from ac4586e to 1634eef Compare August 24, 2022 14:45
@freddyaboulton freddyaboulton merged commit d97fe6e into main Aug 24, 2022
@freddyaboulton freddyaboulton deleted the 2048-add-ds-demo-to-landing-page branch August 24, 2022 15:09
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.

Add a data science demo to the landing page

2 participants