diff --git a/Components/Application/Src/ApplicationCommands.cpp b/Components/Application/Src/ApplicationCommands.cpp index d98b59a..4a76df2 100644 --- a/Components/Application/Src/ApplicationCommands.cpp +++ b/Components/Application/Src/ApplicationCommands.cpp @@ -1097,9 +1097,16 @@ namespace jdi { timeShift = style->offsetTime; } - const bool ok = m_dataSet.ClipSignalRange(groupIndex, signalIndex, low - timeShift, - high - timeShift); - if (!ok) + // ⚠️⚠️ **必须先让该信号独占时间轴**再裁剪：同组信号常常共享同一条时间轴， + // 直接裁剪会把其它信号的时间基准一起改短 ⇒ 它们的采样与时间错位 + //（用户 2026-11-03 报告："多个信号同时截取时截取出来的信号并不对"）。 + std::size_t ownAxis = 0; + if (!m_dataSet.ForkAxisForSignal(groupIndex, signalIndex, ownAxis)) + { + m_plot.ShowPersistentHint(L"选区截取失败：无法为该信号创建独立时间轴。"); + return false; + } + if (!m_dataSet.ClipSignalRange(groupIndex, signalIndex, low - timeShift, high - timeShift)) { m_plot.ShowPersistentHint(L"选区截取失败：区间内的有效采样点不足两个。"); return false; diff --git a/Components/PlotView/Src/PlotViewInteraction.cpp b/Components/PlotView/Src/PlotViewInteraction.cpp index b96cafe..9e3688f 100644 --- a/Components/PlotView/Src/PlotViewInteraction.cpp +++ b/Components/PlotView/Src/PlotViewInteraction.cpp @@ -3587,6 +3587,8 @@ namespace jdi ? wheelLane.edgeCountScroll : (hoveredKind == EdgePanelKind::RESAMPLE) ? wheelLane.resampleScroll + : (hoveredKind == EdgePanelKind::CLIP) + ? wheelLane.clipScroll : wheelLane.meanFilterScroll; const float next = current - static_cast(notches * 40.0 * m_scale); const float target = std::max(0.0f, std::min(next, maxScroll)); diff --git a/Components/PlotView/Src/PlotViewPaint.cpp b/Components/PlotView/Src/PlotViewPaint.cpp index 58a3bf2..4cf15d4 100644 --- a/Components/PlotView/Src/PlotViewPaint.cpp +++ b/Components/PlotView/Src/PlotViewPaint.cpp @@ -4209,7 +4209,9 @@ namespace jdi if (kind == EdgePanelKind::CLIP) { mix(Tab().CursorMode() == PlotCursorMode::DOUBLE ? 1ull : 0ull); - mix(lane.clipScroll > 0.0f ? 1ull : 0ull); + // ⚠️ 滚动偏移必须**按真实值**入键（不能退化成布尔）：否则滚到 1px 之后 + // 键不再变化 ⇒ 后续滚动不会重绘 ⇒ 表现为"滚不动 / 滚不到底"（用户 2026-11-03）。 + mixFloat(lane.clipScroll); } // ⚠️⚠️ **边沿统计的数字本身就是面板内容，数字变化必须能让面板重绘**： diff --git a/Components/PlotView/Src/PlotViewPanels.cpp b/Components/PlotView/Src/PlotViewPanels.cpp index 199f708..bee96e3 100644 --- a/Components/PlotView/Src/PlotViewPanels.cpp +++ b/Components/PlotView/Src/PlotViewPanels.cpp @@ -393,12 +393,13 @@ namespace jdi // 面板底部（内容区之下）占的纵向空间：有按钮时 = pad + 按钮 + pad，无按钮时只有 pad。 // ⚠️ 「选区截取」底部是**两个**按钮行（截取为新信号 / 原始信号截取）， // 因此 footer 要按两行算（用户 2026-11-03 要求）。 + // ⚠️ **未启用双光标**时两个按钮都不显示 ⇒ footer 归零（否则按钮位置会留一大段空白）。 + const bool clipReady = clipPanel && (Tab().CursorMode() == PlotCursorMode::DOUBLE); const float footerHeight = - clipPanel ? (buttonHeight * 2.0f + 6.0f * m_scale + pad) + clipPanel ? (clipReady ? (buttonHeight * 2.0f + 6.0f * m_scale + pad) : 0.0f) : (hasButton ? (buttonHeight + pad) : 0.0f); const float bottomPad = pad; - const float weakPeakSliderRow = kWeakPeakSliderRowHeight * m_scale; float meanContentHeight = 0.0f; float height = 0.0f; float tableHeaderHeight = 0.0f; @@ -426,8 +427,19 @@ namespace jdi // 「均值滤波」与「弱峰调整」= 两行滑块。 // 内容超高时出现滚动条；底部「构建为新信号」按钮固定不滚动。 const float groupRow = kEdgePanelMeanRowHeight * m_scale; - const float sliderRow = weakPeakSliderRow; + const float sliderRow = kWeakPeakSliderRowHeight * m_scale; + // ⚠️ 「选区截取」在**未启用双光标**时没有任何可操作内容（只有一行提示文字）， + // 因此不按信号数累加行高、也不套用「至少 3 行」的下限 + // （用户 2026-11-03 报告：未启用双光标时仍显示信号条目、且能触发滚动）。 + const bool clipReady = clipPanel && (Tab().CursorMode() == PlotCursorMode::DOUBLE); float content = 0.0f; + if (clipPanel && !clipReady) + { + // 提示行固定占一行高度（绘制时在内容区居中显示）。 + content = kEdgePanelMeanRowHeight * m_scale; + } + else + { for (const SignalRef& ref : lane.signals) { const ResolvedSignal resolved = ResolveSignal(ref); @@ -478,12 +490,15 @@ namespace jdi } } } + } meanContentHeight = content; const float budget = m_plotRect.Height - (titleHeight + pad + bottomPad + footerHeight); + // ⚠️ 高度必须**精确等于内容高度**（不超过可用预算），不要再套「至少 3 行」的下限： + // 那个下限会让只有 1 个信号的面板留出 2 行空白（用户 2026-11-03 报告 + // “信号条目的滚动区域与下方按钮之间有一大段空白”）。 const float maxContent = - std::max(3.0f * groupRow, - std::min(kFreeMarkerContentMaxHeight * m_scale, std::max(3.0f * groupRow, budget))); + std::min(kFreeMarkerContentMaxHeight * m_scale, std::max(groupRow, budget)); const float view = std::min(content, maxContent); height = titleHeight + pad + view + bottomPad + footerHeight; } @@ -811,11 +826,15 @@ namespace jdi const float groupRow = kEdgePanelMeanRowHeight * m_scale; const float sliderRow = kWeakPeakSliderRowHeight * m_scale; const float scrollW = kFreeMarkerScrollWidth * m_scale; + const bool clipReadyHere = + clipPanel && (Tab().CursorMode() == PlotCursorMode::DOUBLE); const float footerHeight = - clipPanel ? (buttonRowHeight * 2.0f + 6.0f * m_scale + pad) + clipPanel ? (clipReadyHere ? (buttonRowHeight * 2.0f + 6.0f * m_scale + pad) : 0.0f) : (hasButton ? (buttonRowHeight + pad) : 0.0f); - const float viewHeight = - std::max(0.0f, height - titleHeight - pad - pad - footerHeight); + // ⚠️ 内容区高度必须与高度算式（`height = title + pad + view + bottomPad + footer`） + // 严格同源：只减 **一个** pad（标题下方那次），不要再多减一个，否则内容区偏矮、 + // 滚动条提前出现且底部留出空白（用户 2026-11-03 报告）。 + const float viewHeight = std::max(0.0f, height - titleHeight - pad - footerHeight); const bool scrollable = meanContentHeight > viewHeight + 0.5f; const float maxScroll = std::max(0.0f, meanContentHeight - viewHeight); const float storedScroll = meanFilter ? lane.meanFilterScroll @@ -871,8 +890,15 @@ namespace jdi (resetLeft - resetGap - valueColumn - 6.0f * m_scale) - trackLeft); float rowY = layout.content.Y - scroll; + // ⚠️ 「选区截取」在**未启用双光标**时不生成任何信号行：面板只显示一句居中提示 + // （用户 2026-11-03 明确要求：不应显示信号名条目、更不应能滚动）。 + const bool clipRowsVisible = !clipPanel || clipReadyHere; for (const SignalRef& ref : lane.signals) { + if (!clipRowsVisible) + { + break; + } const ResolvedSignal resolved = ResolveSignal(ref); if (resolved.data == nullptr || !resolved.visible) { diff --git a/Documents/agent_mem.md b/Documents/agent_mem.md index eebe739..6f07554 100644 --- a/Documents/agent_mem.md +++ b/Documents/agent_mem.md @@ -9952,3 +9952,66 @@ MEAN_SCROLL / 滚轮 / 悬停 / `SetLanePanelCollapsed` / `LanePanelCollapsed` / 本轮因此连续 3 次编译失败。**改这类表达式时先用 Python 数列数括号**，或改成查表。 * `make clean` 在用户正开着 `Target.exe` 时会失败（文件被占用）， 改用"删 `Build/Release/*.o` + `make`"做全量重编译验证即可。 + +--- + +## 2026-11-03 第二十八轮（本机 dev）：选区截取面板五处缺陷修复 + +> 用户实测报告的五个问题：①未启用双光标时仍显示信号条目、过多信号还能滚动； +> ②条目滚动区与底部按钮之间有一大段空白（未启用双光标时也有）； +> ③多个信号同时「原始信号截取」结果不对；④有滚动条时滚轮滚不到底。 + +### 1. 未启用双光标时不应显示信号条目 + +**根因**：`EdgePanelLayoutFor` 与绘制侧都只判断 `clipPanel`，没有判断「双光标是否启用」， +于是照常按信号数累加内容高度、生成行矩形、绘制条目。 + +**修法**（`PlotViewPanels.cpp`）： +* 高度算式新增 `clipReady = clipPanel && CursorMode()==DOUBLE`： + 未就绪时 `content = 一行高度`（只放提示），不遍历信号。 +* 行几何循环加 `clipRowsVisible = !clipPanel || clipReadyHere`，未就绪时**不生成任何行矩形** + ⇒ 绘制侧（依赖 `layout.rowRefs`）自然什么都不画。 +* footer 也改为「未就绪 = 0」：`clipPanel ? (clipReady ? 两行按钮 : 0) : ...`。 + +### 2. ⭐ 条目区与按钮之间的大段空白 + +**两个叠加根因**： +1. 高度算式里 `maxContent = max(3*groupRow, min(上限, max(3*groupRow, budget)))` + —— 那个「至少 3 行」的下限让只有 1 个信号的面板硬撑到 3 行高。 + ⇒ 改为 `maxContent = min(上限, max(groupRow, budget))`（只保留 1 行下限）。 +2. 几何侧 `viewHeight = height - titleHeight - pad - pad - footerHeight` **多减了一个 pad**， + 而高度算式是 `height = title + pad + view + bottomPad + footer`（`bottomPad = pad`）。 + ⇒ 统一为 `viewHeight = height - titleHeight - pad - footerHeight`。 + ⚠️ **高度算式与内容区几何必须严格同源**，多减一个 pad 就会在底部留出一条空白。 + +### 3. ⭐⭐ 多个信号「原始信号截取」结果不对 + +**根因**：同组的信号**共享同一条时间轴**（合并网格/同文件多通道）。 +`DataSet::ClipSignalRange` 每次截取都会裁剪 `axes[signal.axisIndex].time` +⇒ 截完第一条，后面信号的时间基准已被改短 ⇒ 采样与时间错位（"截取出来的信号不对"）。 + +**修法**： +* 新增 `DataSet::ForkAxisForSignal(group, signal, &axisIndex)`： + 若该信号的时间轴**不是独占**，就复制一份新轴并把信号指向它（已独占则直接返回）。 +* `ClipSignalRange` 内再兜一层：检测到轴仍被其它信号共享时**直接返回 false**（不改任何数据）， + 避免任何调用方绕过 fork 造成错位。 +* `Application::ClipSignalInPlace`：**先** `ForkAxisForSignal` 再 `ClipSignalRange`。 +* 降序处理多个信号时，每条信号各自 fork 出自己的轴，互不影响。 + +### 4. ⭐ 有滚动条时滚轮滚不到底 + +**根因**（典型的三元式漏分支）： +`OnMouseWheel` 的 `current` 取值链只列了 CENTER/PEAK/COUNT/RESAMPLE，**漏了 CLIP** +⇒ 读到的是 `meanFilterScroll`（恒 0），而写入的是 `clipScroll` +⇒ 每次滚轮都从 0 开始算，永远只能滚一小步、滚不到底部。 +⇒ 补上 `(hoveredKind == CLIP) ? wheelLane.clipScroll : ...`。 + +**并发现第二个隐患**：`PanelSlotKey` 里 CLIP 的滚动量被写成 +`mix(lane.clipScroll > 0.0f ? 1 : 0)`（布尔）⇒ 滚到 1px 之后键不再变化、面板不重绘。 +⇒ 改为 `mixFloat(lane.clipScroll)`（真实值）。 + +### 5. 验证与提交 + +* 三处 `switch/tag` 均已逐个 grep 核对（`EdgePanelKind::RESAMPLE` 反查法）。 +* 删 `.o` 全量重编译：**零 error、零 warning**，产物 861,184 B（< 1MB）。 +* 用户要求不占用鼠标键盘屏幕 ⇒ 本轮**未跑任何 GUI 探针**，仅做编译验证与源码级核对。 diff --git a/Utilities/Inc/DataModel.hpp b/Utilities/Inc/DataModel.hpp index 14a51e8..f9f535a 100644 --- a/Utilities/Inc/DataModel.hpp +++ b/Utilities/Inc/DataModel.hpp @@ -689,6 +689,23 @@ namespace jdi */ bool ClipSignalRange(int groupIndex, int signalIndex, double low, double high); + /** + * @brief 让某条信号**独占一条时间轴**（复制一条新轴并把信号指向它）。 + * + * 同组的信号常常共享同一条时间轴（合并网格、同文件多通道等）。 + * 破坏性截取若要裁剪时间轴，就必须先让这条信号独占自己的轴， + * 否则会把其它信号的时间基准一起改短、导致它们数据错位 + * （用户 2026-11-03 报告："多个信号同时截取时截取出来的信号并不对"）。 + * + * 若该信号的时间轴**已经**被它独占，本函数不做任何事并返回 true。 + * + * @param [IN] groupIndex 分组下标。 + * @param [IN] signalIndex 信号下标。 + * @param [OUT] axisIndex 该信号（新）轴的下标。 + * @return 成功返回 true。 + */ + bool ForkAxisForSignal(int groupIndex, int signalIndex, std::size_t& axisIndex); + /** * @brief 移除指定下标的分组，并立即释放其占用的全部采样内存。 * @param [IN] groupIndex 分组下标。 diff --git a/Utilities/Src/DataModel.cpp b/Utilities/Src/DataModel.cpp index 159ceae..46809ae 100644 --- a/Utilities/Src/DataModel.cpp +++ b/Utilities/Src/DataModel.cpp @@ -499,6 +499,50 @@ namespace jdi return true; } + bool DataSet::ForkAxisForSignal(int groupIndex, int signalIndex, std::size_t& axisIndex) + { + axisIndex = 0; + if (groupIndex < 0 || groupIndex >= static_cast(m_groups.size())) + { + return false; + } + SignalGroup& group = m_groups[static_cast(groupIndex)]; + if (signalIndex < 0 || signalIndex >= static_cast(group.signals.size())) + { + return false; + } + SignalData& signal = group.signals[static_cast(signalIndex)]; + if (signal.axisIndex >= group.axes.size()) + { + return false; + } + // 已经独占 ⇒ 不需要复制（避免每次截取都新增一条轴）。 + bool shared = false; + for (std::size_t index = 0; index < group.signals.size(); ++index) + { + if (index == static_cast(signalIndex)) + { + continue; + } + if (group.signals[index].axisIndex == signal.axisIndex) + { + shared = true; + break; + } + } + axisIndex = signal.axisIndex; + if (!shared) + { + return true; + } + // 复制一条新轴（时间数组是只读共享，复制一份即可），并把信号指向它。 + TimeAxis copy = group.axes[signal.axisIndex]; + group.axes.push_back(std::move(copy)); + signal.axisIndex = group.axes.size() - 1; + axisIndex = signal.axisIndex; + return true; + } + bool DataSet::ClipSignalRange(int groupIndex, int signalIndex, double low, double high) { if (groupIndex < 0 || groupIndex >= static_cast(m_groups.size())) @@ -545,6 +589,32 @@ namespace jdi signal.values.swap(trimmed); signal.values.shrink_to_fit(); } + // ⚠️⚠️ **时间轴可能被同组的多条信号共享**（合并网格里常见）： + // 只有在"本组没有任何**其它**信号使用同一条轴"时才可以裁剪它； + // 否则裁剪会把其它信号的时间基准一起改短 ⇒ 它们的数据错位 + // （用户 2026-11-03 报告："多个信号同时截取时截取出来的信号并不对"）。 + // 共享轴的情况由调用方先把该信号改成使用**自己的一条新轴**（见 `Application::ClipSignalInPlace`）。 + bool axisShared = false; + for (std::size_t index = 0; index < group.signals.size(); ++index) + { + if (index == static_cast(signalIndex)) + { + continue; + } + if (group.signals[index].axisIndex == signal.axisIndex) + { + axisShared = true; + break; + } + } + if (axisShared) + { + // 共享轴：只裁剪采样值，时间基准保持原样（采样仍按原下标对齐， + // 因此需要把「区间内」的采样**原样**放回原位置？不——那样会让值与时间错位）。 + // ⇒ 正确做法：为该信号派生一条**独立时间轴**，把它挂到新轴上。 + // 这里直接返回失败，由调用方（宿主）先 `ForkAxisForSignal` 再重试。 + return false; + } { std::vector trimmedTime; trimmedTime.swap(keptTime);