Skip to content

Tweaks to simplify working with coding agents#2086

Merged
speth merged 5 commits intoCantera:mainfrom
speth:codex-tweaks
Feb 11, 2026
Merged

Tweaks to simplify working with coding agents#2086
speth merged 5 commits intoCantera:mainfrom
speth:codex-tweaks

Conversation

@speth
Copy link
Member

@speth speth commented Feb 10, 2026

Changes proposed in this pull request

  • Provide workaround for calling sysctl, which is not allowed within Codex's sandbox
  • Avoid repeated instantiations of std::locale objects which slowed down running tests within Codex's sandbox
  • Add a couple of configuration modifications to make working with Pixi easier. These are just the changes that pixi init wants to make to already-tracked files. I'm holding off on adding pixi.toml or pixi.lock files for the time being. The motivation for this is in part that I think it's a little easier to get agents to run build and test commands when they don't have to figure out what Conda environment they're supposed to activate.
  • A couple of other minor changes that I didn't want to roll into Convert kinetics & equilibrium code to use std::span #2085.

AI Statement (required)

  • No generative AI was used. This contribution was written entirely without AI assistance.

Checklist

  • The pull request includes a clear description of this code change
  • Commit messages have short titles and reference relevant issues
  • Build passes (scons build & scons test) and unit tests address code coverage
  • Style & formatting of contributed code follows contributing guidelines
  • AI Statement is included
  • The pull request is ready for review

This allows running scons in sandboxes like those used by OpenAI Codex.
Repeated calls to 'locale' in the OpenAI Codex sandbox are very slow.
Making this static resolves the problem, and also modestly speeds up
input parsing in general.
@codecov
Copy link

codecov bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.43%. Comparing base (9470b53) to head (46e6bc1).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2086   +/-   ##
=======================================
  Coverage   77.42%   77.43%           
=======================================
  Files         451      451           
  Lines       52718    52714    -4     
  Branches     8839     8838    -1     
=======================================
+ Hits        40818    40819    +1     
+ Misses       8931     8926    -5     
  Partials     2969     2969           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@speth speth marked this pull request as ready for review February 10, 2026 21:12
Copy link
Member

@ischoegl ischoegl left a comment

Choose a reason for hiding this comment

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

Thanks - this all looks good to me!

@speth speth merged commit 93f8c18 into Cantera:main Feb 11, 2026
91 of 92 checks passed
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