Skip to content

Add TDMS support and data pipeline helpers - #1

Open
vudxlab wants to merge 7 commits into
masterfrom
codex/update-process_data.py-for-data-handling
Open

vudxlab wants to merge 7 commits into
masterfrom
codex/update-process_data.py-for-data-handling

Conversation

@vudxlab

@vudxlab vudxlab commented Nov 25, 2025

Copy link
Copy Markdown
Owner

Summary

  • add shared pipeline functions in process_data for discovering files, loading .mat/.tdms data, processing, and visualization
  • support TDMS channel extraction alongside existing MATLAB loader and ensure mixed datasets are handled
  • simplify main to minimal CLI that delegates to the new pipeline interfaces

Testing

  • Not run (not requested)

Codex Task

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread process_data.py Outdated
import pandas as pd
from scipy.signal import butter, lfilter, decimate
import scipy.io
from nptdms import TdmsFile

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Handle missing TDMS dependency gracefully

The module now imports TdmsFile unconditionally, so python main.py or any .mat-only workflow will raise ModuleNotFoundError: No module named 'nptdms' on environments that only had the previous MATLAB dependencies. TDMS support was added as an optional feature, but this import makes it a hard requirement and breaks existing usage unless users install an extra package first. Guard the import or defer it until a .tdms file is actually processed so .mat runs continue to work.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant