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

Website Gallery page building #850

Merged
merged 22 commits into from
Dec 8, 2023
Merged

Website Gallery page building #850

merged 22 commits into from
Dec 8, 2023

Conversation

skzhang1
Copy link
Collaborator

@skzhang1 skzhang1 commented Dec 3, 2023

Why are these changes needed?

There are many interesting examples and resources from discord, twitter, linkedin, medium, youtube, reddit, github.
We need to build a gallery page to collect these interesting examples and encourage the community to enrich the content.

Related issue number

Closes #830

Tasks

  • Change the gallery page style. Show each example with title + icon + link. Now it is a markdown page.

Checks

@codecov-commenter
Copy link

codecov-commenter commented Dec 3, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 26.58%. Comparing base (1b4fb8f) to head (e0bbe74).
Report is 786 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #850   +/-   ##
=======================================
  Coverage   26.58%   26.58%           
=======================================
  Files          28       28           
  Lines        3732     3732           
  Branches      847      847           
=======================================
  Hits          992      992           
  Misses       2667     2667           
  Partials       73       73           
Flag Coverage Δ
unittests 26.52% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@skzhang1 skzhang1 added the documentation Improvements or additions to documentation label Dec 3, 2023
@sonichi
Copy link
Contributor

sonichi commented Dec 3, 2023

Does the page render as a gallery?

@skzhang1
Copy link
Collaborator Author

skzhang1 commented Dec 3, 2023

@sonichi Now it is rendered using a markdown page. I am now trying to make it render as a gallery.

Copy link
Contributor

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

Is it possible to add tags for each? It'll be hard to browse.

website/docs/Gallery.md Outdated Show resolved Hide resolved
website/docs/Gallery.md Outdated Show resolved Hide resolved
@skzhang1
Copy link
Collaborator Author

skzhang1 commented Dec 3, 2023

Is it possible to add tags for each? It'll be hard to browse.

Screenshot 2023-12-03 at 14 25 07

Currently I show the catalog on the right. Users could also search them using the search box on the top.

@sonichi
Copy link
Contributor

sonichi commented Dec 3, 2023

Is it possible to add tags for each? It'll be hard to browse.

Screenshot 2023-12-03 at 14 25 07

Currently I show the catalog on the right. Users could also search them using the search box on the top.

adding tag will help search.

@victordibia
Copy link
Collaborator

@sonichi Now it is rendered using a markdown page. I am now trying to make it render as a gallery.

Hi @skzhang1 , my current intuition is that we will benefit from building a custom (reusable) react component that can format data as a gallery. I am not sure how to accomplish it yet, but can do some research and share my findings.

@victordibia
Copy link
Collaborator

victordibia commented Dec 4, 2023

My findings so far are as follows:

  • We can create an .mdx file instead of .md for the galler. .mdx lets us add custom React components.
  • We can define a gallery.json and represent each item as an entry
[
  {
    "title": "AutoGen Playground",
    "link": "https://huggingface.co/spaces/thinkall/AutoGen_Playground",
    "description": "A space to explore the capabilities of AutoGen.",
    "image": "default.png",
    "tags": ["web", "app"]
  },
  ...
  ]
  • Create a custom react component that loads and renders the gallery. We can implement behaviors like search by tags etc.

I have some sample code for this .. the result being below.

@skzhang1 is it ok if I committed the code to this PR and then you can modify it as needed? let me know.

@sonichi .. is this close to what you are envisioning?

image

@sonichi
Copy link
Contributor

sonichi commented Dec 4, 2023

Yes. Looks great!

@skzhang1
Copy link
Collaborator Author

skzhang1 commented Dec 4, 2023

@sonichi Now it is rendered using a markdown page. I am now trying to make it render as a gallery.

Hi @skzhang1 , my current intuition is that we will benefit from building a custom (reusable) react component that can format data as a gallery. I am not sure how to accomplish it yet, but can do some research and share my findings.

@victordibia It looks great to me! Can you make a commit for that? We will enrich the content correspondingly.

@skzhang1 skzhang1 mentioned this pull request Dec 4, 2023
@sonichi sonichi mentioned this pull request Dec 4, 2023
2 tasks
@victordibia
Copy link
Collaborator

victordibia commented Dec 4, 2023

@victordibia It looks great to me! Can you make a commit for that? We will enrich the content correspondingly.

@skzhang1, I made a commit. Thanks for taking the lead and helping with this feature, it will really help the community
Please feel free to double check and modify the content of gallery.json for completeness.

Note: You might have to run yarn install again to update dependencies (mainly support the the antd library)

@skzhang1
Copy link
Collaborator Author

skzhang1 commented Dec 4, 2023

@victordibia Thanks for your help! I made small changes to the content. Now this PR looks good to me. I will add icons to these applications in another PR. I think we can merge these PR and encourage uses to submit their demos in the community at first. Can you check the content again? @sonichi @qingyun-wu Thanks!

Copy link
Collaborator

@victordibia victordibia left a comment

Choose a reason for hiding this comment

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

Looks good to me.
I agree that it might be a good idea add some icons or color coding to each tag.
We might also want an author field to indicate if it is a community example or by the autogen team. And yes, let us work on that on a different PR.

Thanks again for working on this!

Copy link
Contributor

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

I didn't go through all the examples. I checked a few and commented.

website/docs/Gallery.mdx Show resolved Hide resolved
website/src/data/gallery.json Outdated Show resolved Hide resolved
website/src/data/gallery.json Outdated Show resolved Hide resolved
website/src/data/gallery.json Outdated Show resolved Hide resolved
@skzhang1 skzhang1 requested a review from sonichi December 5, 2023 15:05
Copy link
Contributor

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

For the case when image is not provided, I meant rendering the image contained in the linked page, like the gallery forum in discord: https://discord.com/channels/1153072414184452236/1161015724521836634
It it's hard to do it here, we can use a common default image as proposed, but the default image needs to be replaced by a more commonly representative image like the autogen landing image.

website/docs/Ecosystem.md Outdated Show resolved Hide resolved
website/docs/img/ecosystem_SemanticKernel.png Outdated Show resolved Hide resolved
website/src/data/gallery.json Outdated Show resolved Hide resolved
website/src/data/gallery.json Outdated Show resolved Hide resolved
website/src/data/gallery.json Show resolved Hide resolved
website/static/img/gallery/default.png Outdated Show resolved Hide resolved
@qingyun-wu qingyun-wu added this pull request to the merge queue Dec 8, 2023
Merged via the queue into main with commit 69aaea5 Dec 8, 2023
16 checks passed
@victordibia victordibia deleted the web_gallery branch December 18, 2023 01:28
rlam3 pushed a commit to rlam3/autogen that referenced this pull request Dec 19, 2023
* initial draft

* update dependences, add antd for layout (note yarn install needed her)

* update gallery to use mdx file format to support react components

* add custom react component for gallery with tag based filtering and url state management

* add styling for gallery objects

* add gallery data structure and default image

* improve layout for gallerypage

* revise Postgres demo

* add default value when image is not provided

* move semetic kernal to ecosystem

* update tags

* update tags

* update ecosystem

* update default fig

* update tags

* reformat

---------

Co-authored-by: “skzhang1” <“shaokunzhang529@gmail.com”>
Co-authored-by: Victor Dibia <victordibia@microsoft.com>
whiskyboy pushed a commit to whiskyboy/autogen that referenced this pull request Apr 17, 2024
…s specified, then effectively zero-shot AutoML is used (microsoft#850)

* Added an info reminding user that if no time_budget and no max_iter is specified, then effectively zero-shot AutoML is used

* moved message to line 2818

* Update flaml/automl/automl.py

Co-authored-by: Chi Wang <wang.chi@microsoft.com>

* Update flaml/automl/automl.py

Co-authored-by: Chi Wang <wang.chi@microsoft.com>

Co-authored-by: Chi Wang <wang.chi@microsoft.com>
Co-authored-by: Qingyun Wu <qingyun.wu@psu.edu>
whiskyboy pushed a commit to whiskyboy/autogen that referenced this pull request Apr 17, 2024
* initial draft

* update dependences, add antd for layout (note yarn install needed her)

* update gallery to use mdx file format to support react components

* add custom react component for gallery with tag based filtering and url state management

* add styling for gallery objects

* add gallery data structure and default image

* improve layout for gallerypage

* revise Postgres demo

* add default value when image is not provided

* move semetic kernal to ecosystem

* update tags

* update tags

* update ecosystem

* update default fig

* update tags

* reformat

---------

Co-authored-by: “skzhang1” <“shaokunzhang529@gmail.com”>
Co-authored-by: Victor Dibia <victordibia@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Discoverability of examples and resources
6 participants