-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Helical extrusion feature during extrusion around an axis #25089
Conversation
6f5aa31
to
f9275f5
Compare
Job Documentation on dfa374e wanted to post the following: View the site here This comment will be updated on new commits. |
85302f9
to
3395016
Compare
Job Coverage on dfa374e wanted to post the following: Framework coverage
Modules coverageCoverage did not change Full coverage reportsReports
This comment will be updated on new commits. |
3395016
to
dbb4207
Compare
and trying to get their shape corrected refs idaholab#23972
Add a test for helicoidal extrusion refs idaholab#25486
27a5467
to
7ab3a06
Compare
Job Precheck on 348df45 wanted to post the following: Your code requires style changes. A patch was auto generated and copied here
Alternatively, with your repository up to date and in the top level of your repository:
|
541b987
to
7d7d625
Compare
more constness cache node lists Co-authored-by: Alex Lindsay <alexander.lindsay@inl.gov>
7d7d625
to
dfa374e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
failures not related
Thanks for the review! |
Useful to make helicoidal fuel like the Lightbridge fuel
One thing to note is that we could also just parametrize everything with "s" being a curve absciss,
then use parsed functions for x(s), y(s), z(s). This would cover regular extrusion, helical, and many more.
Fun (complex/parsed) extrusion like projects I could see:
The additional changes are things I ended up needing in the project to get the geometry to mesh properly and be able to correct the volume.
closes #25486