Skip to content

Rename python Driver method 'Postprocessing' to 'Finalize'#1976

Merged
pcarruscag merged 2 commits intosu2code:developfrom
kursatyurt:rename_postprocessing
Mar 21, 2023
Merged

Rename python Driver method 'Postprocessing' to 'Finalize'#1976
pcarruscag merged 2 commits intosu2code:developfrom
kursatyurt:rename_postprocessing

Conversation

@kursatyurt
Copy link
Copy Markdown
Contributor

Proposed Changes

The driver method Postprocessing() actually frees all the containers and finalizes the driver. As it does not reflect what is actually done and is easy to confuse with the method Postprocess(), the PR proposes renaming the method to Finalize().

PR Checklist

Put an X by all that apply. You can fill this out after submitting the PR. If you have any questions, don't hesitate to ask! We want to help. These are a guide for you to know what the reviewers will be looking for in your contribution.

Copy link
Copy Markdown
Member

@pcarruscag pcarruscag left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@pcarruscag pcarruscag changed the title Rename Driver method Postprocessing to Finalize Rename python Driver method 'Postprocessing' to 'Finalize' Mar 21, 2023
@pcarruscag pcarruscag merged commit 15fabde into su2code:develop Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants