Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ParsedCurveGenerator #22848

Closed
miaoyinb opened this issue Dec 1, 2022 · 2 comments · Fixed by #22849
Closed

Add ParsedCurveGenerator #22848

miaoyinb opened this issue Dec 1, 2022 · 2 comments · Fixed by #22849
Labels
C: Framework T: task An enhancement to the software.

Comments

@miaoyinb
Copy link
Contributor

miaoyinb commented Dec 1, 2022

Reason

It has been found needed to have a general curve mesh generator to enable meshing of some complex shapes. The mesh generator should be able to work with existing FillBetweenPointVectorsTools and XYDelaunayGenerator to expand their meshing capabilities.

Design

ParsedCurveGenerator will be added to generate a curve mesh that is defined by (x(t), y(t), z(t)) using the Function Parser library that is already adopted by MOOSE.

FillBetweenCurvesGenerator will be added to work with the curves generated using the FillBetweenPointVectorsTools.

The generated curves (if closed) should be naturally compatible with XYDelaunayGenerator.

Impact

Enable meshing of some complex geometry features. This has been found needed for some reactor designs.

@miaoyinb miaoyinb added the T: task An enhancement to the software. label Dec 1, 2022
miaoyinb added a commit to miaoyinb/moose that referenced this issue Dec 1, 2022
@GiudGiud
Copy link
Contributor

GiudGiud commented Dec 1, 2022

Does the curve need to be closed?

@miaoyinb
Copy link
Contributor Author

miaoyinb commented Dec 2, 2022

Does the curve need to be closed?

Both closed and open curves can be meshed.

miaoyinb added a commit to miaoyinb/moose that referenced this issue Dec 2, 2022
miaoyinb added a commit to miaoyinb/moose that referenced this issue Dec 2, 2022
MengnanLi91 pushed a commit to MengnanLi91/moose that referenced this issue Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Framework T: task An enhancement to the software.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants