diff --git a/CHANGELOG.md b/CHANGELOG.md index 8486031..7f00524 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) ## [Unreleased] ### Fixed +- Fixed bug where --subset in combination with the subscriber caused errors ### Added ## [1.15.0] diff --git a/subscriber/podaac_access.py b/subscriber/podaac_access.py index e69da75..48be4b3 100644 --- a/subscriber/podaac_access.py +++ b/subscriber/podaac_access.py @@ -221,13 +221,6 @@ def validate(args): 'Please specify exactly one flag ' 'from -dc, -dy, -dydoy, or -dymd') - if args.subset and args.search_cycles: - # Cycle+Subset are not supported, because Harmony does not - # currently accept Cycle. - raise ValueError( - 'Error: Incompatible Parameters. You\'ve provided both cycles and subset, which is ' - 'not allowed. Please provide either cycles or subset separately, but not both.') - if args.subset and args.bbox: bounds = list(map(float, args.bbox.split(','))) if bounds[0] > bounds[2]: diff --git a/subscriber/podaac_data_downloader.py b/subscriber/podaac_data_downloader.py index 08408ae..aff5956 100755 --- a/subscriber/podaac_data_downloader.py +++ b/subscriber/podaac_data_downloader.py @@ -44,6 +44,13 @@ def validate(args): if None in [args.endDate, args.startDate] and args.search_cycles is None and args.granulename is None: raise ValueError( "Error parsing command line arguments: Both --start-date and --end-date must be specified") # noqa E50 + if args.subset and args.search_cycles: + # Cycle+Subset are not supported, because Harmony does not + # currently accept Cycle. + raise ValueError( + 'Error: Incompatible Parameters. You\'ve provided both cycles and subset, which is ' + 'not allowed. Please provide either cycles or subset separately, but not both.' + ) def create_parser():