Skip to content

Conversation

@rwalkerlewis
Copy link
Owner

@rwalkerlewis rwalkerlewis commented Dec 8, 2025

Fix Python 2 print statements and invalid escape sequences to resolve CI test failures.

This PR addresses two main categories of Python syntax issues:

  1. Python 2 print statements: Converted print statements in legacy data generation and restart check scripts to Python 3 print() function calls.
  2. Invalid escape sequences: Changed regular expression strings in TestPetscVersion.py and LaTeX strings in plot_slipfn.py to raw strings (r"...") to correctly handle backslashes and prevent SyntaxWarnings.

Open in Cursor Open in Web


Note

Converts legacy print statements to Python 3, fixes invalid escape sequences, and adds thermoelastic/thermoporoelastic pytest modules.

  • Tests:
    • Add materials/TestThermoelasticity.py and materials/TestThermoporoelasticity.py; register in tests/pytests/Makefile.am and materials/__init__.py.
    • Extend materials/TestAuxiliarySubfields.py to include AuxSubfieldsThermoelasticity and AuxSubfieldsThermoporoelasticity.
    • Use raw regex strings in utils/TestPetscVersion.py.
  • Scripts/Utilities:
    • Update Python 2 print statements to Python 3 print() across playpen/viscorestart/*.py, tests/libtests/bc/data/absorbingdampers.py, tests/libtests/faults/data/{adjustsoln.py,cohesivedyn.py,slipth.py}.
    • Minor output tweak in playpen/viscorestart/run_all.py.
  • Docs:
    • Use raw LaTeX strings for tick labels in docs/user/physics/faults/figs/plot_slipfn.py.

Written by Cursor Bugbot for commit f866c93. This will update automatically on new commits. Configure here.

This commit updates print statements to use the new print() function syntax and modifies regular expression patterns to use raw strings.

Co-authored-by: langleyreview <[email protected]>
@cursor
Copy link

cursor bot commented Dec 8, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@rwalkerlewis rwalkerlewis marked this pull request as ready for review December 8, 2025 07:18
@rwalkerlewis
Copy link
Owner Author

@copilot fix stage 1 CI tests. it is failing at build docker image

Copy link

Copilot AI commented Dec 8, 2025

@rwalkerlewis I've opened a new pull request, #10, to work on those changes. Once the pull request is ready, I'll request review from you.

Investigate CI Stage 1 Docker build failure
@rwalkerlewis rwalkerlewis merged commit 1b2e834 into thermo Dec 8, 2025
1 check passed
@rwalkerlewis rwalkerlewis deleted the cursor/check-and-fix-ci-tests-claude-4.5-opus-high-thinking-ef57 branch December 8, 2025 07:43
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.

3 participants