Updates defaults for process map and comments out tqdm description.

Author: ppak10

src/am/simulator/tool/process_map/models/process_map.py

@@ -62,7 +62,8 @@ def run(self, num_proc: int = 1):
         if num_proc <= 1:
 
             # Iterates through points z (inner) -> y (middle) -> x (outer)
-            for data_point in tqdm(_data_points, desc="Running Process Map"):
+            # for data_point in tqdm(_data_points, desc="Running Process Map"):
+            for data_point in tqdm(_data_points):
                 # Copies build parameters to a new object to pass as overrides.
                 modified_build_parameters = deepcopy(self.build_parameters)
 
@@ -104,7 +105,7 @@ def run(self, num_proc: int = 1):
                 for future in tqdm(
                     as_completed(futures),
                     total=len(futures),
-                    desc="Running Process Map",
+                    # desc="Running Process Map", # Causes invalid json warning in claude-desktop
                 ):
                     result = (
                         future.result()
@@ -337,10 +338,10 @@ def plot(
             max_z_value_magnitude = _plot_data.axes[2][-1].magnitude
 
             # z is often layer height or hatch spacing
-            z_values = _plot_data.axes[2]
-            z_values.reverse()
+            # Use reversed() to avoid mutating the original list
+            z_values = list(reversed(_plot_data.axes[2]))
 
-            z_units = f"{_plot_data.axes[2][0].units:~}"
+            z_units = _plot_data.axes[2][0].units
             z_label = _plot_data.parameter_names[2].replace("_", " ").title()
 
             for z_idx, z_value in enumerate(z_values):
@@ -359,14 +360,17 @@ def plot(
                 layer_cmap = get_colormap_segment(position, cmap)
 
                 # Plotting
-                data_2d = data[:, :, -z_idx]
+                # Use -(z_idx + 1) since -0 equals 0, not -1
+                data_2d = data[:, :, -(z_idx + 1)]
                 # Mask all the False values so only True (1) areas are drawn
                 data_2d_masked = np.ma.masked_where(
                     ~np.array(data_2d, dtype=bool), data_2d
                 )
                 ax.imshow(
                     data_2d_masked,
                     cmap=layer_cmap,
+                    vmin=0,
+                    vmax=2,
                     aspect="auto",
                     origin="lower",
                     extent=extent,

src/am/simulator/tool/process_map/models/process_map_parameter_range.py

@@ -8,15 +8,15 @@
     "beam_power": {
         "start": (100, "watts"),
         "stop": (1000, "watts"),
-        "step": (100, "watts"),
+        "step": (25, "watts"),
     },
     "scan_velocity": {
         "start": (100, "millimeter / second"),
         "stop": (2000, "millimeter / second"),
-        "step": (100, "millimeter / second"),
+        "step": (25, "millimeter / second"),
     },
     "layer_height": {
-        "start": (25, "microns"),
+        "start": (50, "microns"),
         "stop": (100, "microns"),
         "step": (25, "microns"),
     },

src/am/simulator/tool/process_map/utils/parameter_ranges.py

@@ -115,22 +115,23 @@ def inputs_to_parameter_ranges(*input_tuples: ProcessMapParameterRangeInputTuple
     """
     parameter_ranges = []
 
+    # Should be just "beam_power", "scan_velocity", and "layer_height"
+    keys = list(DEFAULTS.keys())[:3]
+
     # If no parameters provided, use defaults in order
     if all(all(v is None for v in param) for param in input_tuples):
 
-        # Should be just "beam_power", "scan_velocity", and "layer_height"
-        keys = list(DEFAULTS.keys())[:3]
-
         for key in keys:
             parameter_range = ProcessMapParameterRange(name=key)
             parameter_ranges.append(parameter_range)
 
         return parameter_ranges
 
-    for shorthand, name, range_values, units in input_tuples:
+    for index, (shorthand, name, range_values, units) in enumerate(input_tuples):
         parameter_range = parse_options(shorthand, name, range_values, units)
 
-        if parameter_range is not None:
-            parameter_ranges.append(parameter_range)
+        if parameter_range is None:
+            parameter_range = ProcessMapParameterRange(name=keys[index])
+        parameter_ranges.append(parameter_range)
 
     return parameter_ranges