Skip to content

Generalized Tecplot ASCII and Binary Readers/Writers #95

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
lamkina wants to merge 30 commits into
base: main
Choose a base branch
from
Open

Generalized Tecplot ASCII and Binary Readers/Writers #95

Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
6c8fc28
Implemented tecplot ascii and binary readers and writers for n-dimens…
Jun 29, 2024
16191ce
Fixing black
Jun 29, 2024
3718d06
Adding docs for tecplot module
Jun 29, 2024
b277258
Bumped the minor version
Jun 29, 2024
0f35428
Fixed exception in external io test to catch all errors
Jun 29, 2024
b03bcac
Adding zone input validation, better use of enums, refactoring, and b…
Jul 20, 2024
9ebce66
Fixing regex for zone name matching
Aug 4, 2024
0288bee
Added terminating comma to regex pattern for zone header
Aug 4, 2024
d8fdc99
Fixed regex for zone name matching
Sep 13, 2024
660609b
Updating tecplot writers in the weight problem
Sep 13, 2024
f88c903
Fixing if checks for types
Sep 13, 2024
47903db
Updating tecplot writers in the aero solver
Sep 13, 2024
75f59ea
Fixing strand ID and solution time bugs in ASCII writer
Sep 20, 2024
80196c6
Fixed random data in ordered zones and added stress testing
Sep 20, 2024
63169ee
Added block format writer and reader to adhere to line width constraints
Sep 24, 2024
a503e8c
Added line length test for ASCII files
Sep 24, 2024
fc5c1c2
Improved the ascii data reader to use multiple separators and ignore …
Sep 24, 2024
066a6bd
Added full reader and writer separator support with tests
Sep 24, 2024
449c9b9
Cleaning up documentation, imports, and api
Sep 25, 2024
c2baf7c
Added tri connectivity property for FE zones with tests
Nov 8, 2024
fc5fbe8
Flake 8 is always watching
Nov 8, 2024
1e44f71
Added unique indices and nodes to fe zones
Nov 8, 2024
b9b83aa
New connectivity handling for FE zones with unique data and connectiv…
Nov 10, 2024
b30741e
Fixed remap connectivity bug
Nov 12, 2024
99e04c7
Switched remap connectivity to numpy for sppeeeed
Nov 12, 2024
7fc9a08
Updgraded the zone name matching to be more robust
Nov 19, 2024
512543f
Fixing ordered zones i,j,k ordering bug
May 2, 2025
61044d1
Merge branch 'main' into file_io
gawng May 15, 2025
036c57d
Fixing small bug with header regex
Aug 22, 2025
3399dda
Merge branch 'file_io' of github.com:lamkina/baseclasses into file_io
Aug 22, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions baseclasses/utils/tecplotIO.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1154,8 +1154,8 @@ def _readZoneHeader(self, lines: List[str], iCurrent: int) -> Tuple[Dict[str, An
headerString = ", ".join(header)

# Use regex to parse the header information
zoneNameMatch = re.search(r'zone t\s*=\s*"([^"]*)"', headerString, re.IGNORECASE)
zoneName = zoneNameMatch.group(1) if zoneNameMatch else None
zoneNameMatch = re.search(r'(zone t)\s*=\s*[\'""]?([^\'""\n,]+)[\'""]?(?=[,\n]|$)', headerString, re.IGNORECASE)

@A-CGray A-CGray Nov 19, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For future people's sake, can you add a comment here explaining what this regex matches?

@lamkina lamkina Nov 19, 2024 •
edited
Loading

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yeah I need to review and add more tests for the regex patterns. To my knowledge, there's no established rules published by Tecplot that describe the formatting of the ASCII header data. I'm considering simpler approach that removes the regex and just assumes all data follows an = sign and terminates with either a comma or newline unless it's surrounded by single or double quotes.

zoneName = zoneNameMatch.group(2) if zoneNameMatch else None

zoneTypeMatch = re.search(r"zonetype\s*=\s*(\w+)", headerString, re.IGNORECASE)
zoneType = zoneTypeMatch.group(1) if zoneTypeMatch else "ORDERED"
Expand Down