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

Provide Express translations of Core example apps #985

Open
6 of 18 tasks
gadenbuie opened this issue Jan 10, 2024 · 0 comments
Open
6 of 18 tasks

Provide Express translations of Core example apps #985

gadenbuie opened this issue Jan 10, 2024 · 0 comments
Labels

Comments

@gadenbuie
Copy link
Collaborator

gadenbuie commented Jan 10, 2024

Background

These apps are all from shiny/examples but as far as we can tell they are not directly exposed to users, and so translating them is not on the critical path for the Shiny Express release. We still may want to translate them as an exercise, but it should go to the bottom of the list.

Process

  1. Before working on an app, add your github handle next to the task item or task group that you're working on, so others don't duplicate your work.
  2. Create an alternative app-express.py app that provides an Express variation of the Shiny Core apps listed below.
  3. If you start to work on an app and decide we should not translate it, please leave a note as to why not.
  4. Submit PRs against main with as many or as few apps as you were able to translate in a session. Please tag @gadenbuie for review so I can track progress.

Tip

I created an example PR translating the input_action_button() example from Core to Express syntax. You can use this example as a reference. Translating an example app below will take more work, but follows a similar pattern.

Apps

  • examples/airmass/app.py (GS: We should remove this app, it's not beautiful or particularly impressive, only meaningful to people who understand celestial bodies, and the map selection isn't a very good experience)
  • examples/annotation-export/app.py (GS: Brushing now supported in express Plot brushing API in Express #993)
  • examples/brownian/app.py (GS: This app has an npm install step, which makes me think we should remove it)
  • examples/cpuinfo/app.py
  • examples/dataframe/app.py (GS: Not exposed to users)
  • examples/duckdb/app.py (GS: Can't translate as this uses modules)
  • examples/event/app.py
  • examples/global_pyplot/app.py
  • examples/headers/app.py
  • examples/inputs-update/app.py
  • examples/load_balance/app.py
  • examples/model-score/app.py
  • examples/moduleapp/app.py
  • examples/penguins/app.py
  • examples/req/app.py
  • examples/static_plots/app.py
  • examples/typed_inputs/app.py
  • examples/ui-func/app.py
@gadenbuie gadenbuie mentioned this issue Jan 10, 2024
55 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant