sot / obc_rate_noise_plots

0 stars 0 forks source link

Package #5

Closed javierggt closed 4 months ago

javierggt commented 5 months ago

This PR adds all the usual stuff to turn this into a package. After this PR:

The conda recipe is in skare3 (sot/skare3/pull/1320).

Testing

jeanconn commented 5 months ago

Regarding "I do not like installing the task schedule in the module area. Suggestions are welcome." I think we've just moved to installing them in the module/package area. The one thing I don't like is that we have extra task schedules floating around in share and data from other tools that are no longer being used (because the real ones are in the python package/module area).

javierggt commented 5 months ago

I think we've just moved to installing them in the module/package area.

Have we? When? Where? Then the path of the task schedule changes with python versions and such.

javierggt commented 5 months ago

The one thing I don't like is that we have extra task schedules floating around in share and data from other tools that are no longer being used

If things are properly installed in a package, when you remove a package the task schedule goes away, so there will be no stray task schedules.

Agasc is one package that has it in the share area.

jeanconn commented 5 months ago

"If things are properly installed in a package, when you remove a package the task schedule goes away, so there will be no stray task schedules." Yes -- there are just still old ones from ska2 and ones from packages that weren't appropriately packages.

jeanconn commented 5 months ago

Regarding "Have we? When? Where? Then the path of the task schedule changes with python versions and such." Yes. Tom added this "-package" option to task schedule that uses the module-determined path to run the task

https://github.com/sot/task_schedule/blob/128ffa073ce44bc6cd84bf5db144cb2218de0a60/task_schedule3.pl#L85

so all the jobs on the Ska cron jobs using the -package flag are finding the task schedule in the python package directory.

javierggt commented 5 months ago

It is unrelated to this PR, but agasc has two task schedules (https://github.com/sot/agasc/tree/master/task_schedules) so that -package option would not work.

javierggt commented 5 months ago

I'm moving the task schedule into the package then. That would not have been my choice, but since everything already uses that convention...

jeanconn commented 5 months ago

"It is unrelated to this PR, but agasc has two task schedules (https://github.com/sot/agasc/tree/master/task_schedules) so that -package option would not work."

You can either put the task schedule in a submodule or combine the -config option with the -package option to get around that as needed (like I just did for the acdc warm pixel plot).