Skip to content

Adjustment for gempyor.seir.write_seir args#589

Merged
TimothyWillard merged 1 commit into
devfrom
push-uyuyyqtonqxl
Aug 4, 2025
Merged

Adjustment for gempyor.seir.write_seir args#589
TimothyWillard merged 1 commit into
devfrom
push-uyuyyqtonqxl

Conversation

@TimothyWillard
Copy link
Copy Markdown
Contributor

Follow up to #585 to account for the arguments of gempyor.seir.write_seir taking the out dataframe instead of the states, leaving it to the user to call gempyor.seir.states2DF.

@TimothyWillard TimothyWillard added bug Defects or errors in the code. gempyor Concerns the Python core. medium priority Medium priority. next release Marks a PR as a target to include in the next release. labels Jul 25, 2025
@TimothyWillard TimothyWillard force-pushed the push-uyuyyqtonqxl branch 3 times, most recently from a8650e2 to 1438cbc Compare July 30, 2025 16:57
@TimothyWillard TimothyWillard mentioned this pull request Jul 30, 2025
31 tasks
Copy link
Copy Markdown
Contributor

@pearsonca pearsonca left a comment

Choose a reason for hiding this comment

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

seems good in the surgical change sense.

... what a mess generally tho. this almost all ultimately needs to change

@TimothyWillard TimothyWillard self-assigned this Jul 31, 2025
Follow up to #585 to account for the arguments of
`gempyor.seir.write_seir` taking the out dataframe instead of the
states, leaving it to the user to call `gempyor.seir.states2DF`.
@TimothyWillard TimothyWillard merged commit 2434d72 into dev Aug 4, 2025
12 checks passed
@TimothyWillard TimothyWillard deleted the push-uyuyyqtonqxl branch August 4, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Defects or errors in the code. gempyor Concerns the Python core. medium priority Medium priority. next release Marks a PR as a target to include in the next release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants