Please merge from main and revert #6910

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

       15 files  ±0         15 suites  ±0   6h 28m 57s ⏱️ + 1m 22s
  3 005 tests ±0    2 921 ✔️ +1    83 💤 +1  1 ❌  - 2 
22 239 runs  ±0  21 268 ✔️  - 2  970 💤 +4  1 ❌  - 2 

For more details on these failures, see this check.

Results for commit 715804c. ± Comparison against base commit 58a5a3c.

♻️ This comment has been updated with latest results.

Oh interesting I didn't know this was changing. I expect we use _ipython_display_ in other places in Dask outside of distributed.

@ian-r-rose could you advise on the right way to handle this?

The tricky thing is supporting both ipywidgets~=7 and ipywidgets~=8 here. Generally speaking, we should prefer _repr_mimebundle_ to _ipython_display_, which is why ipywidgets made the change. The difference is that _repr_mimebundle_ is supposed to just return a dictionary keyed by mimetype, and the frontend can decide which repr it wants to/is able to display. _ipython_display_, on the other hand, is supposed to display as a side effect.

Since we probably don't want to go ipywidgets==8 exclusively, I don't think we can fully commit to using _repr_mimebundle_ everywhere. I've pushed a commit with what I think is the best way forward: we stick with _ipython_display_ for now but use _repr_mimebundle_ when available.

Test failure is unrelated.

FAILED distributed/tests/test_client.py::test_reconnect - asyncio.exceptions....

Merging.