Fix type annotation and docstring coverage issues

- Add return type annotation to BaseConfig.get() method
- Remove sppo_eta field from AlgoConfig class
- Add missing docstrings to profiler functions:
  - log_gpu_memory_usage in performance.py
  - mark_start_range, mark_end_range, mark_annotate in profile.py

Resolves Type Annotation and Docstring Coverage CI checks

Co-Authored-By: H <linhaibin.eric@gmail.com>

verl/base_config.py

@@ -25,7 +25,7 @@ class BaseConfig(collections.abc.Mapping):
     This allows instances of this class to be used like dictionaries.
     """
 
-    def get(self, key: str, default: Any = None):
+    def get(self, key: str, default: Any = None) -> Any:
         """Get the value associated with the given key. If the key does not exist, return the default value.
 
         Args:

verl/trainer/config/algorithm.py

@@ -61,6 +61,3 @@ class AlgoConfig(BaseConfig):
 
     # Filter groups parameters (used in DAPO and Entropy)
     filter_groups: Optional[FilterGroupsConfig] = None  # Filter groups configuration
-
-    # SPPO parameters
-    sppo_eta: Optional[float] = None  # SPPO eta parameter

verl/utils/profiler/performance.py

@@ -45,6 +45,14 @@ def _get_current_mem_info(unit: str = "GB", precision: int = 2) -> Tuple[str]:
 
 
 def log_gpu_memory_usage(head: str, logger: logging.Logger = None, level=logging.DEBUG, rank: int = 0):
+    """Log GPU memory usage information.
+
+    Args:
+        head (str): A descriptive header for the memory usage log message.
+        logger (logging.Logger, optional): Logger instance to use for logging. If None, prints to stdout.
+        level: Logging level to use. Defaults to logging.DEBUG.
+        rank (int): The rank of the process to log memory for. Defaults to 0.
+    """
     if (not dist.is_initialized()) or (rank is None) or (dist.get_rank() == rank):
         mem_allocated, mem_reserved, mem_used, mem_total = _get_current_mem_info()
         message = f"{head}, memory allocated (GB): {mem_allocated}, memory reserved (GB): {mem_reserved}, device memory used/total (GB): {mem_used}/{mem_total}"

verl/utils/profiler/profile.py

@@ -113,14 +113,39 @@ def stop_trace(self):
 
 
 def mark_start_range(message: Optional[str] = None, color: Optional[str] = None, domain: Optional[str] = None, category: Optional[str] = None) -> None:
+    """Start a profiling range marker (no-op implementation).
+
+    Args:
+        message (Optional[str]): Message to associate with the range marker.
+        color (Optional[str]): Color for the marker visualization.
+        domain (Optional[str]): Domain for the marker.
+        category (Optional[str]): Category for the marker.
+    """
     pass
 
 
 def mark_end_range(range_id: str) -> None:
+    """End a profiling range marker (no-op implementation).
+
+    Args:
+        range_id (str): Identifier of the range to end.
+    """
     pass
 
 
 def mark_annotate(message: Optional[str] = None, color: Optional[str] = None, domain: Optional[str] = None, category: Optional[str] = None) -> Callable:
+    """Decorator to annotate a function with profiling markers (no-op implementation).
+
+    Args:
+        message (Optional[str]): Message to associate with the annotation.
+        color (Optional[str]): Color for the marker visualization.
+        domain (Optional[str]): Domain for the marker.
+        category (Optional[str]): Category for the marker.
+
+    Returns:
+        Callable: Decorator function that returns the original function unchanged.
+    """
+
     def decorator(func):
         return func