DiamondLightSource / blueapi

Apache License 2.0
3 stars 5 forks source link

Restructure rest and service modules #567

Open joeshannon opened 1 month ago

joeshannon commented 1 month ago

As suggested in https://github.com/DiamondLightSource/blueapi/pull/538#discussion_r1674033258

stan-dot commented 1 week ago

@callumforrester , @DiamondJoseph what are your thoughts?

callumforrester commented 1 week ago

Agree this should be done as a low priority

stan-dot commented 1 week ago

I must say it feels a bit underspecified at the moment @callumforrester . the difference between what we have now is very minimal - once the handler was removed

callumforrester commented 1 week ago

That's understandable, to add justification, our current structure is:

Which is confusing, main and interface are the top-level runtimes for each process, but that's not obvious. The proposed structure makes it more obvious.