Fix Disp variable mapping to use tel_list_matrix before distance sorting

[Copilot]

Disp variables (DispTelList_T, Disp_T, DispXoff_T, etc.) were directly copied from column positions without mapping through tel_list_matrix, causing misalignment when telescope IDs are non-contiguous (e.g., [0, 2, 5]).

Changes

• Remove special-case handling for Disp variables in _flatten_variable_columns
• Apply same tel_list_matrix mapping to Disp variables as other per-telescope variables
• Ensures correct alignment: column position → telescope ID → distance sort

Example

For telescopes [0, 2, 5] with Disp values [10, 20, 30]:

Before: Values incorrectly assigned to telescope IDs [0, 1, 2]
After: Values correctly assigned to telescope IDs [0, 2, 5]

# Previously: Disp variables had special handling
if var.startswith("Disp"):
    base[:, :n_cols] = data[:, :n_cols]  # Direct copy, wrong

# Now: Unified mapping for all variables
full_matrix[row_indices[valid_mask], tel_ids[valid_mask]] = data[
    row_indices[valid_mask], col_indices[valid_mask]
]  # Correct mapping via tel_list_matrix

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.