tektronix / tektronix/tm_data_types
[BUG]: reading mat file may read entire file twice
Open
Nobody has claimed this yet.
bug
- Dominant language
- Python
- Stars
- 8
- Forks
- 14
- Avg merge
- 3d 10h
- Merged PRs (30d)
- 4
Description
Description of the bug
There is no exception clause when checking the type for the .mat file, so it may read through the entire thing before erroring out.
Steps To Reproduce
No response
Environment Information
No response
Additional Information
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the Python entry point that checks the type of a .mat file and reads its contents, then reproduce the reported path with a representative file. Confirm the fix by verifying that type checking does not consume and reread the entire file before reporting an error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100