review comments

Author: mrm9084

featuremanagement/_defaultfilters.py

@@ -84,10 +84,6 @@ def evaluate(self, context: Mapping[Any, Any], **kwargs: Any) -> bool:
         start_time: Optional[datetime] = parsedate_to_datetime(start) if start else None
         end_time: Optional[datetime] = parsedate_to_datetime(end) if end else None
 
-        if not start and not end:
-            logging.warning("%s: At least one of Start or End is required.", TimeWindowFilter.__name__)
-            return False
-
         if (start_time is None or start_time <= current_time) and (end_time is None or current_time < end_time):
             return True
 

featuremanagement/_time_window_filter/_models.py

@@ -109,7 +109,7 @@ def __init__(self, range_data: Dict[str, Any]):
             except TypeError as e:
                 # Python 3.9 and earlier throw TypeError if the string is not in RFC 2822 format.
                 raise ValueError(f"Invalid value for EndDate: {end_date_str}") from e
-        self.num_of_occurrences = range_data.get("NumberOfOccurrences", math.pow(2, 63) - 1)
+        self.num_of_occurrences = range_data.get("NumberOfOccurrences", 2 ** 63 - 1)
         if self.num_of_occurrences <= 0:
             raise ValueError("The number of occurrences must be greater than 0.")