enrico icon indicating copy to clipboard operation
enrico copied to clipboard

`enrico/RunGtlike.py` should not be a command line tool

Open cdeil opened this issue 11 years ago • 3 comments

It's cleaner if library code and command line scripts are separated.

Therefore I suggest to remove the if __name__ == '__main__': part from enrico/RunGTlike.py.

If I see correctly the only place where RunGTlike.py is used as a command line tool is in enrico.energybin.RunEbin?

In that case we could simply call enrico.RunGTlike.run() there instead of calling RunGTlike.py as a command line tool?

@davidsanchez If you agree I think I could quickly make the change.

cdeil avatar Sep 21 '13 12:09 cdeil

I just saw: enrico/appertureLC.py currently is also a command line tool that is called from bin/enrico_applc. Same cleanup could be done here.

cdeil avatar Sep 21 '13 12:09 cdeil

I will have a look

davidsanchez avatar Sep 23 '13 07:09 davidsanchez

I don't know how to do that. RunGTlike is call when submitting a job and need to be call in a command line

davidsanchez avatar Sep 23 '13 10:09 davidsanchez