From bf53fd438f20c791d3ebc55ff2ce3a5d294dcb0d Mon Sep 17 00:00:00 2001 From: "Fabian P. Schmidt" Date: Tue, 5 Nov 2019 12:54:28 +0100 Subject: [PATCH] schedule_single_station.py: Optimize min_riseset calculation If "--horizon" is set, the calculated min_riseset was never used and an the implicit default of observer.horizon=0.0 was used. Instead of this implicit default, it is set explicitely now. --- schedule_single_station.py | 26 ++++++++++++++------------ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/schedule_single_station.py b/schedule_single_station.py index 8167ed6..9ca12fb 100755 --- a/schedule_single_station.py +++ b/schedule_single_station.py @@ -181,19 +181,21 @@ def main(): min_culmination = args.min_culmination # Set minimum rise/set elevation - if args.min_riseset is None: - min_riseset = ground_station['min_horizon'] - else: - if args.min_riseset < 0.0: - min_riseset = 0.0 - elif args.min_riseset > 90.0: - min_riseset = 90.0 - else: - min_riseset = args.min_riseset - - # Use minimum altitude for computing rise and set times (horizon to horizon otherwise) if not args.horizon: - observer.horizon = str(min_riseset) + # Use minimum altitude for computing rise and set times (horizon to horizon otherwise) + if args.min_riseset is None: + min_riseset = ground_station['min_horizon'] + else: + if args.min_riseset < 0.0: + min_riseset = 0.0 + elif args.min_riseset > 90.0: + min_riseset = 90.0 + else: + min_riseset = args.min_riseset + else: + min_riseset = 0.0 + + observer.horizon = str(min_riseset) # Minimum duration of a pass min_pass_duration = settings.MIN_PASS_DURATION