Skip to content

fix: add missing widget Python files to meson.build install list#3

Draft
Copilot wants to merge 1 commit intomainfrom
copilot/fix-action-run-issue
Draft

fix: add missing widget Python files to meson.build install list#3
Copilot wants to merge 1 commit intomainfrom
copilot/fix-action-run-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

Summary

Fixes the Windows CI failure (run #23066915267) caused by:

ModuleNotFoundError: No module named 'themachinethatgoesping.pingprocessing.widgets.wciviewer_jupyter'

Root Cause

12 new Python files in the widgets directory were not added to the sources list in python/meson.build, so meson did not install them. Since widgets/__init__.py imports these modules unconditionally, the missing files caused ModuleNotFoundError at runtime.

Changes

Added the following missing files to python/meson.build:

  • control_jupyter.py
  • control_qt.py
  • control_spec.py
  • echogramviewer_core.py
  • echogramviewer_jupyter.py
  • echogramviewer_qt.py
  • mapviewer_core.py
  • mapviewer_jupyter.py
  • mapviewer_qt.py
  • wciviewer_core.py
  • wciviewer_jupyter.py
  • wciviewer_qt.py

Fixes Windows CI failure: ModuleNotFoundError for wciviewer_jupyter and
other new widget modules that were not included in the meson install sources.

Agent-Logs-Url: https://github.com/themachinethatgoesping/pingprocessing/sessions/9f8b2d08-c414-42aa-897e-e01ca39fa6c9

Co-authored-by: peter-urban <3885740+peter-urban@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants