Skip to content

Adding surface postprocessor#1048

Open
andrsd wants to merge 3 commits into
Open-Sn:mainfrom
andrsd:surf-pps
Open

Adding surface postprocessor#1048
andrsd wants to merge 3 commits into
Open-Sn:mainfrom
andrsd:surf-pps

Conversation

@andrsd

@andrsd andrsd commented May 1, 2026

Copy link
Copy Markdown
Collaborator

PR Checklist

  • I have updated the user guide and/or Python API documentation if necessary.

@andrsd andrsd self-assigned this May 1, 2026
@andrsd andrsd added the enhancement New feature or request label May 1, 2026
@andrsd andrsd marked this pull request as draft May 1, 2026 15:56
@andrsd

andrsd commented May 1, 2026

Copy link
Copy Markdown
Collaborator Author

This is on top of PR 1047.
I will add doco sometime today.
This is to show the code and get feedback.
Testing is not as extensive as for the volumetric PPS, so any ideas on improving that are welcome.

@andrsd andrsd requested review from ragusa and wdhawkins May 1, 2026 15:58
@andrsd andrsd force-pushed the surf-pps branch 3 times, most recently from b028f17 to 119541b Compare May 8, 2026 13:00
@andrsd andrsd marked this pull request as ready for review May 13, 2026 18:18
problem=phys,
value_type="min",
current_type="net",
boundaries=["xmin"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can we add an example with boundary IDs as well?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I am assuming you are asking to add a tutorial. What problem should I use to demonstrate this on? These tests are kinda contrived. I can reuse the test, but if you have something better in mind, I would use that instead, I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants