by default, no fieldrot_corr if fieldrot is provided

desihub · Jan 29, 2025 · b870d9c · b870d9c
1 parent a4e6777
commit b870d9c
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/py/fiberassign/scripts/assign.py b/py/fiberassign/scripts/assign.py
@@ -112,7 +112,8 @@ def parse_assign(optlist=None):
     parser.add_argument("--fieldrot_corr", type=bool, required=False,
                         default=None,
                         help="apply correction to computed fieldrot "
-                        "(default: False if rundate<rundate_cutoff, True else")
+                        "(default: False if args.fieldrot is provided or "
+                        "if args.rundate<rundate_cutoff, True else")
 
     parser.add_argument("--dir", type=str, required=False, default=None,
                         help="Output directory.")
@@ -280,10 +281,13 @@ def parse_assign(optlist=None):
         args.obsdate = '{}-{}-{}'.format(year, mm, dd)
 
     # Apply the field rotation correction?
-    # - True if rundate!=None and rundate>=rundate_cutoff
+    # - True if fieldrot=None, rundate!=None and rundate>=rundate_cutoff
     # - False else
     if args.fieldrot_corr is None:
-        _, args.fieldrot_corr = get_obsdate(rundate=args.rundate)
+        if args.fieldrot is None:
+            _, args.fieldrot_corr = get_obsdate(rundate=args.rundate)
+        else:
+            args.fieldrot_corr = False
 
     # Set output directory
     if args.dir is None: