search

conda-forge / guardian-feedstock

A conda-smithy repository for guardian.
BSD 3-Clause "New" or "Revised" License
0 stars 5 forks source link

Add ligo-medm as dependency #15

Open rhyspoulton opened 2 years ago

rhyspoulton commented 2 years ago

ligo-medm is a dependency of this package since it is used by the code as a interface, so it should be added as a requirement to the package.

jrollins commented 2 years ago

@rhyspoulton In some sense MEDM is an optional dependency of guardian, since it's only needed on a "workstation" where one would interact with the guardian processes over EPICS, and is not needed on a machine where guardian daemons are run in a headless environment.

One possibility would be to break out guardmedm into a separate package.

rhyspoulton commented 2 years ago

This makes sense to me as we currently have a separate conda environment for the workstations (in the control room) in VIRGO to the IGWN Conda environments. So we would install guardmedm in the conda environment used by those workstations removing the need to install in the IGWN Conda environment.

duncanmmacleod commented 2 years ago

If guardmedm is going to be separated into a new package, I would like to see this documented/defined in the upstream package first, most likely in the Debian packaging, and then mirrored here - rather than have the conda-forge recipe wander off on its own.