Eva Lott

Results 16 comments of Eva Lott

@tacaswell We edited the test to try and match @coretl's [idea from yesterday](https://github.com/bluesky/bluesky/pull/1634#issuecomment-1931673543), however I ran into issues in `run_engine.py` that I found hard to track down. Specifically this [code](https://github.com/tacaswell/bluesky/blob/26ef337d8cb8ff2a9fc01caa4d545c5b2aedd050/bluesky/run_engine.py#L1460-L1463)...

I'll start working on this now.

It was during [this job]( https://github.com/bluesky/bluesky/actions/runs/6773999783/job/18410110459#step:9:1771), seems to be quite a rare failure.

Hmmm it does seem like this problem hasn't appeared in any recent runs. Happy to call this stale.

> fixed the typing for Movable, but not for Readable Will fix now (after lunch)

This is a problem on the ophyd side. The following would need to be changed to the proper type hint. https://github.com/bluesky/ophyd/blob/1e5f03752f6a6b36ae5738ebbdf4f7645129b20d/ophyd/device.py#L563-L588 There could maybe be a PR to correct all...

Closing since https://github.com/bluesky/bluesky/issues/1809#issuecomment-2630708730 should be handled in ophyd: https://github.com/bluesky/ophyd/issues/1232

This will require dropping python 3.8 since numpydoc 1.8.0 will require python vesion >=3.9

> DatumPage.datum_kwargs type is Dict[str, List], while it should instead be Dict[str, List[Any]]. I don't exactly know if I'm missing an option in the generation script or if it is...

> Pytest-asyncio wants to transition from using low-level loop calls to using asyncio.Runner. I don't know how this feature can be integrated from the top of my head. I'm sure...