refactor(config): consolidate protection into routing and normalize CLI defaults

Major configuration architecture improvements:
- Refactor protection mechanism from separate `protection_mode` setting to `route_method=1plus1_protection`
- Normalize all CLI parameter defaults to None for better config file precedence
- Move k_paths from general_settings to routing_settings section
- Update all example configs, templates, and tests to reflect new structure
- Add request tracking infrastructure in SDNController for failure handling
- Enhance backup path management in routing properties

This change simplifies the configuration model by treating 1+1 protection
as a routing method rather than a separate protection layer, making the
system more intuitive and reducing configuration complexity.

Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: ryanmccann1024

README.md

@@ -151,7 +151,7 @@ python -m fusion.cli.run_sim \
   --failure_type geo \
   --geo_center_node 5 \
   --geo_hop_radius 2 \
-  --protection_mode 1plus1
+  --route_method 1plus1_protection
 ```
 
 ### Example Configurations
@@ -174,8 +174,10 @@ failure_type = geo
 geo_center_node = 5
 geo_hop_radius = 2
 
+[routing_settings]
+route_method = 1plus1_protection
+
 [protection_settings]
-protection_mode = 1plus1
 protection_switchover_ms = 50.0
 ```
 

fusion/cli/parameters/analysis.py

@@ -29,19 +29,19 @@ def add_statistics_args(parser: argparse.ArgumentParser) -> None:
     stats_group.add_argument(
         "--snapshot_step",
         type=int,
-        default=100,
+        default=None,
         help="Number of requests between snapshots",
     )
     stats_group.add_argument(
         "--save_step",
         type=int,
-        default=1000,
+        default=None,
         help="Number of requests between saving results",
     )
     stats_group.add_argument(
         "--print_step",
         type=int,
-        default=1000,
+        default=None,
         help="Number of requests between progress updates",
     )
     stats_group.add_argument(
@@ -71,7 +71,10 @@ def add_plotting_args(parser: argparse.ArgumentParser) -> None:
     )
     add_plot_format_args(plot_group)
     plot_group.add_argument(
-        "--plot_dpi", type=int, default=300, help="Resolution (DPI) for generated plots"
+        "--plot_dpi",
+        type=int,
+        default=None,
+        help="Resolution (DPI) for generated plots",
     )
     plot_group.add_argument(
         "--show_plots",
@@ -97,7 +100,7 @@ def add_export_args(parser: argparse.ArgumentParser) -> None:
         "--file_type",
         type=str,
         choices=["json", "csv", "excel", "tsv"],
-        default="json",
+        default=None,
         help="Default file format for data export",
     )
 

fusion/cli/parameters/network.py

@@ -25,7 +25,10 @@ def add_topology_args(parser: argparse.ArgumentParser) -> None:
         help="Network topology name (e.g., 'NSFNet', 'USbackbone60')",
     )
     network_group.add_argument(
-        "--cores_per_link", type=int, default=1, help="Number of cores per fiber link"
+        "--cores_per_link",
+        type=int,
+        default=None,
+        help="Number of cores per fiber link",
     )
     network_group.add_argument(
         "--bw_per_slot",
@@ -96,31 +99,31 @@ def add_spectrum_bands_args(parser: argparse.ArgumentParser) -> None:
     spectrum_group.add_argument(
         "--c_band",
         type=int,
-        default=96,
+        default=None,
         help="Number of spectral slots in C-band (1530-1565nm)",
     )
     spectrum_group.add_argument(
         "--l_band",
         type=int,
-        default=0,
+        default=None,
         help="Number of spectral slots in L-band (1565-1625nm)",
     )
     spectrum_group.add_argument(
         "--s_band",
         type=int,
-        default=0,
+        default=None,
         help="Number of spectral slots in S-band (1460-1530nm)",
     )
     spectrum_group.add_argument(
         "--e_band",
         type=int,
-        default=0,
+        default=None,
         help="Number of spectral slots in E-band (1360-1460nm)",
     )
     spectrum_group.add_argument(
         "--o_band",
         type=int,
-        default=0,
+        default=None,
         help="Number of spectral slots in O-band (1260-1360nm)",
     )
 

fusion/cli/parameters/routing.py

@@ -27,7 +27,7 @@ def add_routing_args(parser: argparse.ArgumentParser) -> None:
     routing_group.add_argument(
         "--k_paths",
         type=int,
-        default=3,
+        default=None,
         help="Number of candidate paths for k-shortest path routing",
     )
 
@@ -51,7 +51,7 @@ def add_spectrum_args(parser: argparse.ArgumentParser) -> None:
     spectrum_group.add_argument(
         "--guard_slots",
         type=int,
-        default=1,
+        default=None,
         help="Number of guard slots between allocations",
     )
     spectrum_group.add_argument(

fusion/cli/parameters/shared.py

@@ -103,6 +103,6 @@ def add_plot_format_args(
     parser.add_argument(
         "--plot_format",
         type=str,
-        default="png",
+        default=None,
         help="Output format for generated plots",
     )

fusion/cli/parameters/snr.py

@@ -28,7 +28,7 @@ def add_snr_args(parser: argparse.ArgumentParser) -> None:
         help="SNR calculation method",
     )
     parser.add_argument(
-        "--input_power", type=float, default=1e-3, help="Input power in Watts"
+        "--input_power", type=float, default=None, help="Input power in Watts"
     )
     parser.add_argument(
         "--egn_model",

fusion/cli/parameters/survivability.py

@@ -79,13 +79,6 @@ def add_protection_args(parser: argparse.ArgumentParser) -> None:
     :type parser: argparse.ArgumentParser
     """
     protection_group = parser.add_argument_group("Protection Settings")
-    protection_group.add_argument(
-        "--protection_mode",
-        type=str,
-        choices=["none", "1plus1"],
-        default=None,
-        help="Protection mechanism to use (none, 1plus1)",
-    )
     protection_group.add_argument(
         "--protection_switchover_ms",
         type=float,

fusion/cli/parameters/training.py

@@ -78,40 +78,40 @@ def add_reinforcement_learning_args(parser: argparse.ArgumentParser) -> None:
     rl_group.add_argument(
         "--learn_rate",
         type=float,
-        default=0.001,
+        default=None,
         help="Learning rate for RL algorithms",
     )
     rl_group.add_argument(
-        "--gamma", type=float, default=0.99, help="Discount factor for future rewards"
+        "--gamma", type=float, default=None, help="Discount factor for future rewards"
     )
     rl_group.add_argument(
         "--epsilon_start",
         type=float,
-        default=1.0,
+        default=None,
         help="Initial epsilon value for epsilon-greedy exploration",
     )
     rl_group.add_argument(
         "--epsilon_end",
         type=float,
-        default=0.01,
+        default=None,
         help="Final epsilon value for epsilon-greedy exploration",
     )
     rl_group.add_argument(
         "--epsilon_update",
         type=str,
         choices=["linear", "exponential", "step", "linear_decay", "exp_decay"],
-        default="linear",
+        default=None,
         help="Epsilon decay strategy",
     )
 
     # Reward configuration
     rl_group.add_argument(
-        "--reward", type=float, default=1.0, help="Reward value for successful actions"
+        "--reward", type=float, default=None, help="Reward value for successful actions"
     )
     rl_group.add_argument(
         "--penalty",
         type=float,
-        default=-1.0,
+        default=None,
         help="Penalty value for unsuccessful actions",
     )
     rl_group.add_argument(
@@ -147,18 +147,18 @@ def add_feature_extraction_args(parser: argparse.ArgumentParser) -> None:
         help="Graph Neural Network architecture type",
     )
     feature_group.add_argument(
-        "--layers", type=int, default=3, help="Number of layers in neural network"
+        "--layers", type=int, default=None, help="Number of layers in neural network"
     )
     feature_group.add_argument(
         "--emb_dim",
         type=int,
-        default=64,
+        default=None,
         help="Embedding dimension for neural networks",
     )
     feature_group.add_argument(
         "--heads",
         type=int,
-        default=8,
+        default=None,
         help="Number of attention heads (for attention-based models)",
     )
     feature_group.add_argument(
@@ -203,7 +203,7 @@ def add_machine_learning_args(parser: argparse.ArgumentParser) -> None:
     ml_group.add_argument(
         "--test_size",
         type=float,
-        default=0.2,
+        default=None,
         help="Fraction of data to use for testing (0.0-1.0)",
     )
     ml_group.add_argument(
@@ -238,20 +238,20 @@ def add_optimization_args(parser: argparse.ArgumentParser) -> None:
     opt_group.add_argument(
         "--optuna_trials",
         type=int,
-        default=100,
+        default=None,
         help="Number of optimization trials for Optuna",
     )
     opt_group.add_argument(
         "--n_trials",
         type=int,
-        default=10,
+        default=None,
         help="Number of trials for grid search or random search",
     )
     opt_group.add_argument(
         "--device",
         type=str,
         choices=["cpu", "cuda", "mps", "auto"],
-        default="auto",
+        default=None,
         help="Computing device for training (cpu/gpu)",
     )
 

fusion/configs/cli_to_config.py

@@ -43,7 +43,7 @@ def __init__(self) -> None:
             "num_requests": ("general_settings", "num_requests"),
             "request_distribution": ("general_settings", "request_distribution"),
             "allocation_method": ("general_settings", "allocation_method"),
-            "k_paths": ("general_settings", "k_paths"),
+            "k_paths": ("routing_settings", "k_paths"),
             "route_method": ("general_settings", "route_method"),
             "save_snapshots": ("general_settings", "save_snapshots"),
             "snapshot_step": ("general_settings", "snapshot_step"),

fusion/configs/examples/README.md

@@ -100,9 +100,10 @@ python -m fusion.cli.run_sim run_sim --config_path fusion/configs/examples/datas
 
 ### Protection Settings
 
+**Note:** Enable 1+1 protection by setting `route_method=1plus1_protection` in routing settings.
+
 | Parameter | Values | Description |
 |-----------|--------|-------------|
-| `protection_mode` | `none`, `1plus1` | Protection mechanism |
 | `protection_switchover_ms` | Float | Switchover latency (ms) |
 | `restoration_latency_ms` | Float | Re-routing latency (ms) |
 | `revert_to_primary` | Boolean | Revert after repair |