Skip to content

fix(cc): look up cc responses by table head, not brittle index - #157

Open
nblair2 wants to merge 1 commit into
sandialabs:mainfrom
nblair2:fix-cc-commands-columns
Open

nblair2 wants to merge 1 commit into
sandialabs:mainfrom
nblair2:fix-cc-commands-columns

Conversation

@nblair2

@nblair2 nblair2 commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Description

Wheny utils.py parsed mm cc responses, it used a brittle list of column names and indexes. When these chagne (like in https://github.com/sandia-minimega/minimega/releases/tag/3.3.0), this breaks. We replace the fixed list with a function to parse the returned header that minimega sends back.

Related Issues/PRs

Brittle index exposed by change in minimega 3.3.0, specifically: sandia-minimega/minimega#1655

Type of Change

Please select the type of change your pull request introduces:

  • Bugfix (fix)
  • Feature (feat)
  • Documentation (docs)
  • Refactor (refactor)
  • Chore (CI, build, dependencies, etc.) (chore)
  • Other (please describe):

Checklist

  • This PR conforms to the process detailed in the Contributing Guide.
  • I have included no proprietary/sensitive information in my code or the PR.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have tested my code (describe below).

Testing

Tested with scorch run that was previously failing, it now passes. Only tested with minimega 3.3.0 - not tested with previous versions.

Additional Notes

N/A

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@nblair2
nblair2 force-pushed the fix-cc-commands-columns branch from fe199b3 to efc9d13 Compare October 8, 2026 20:38
@nblair2
nblair2 marked this pull request as ready for review October 8, 2026 20:40
Comment thread CHANGELOG.md
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