From c126aae90249ac3dd3f0f43714c8d3a038e26ebe Mon Sep 17 00:00:00 2001
From: Ian Bassi <ian.bassi@outlook.com>
Date: Thu, 18 Dec 2025 09:36:10 -0300
Subject: [PATCH] PA Calib Values check (#11493)

* Add input validation for acceleration and speed values

Added a check to ensure that acceleration values are greater than speed values in the PA calibration dialog. Displays a warning message and prevents calibration if the inputs are invalid to guard against swapped or incorrect user entries.

* Add input validation for calibration parameters
---
 src/slic3r/GUI/calib_dlg.cpp | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/slic3r/GUI/calib_dlg.cpp b/src/slic3r/GUI/calib_dlg.cpp
index 8dd338f88e..24eb46b4a5 100644
--- a/src/slic3r/GUI/calib_dlg.cpp
+++ b/src/slic3r/GUI/calib_dlg.cpp
@@ -284,6 +284,19 @@ void PA_Calibration_Dlg::on_start(wxCommandEvent& event) {
     ParseStringValues(m_tiBMAccels->GetTextCtrl()->GetValue().ToStdString(), m_params.accelerations);
     ParseStringValues(m_tiBMSpeeds->GetTextCtrl()->GetValue().ToStdString(), m_params.speeds);
 
+    if (!m_params.accelerations.empty() && !m_params.speeds.empty()) {
+        // Guard against swapped inputs by ensuring acceleration magnitudes exceed speeds.
+        const double min_accel = *std::min_element(m_params.accelerations.begin(), m_params.accelerations.end());
+        const double max_speed = *std::max_element(m_params.speeds.begin(), m_params.speeds.end());
+        if (min_accel <= max_speed) {
+            MessageDialog msg_dlg(nullptr,
+                _L("Acceleration values must be greater than speed values.\nPlease verify the inputs."),
+                wxEmptyString, wxICON_WARNING | wxOK);
+            msg_dlg.ShowModal();
+            return;
+        }
+    }
+
     m_plater->calib_pa(m_params);
     EndModal(wxID_OK);