search

pcdshub / lightpath

LCLS Lightpath Module
https://pcdshub.github.io/lightpath
Other
4 stars 9 forks source link

`python -m lightpath` doesn't start lightpath #146

Closed klauer closed 1 year ago

klauer commented 1 year ago

https://github.com/pcdshub/lightpath/blob/09210e08ad35b233bf4252822ec3dd765ed98a78/lightpath/__main__.py#L30

The intention was: python -m lightpath should call lightpath.__main__.main() but that currently doesn't happen

I think the following formation would be better and make the "main" functionality more accessible to non-CLI entrypoints

  1. Move most functions out of lightpath.__main__ into lightpath.main
  2. In lightpath.__main__ just do from . import main; main()
tangkong commented 1 year ago

This duplicates #135 . I just haven't gotten to this yet 😬

klauer commented 1 year ago

My mental issue stack overfloweth