Skip to content

[WIP] Fix wrong spacing for chemkin spacing default temperature range#2962

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-chemkin-spacing-default-temperature-range
Closed

[WIP] Fix wrong spacing for chemkin spacing default temperature range#2962
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-chemkin-spacing-default-temperature-range

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 22, 2026

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>wrong spacing for chemkin spacing default temperature range</issue_title>
<issue_description>

Bug Description

The spacing for the default thermo temperature range in a chemkin file is different for the surface one than for the gas one:

Surface code

f.write(' 300.000 1000.000 5000.000\n\n')

Gas code

f.write(' 300.000 1000.000 5000.000\n\n')

This prevents the default temperature ranges from getting read in.

How To Reproduce

Generate an RMG surface mechanism and then try to read in the chemkin file.
</issue_description>

Comments on the Issue (you are @copilot in this section)

@rwest
Copy link
Copy Markdown
Member

rwest commented May 22, 2026

I think this is fixed in #2955 ?

@rwest rwest closed this May 22, 2026
@rwest rwest deleted the copilot/fix-chemkin-spacing-default-temperature-range branch May 22, 2026 14:19
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.

wrong spacing for chemkin spacing default temperature range

3 participants