Here's a stab at reworking our high-level observing scripts

AaronParsons commented 2 years ago

Suggest not merging until we test/babysit the startup.

dannyjacobs commented 2 years ago

Ok thanks. Lets test soon as we can.

On Fri, May 6, 2022 at 3:57 PM Aaron Parsons @.***> wrote:

@.**** commented on this pull request.

In control_software/scripts/hera_feng_start.sh https://github.com/HERA-Team/hera_corr_f/pull/103#discussion_r867257591:

@@ -7,11 +7,10 @@ set -e

echo hera_feng_start.sh: Initializing F-Engines > $LOGFILE date >> $LOGFILE -hera_snap_feng_init.py -p -i -d &>> $LOGFILE +hera_snap_feng_init.py -p -i -d -s -e &>> $LOGFILE

Because a) this simplifies the number of scripts and steps, and b) I think it is actually a liability to rebuild the list of hosts between these two stages.

— Reply to this email directly, view it on GitHub https://github.com/HERA-Team/hera_corr_f/pull/103#discussion_r867257591, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAPNVZCVF34R7AFKW2DLMLVIWPWHANCNFSM5VJD5OFA . You are receiving this because your review was requested.Message ID: @.***>

-- Arizona State University School of Earth and Space Exploration Low Frequency Cosmology Phone: (505) 500 4521 Homepage: danielcjacobs.com

AaronParsons commented 2 years ago

Test on site succeeded.

HERA-Team / hera_corr_f

Here's a stab at reworking our high-level observing scripts #103

@.**** commented on this pull request.