Make FilenameProvider optional on PathProvider impl

bluesky / ophyd-async

Hardware abstraction for bluesky written using asyncio

https://blueskyproject.io/ophyd-async

BSD 3-Clause "New" or "Revised" License

12 stars 26 forks source link

Make FilenameProvider optional on PathProvider impl #547

Closed DiamondJoseph closed 3 months ago

DiamondJoseph commented 3 months ago

Closes #546

Required for Diamond expectation of file names.

DiamondJoseph commented 3 months ago

After discussion IRL, we've opted to do both: remove the requirement for the PathProvider to have a FilenameProvider, but also adjust the signature of the FilenameProvider to match the PathProvider. This can propagate from the PathProvider call down.