POCS icon indicating copy to clipboard operation
POCS copied to clipboard

Add support for local fields file

Open jamessynge opened this issue 7 years ago • 3 comments

Just as we support reading from pocs_local.yaml, it would be nice if we could read from a local variant of the fields file. Alternately, if we could specify additional field file(s) in pocs_local.yaml, to augment the default fields file in pocs.yaml.

Why? To avoid having users edit targets/simple.yaml, which may get overridden when the user upgrades to a new version of POCS.

jamessynge avatar Feb 14 '18 12:02 jamessynge

I thought this was already asked and answered but might have been in a private email. The fields file is pulled from the config (here) so the file name can easily be overridden there. I think this is preferable to supporting the "local" mechanism.

wtgee avatar Feb 14 '18 21:02 wtgee

The name can not be overridden except at startup. The re-read-fields-file feature is going to read the same file name. By allowing a local variant, we then allow the user to add and remove a file (e.g. for a single night) without having to stop and restart pocs_shell twice.

And IIUC, the config file is not re-read, so the file name can't be changed while pocs_shell is running.

jamessynge avatar Feb 19 '18 15:02 jamessynge

The idea was to always integrate the scheduler into PAWS so that targets could be added or removed via a better interface, which would interact directly with the pocs.observatory.scheduler.observations list. A local file name could work in some scenarios but we might be better off having a conversation about the overall aims of the scheduler which to date has been very dumb.

wtgee avatar Feb 19 '18 20:02 wtgee