Merge branch 'main' into pop_over

* main:
  feat(Session): Make Session on_flush() and on_flushed() accept async functions (#693)
  Make data frame selection return row numbers, not pandas index value (#677)
  chore(api)!: Rename `ui.navset_pill_card` -> `ui.navset_card_pill` and `ui.navset_tab_card` -> `ui.navset_card_tab` (#681)
  Consolidate all testing into `tests/` folder (#683)
  Fix pyright error (#678)
  Make model score app work on Connect/Shinyapps.io (#657)
  Suppress type check for read_csv
  Synchonize input_file examples
  More realistic file import example (#582)
  Make flaky dataframe test have larger timeout (#675)
  Wrap bare value box value in `p` tag (#668)

Author: schloerke

CHANGELOG.md

@@ -8,22 +8,33 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [UNRELEASED]
 
-### Experimental breaking changes
-
-* `shiny.experimental.ui.sidebar_toggle()` has been renamed to `shiny.experimental.ui.toggle_sidebar()` (#680).
-* `shiny.experimental.ui.tooltip_toggle()` has been renamed to `shiny.experimental.ui.toggle_tooltip()` (#680).
-* `shiny.experimental.ui.tooltip_update()` has been renamed to `shiny.experimental.ui.update_tooltip()` (#680).
-
-
 ### New features
 
 * Added `shiny.render.renderer_components` decorator to help create new output renderers (#621).
 * Added `shiny.experimental.ui.popover()`, `update_popover()`, and `toggle_popover()` for easy creation (and server-side updating) of [Bootstrap popovers](https://getbootstrap.com/docs/5.2/components/popovers/). Popovers are similar to tooltips, but are more persistent, and should primarily be used with button-like UI elements (e.g. `input_action_button()` or icons) (#680).
 * Added `shiny.experimental.ui.toggle_switch()` (#680).
 * Added CSS classes to UI input methods (#680) .
+* `Session` objects can now accept an asynchronous (or synchronous) function for `.on_flush(fn=)`, `.on_flushed(fn=)`, and `.on_ended(fn=)` (#686).
+
+### API changes
+
+* Renamed `shiny.ui.navset_pill_card` to `shiny.ui.navset_card_pill`. `shiny.ui.navset_pill_card` will throw a deprecated warning (#492).
+* Renamed `shiny.ui.navset_tab_card` to `shiny.ui.navset_card_tab`. `shiny.ui.navset_tab_card` will throw a deprecated warning (#492).
+
+#### Experimental API changes
+
+* Renamed `shiny.experimental.ui.navset_pill_card` to `shiny.experimental.ui.navset_card_pill` (#492).
+* Renamed `shiny.experimental.ui.navset_tab_card` to `shiny.experimental.ui.navset_card_tab` (#492).
+* Renamed `shiny.experimental.ui.sidebar_toggle()` to `shiny.experimental.ui.toggle_sidebar()` (#680).
+* Renamed `shiny.experimental.ui.tooltip_toggle()` to `shiny.experimental.ui.toggle_tooltip()` (#680).
+* Renamed `shiny.experimental.ui.tooltip_update()` to `shiny.experimental.ui.update_tooltip()` (#680).
+
 
 ### Bug fixes
 
+* Fixed #646: Wrap bare value box value in `<p />` tags. (#668)
+* Fixed #676: The `render.data_frame` selection feature was underdocumented and buggy (sometimes returning `None` as a row identifier if the pandas data frame's index had gaps in it). With this release, the selection is consistently a tuple of the 0-based row numbers of the selected rows--or `None` if no rows are selected. (#677)
+
 ### Other changes
 
 

Makefile

@@ -77,11 +77,11 @@ check: ## check code quality with black and isort
 	isort --check-only --diff .
 
 test: ## run tests quickly with the default Python
-	python3 tests/asyncio_prevent.py
-	pytest tests
+	python3 tests/pytest/asyncio_prevent.py
+	pytest
 
 # Default `FILE` to `e2e` if not specified
-FILE:=e2e
+FILE:=tests/e2e
 
 e2e: ## end-to-end tests with playwright
 	playwright install --with-deps

docs/_quartodoc.yml

@@ -77,9 +77,9 @@ quartodoc:
         - ui.nav_spacer
         - ui.nav_menu
         - ui.navset_tab
-        - ui.navset_tab_card
+        - ui.navset_card_tab
         - ui.navset_pill
-        - ui.navset_pill_card
+        - ui.navset_card_pill
         - ui.navset_pill_list
         - ui.navset_hidden
     - title: UI panels
@@ -247,8 +247,8 @@ quartodoc:
             - experimental.ui.layout_sidebar
             - experimental.ui.page_navbar
             - experimental.ui.navset_bar
-            - experimental.ui.navset_tab_card
-            - experimental.ui.navset_pill_card
+            - experimental.ui.navset_card_tab
+            - experimental.ui.navset_card_pill
             - experimental.ui.toggle_sidebar
             - experimental.ui.toggle_switch
             - experimental.ui.panel_main

e2e/experimental/value_box/test_valuebox.py

@@ -11,7 +11,7 @@
       print the number of clicks in the console twice.
       """
     ),
-    ui.navset_tab_card(
+    ui.navset_card_tab(
         ui.nav(
             "Sync",
             ui.input_action_button("btn", "Click me"),

examples/event/app.py

@@ -55,4 +55,16 @@ async def update_db(position):
 
 def begin():
     position = init_db()
-    asyncio.create_task(update_db(position))
+
+    # After initializing the database, we need to start a non-blocking task to update it
+    # every second or so. If an event loop is already running, we can use an asyncio
+    # task. (This is the case when running via `shiny run` and shinylive.) Otherwise, we
+    # need to launch a background thread and run an asyncio event loop there. (This is
+    # the case when running via shinyapps.io or Posit Connect.)
+
+    if asyncio.get_event_loop().is_running():
+        asyncio.create_task(update_db(position))
+    else:
+        from threading import Thread
+
+        Thread(target=lambda: asyncio.run(update_db(position)), daemon=True).start()

examples/model-score/scoredata.py

@@ -58,12 +58,8 @@ interface ShinyDataGridProps<TIndex> {
 
 const ShinyDataGrid: FC<ShinyDataGridProps<unknown>> = (props) => {
   const { id, data, bgcolor } = props;
-  const { columns, index, type_hints, data: rowData } = data;
+  const { columns, type_hints, data: rowData } = data;
   const { width, height, filters: withFilters } = data.options;
-  const keyToIndex: Record<string, unknown> = {};
-  index.forEach((value) => {
-    keyToIndex[value + ""] = value;
-  });
 
   const containerRef = useRef<HTMLDivElement>(null);
   const theadRef = useRef<HTMLTableSectionElement>(null);
@@ -192,7 +188,8 @@ const ShinyDataGrid: FC<ShinyDataGridProps<unknown>> = (props) => {
           rowSelection
             .keys()
             .toList()
-            .map((key) => keyToIndex[key])
+            .map((key) => parseInt(key))
+            .sort()
         );
       }
     }

js/dataframe/index.tsx

@@ -20,7 +20,7 @@ export interface DataGridOptions {
 
 export interface PandasData<TIndex> {
   columns: ReadonlyArray<string>;
-  index: ReadonlyArray<TIndex>;
+  // index: ReadonlyArray<TIndex>;
   data: unknown[][];
   type_hints?: ReadonlyArray<TypeHint>;
   options: DataGridOptions;

js/dataframe/types.ts

@@ -1,6 +1,6 @@
 [pytest]
 asyncio_mode=strict
-testpaths=tests
+testpaths=tests/pytest/
 addopts = --strict-markers --durations=6 --durations-min=5.0 --browser webkit --browser firefox --browser chromium --numprocesses auto
 markers =
     examples: Suite of tests to validate that examples do not produce errors (deselect with '-m "not examples"')

pytest.ini

@@ -63,8 +63,10 @@ def summary_data():
 
     @reactive.Calc
     def filtered_df():
+        # input.summary_data_selected_rows() is a tuple, so we must convert it to list,
+        # as that's what Pandas requires for indexing.
         selected_idx = list(req(input.summary_data_selected_rows()))
-        countries = summary_df["country"][selected_idx]
+        countries = summary_df.iloc[selected_idx]["country"]
         # Filter data for selected countries
         return df[df["country"].isin(countries)]