-
Notifications
You must be signed in to change notification settings - Fork 61
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
DEM example with a particle discharge from a plate with ASC and LB #1191
Conversation
doc/source/examples/dem/plate-discharge/images/angle-of-repose.png
Outdated
Show resolved
Hide resolved
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.
one file left. I will look at it this PM
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.
I have review the .rst. I will complete my review tomorrow :)
Really nice example!
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.
Great work! I have no major comment
doc/source/examples/unresolved-cfd-dem/dense-pneumatic-conveying/dense-pneumatic-conveying.rst
Outdated
Show resolved
Hide resolved
doc/source/examples/unresolved-cfd-dem/dense-pneumatic-conveying/dense-pneumatic-conveying.rst
Outdated
Show resolved
Hide resolved
examples/dem/3d-plate-discharge/plate-discharge_post-processing.py
Outdated
Show resolved
Hide resolved
Personally I don't have additional comments atop the already good ones that have been posted here. |
Co-authored-by: Olivier Guévremont <guevremont.o@gmail.com> Co-authored-by: hepap <47506601+hepap@users.noreply.github.com>
9bd41a2
to
5b877ef
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.
Great work! And amazing figures!! I have a few minor comments :)
examples/dem/3d-plate-discharge/plate-discharge_post-processing.py
Outdated
Show resolved
Hide resolved
doc/source/examples/unresolved-cfd-dem/dense-pneumatic-conveying/dense-pneumatic-conveying.rst
Outdated
Show resolved
Hide resolved
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.
I'll just wait for the last comments to be addressed and the ok from @hepap. Once those are OK, you can add the reviewed and ready to merge label :)
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.
Very nice! :)
I think I adressed all comments, thank you guys :) |
…haos-polymtl#1191) Description The example is a simple case of particle discharging from a plate that compares the computational performance and the angle of repose of simulations using the Adaptive Sparse Contacts, the dynamic Load Balancing and both of them from a baseline simulation. The simulations have 52000 particles and run under 45 minutes each. This is the first example describing the ASC method. Co-authored-by: Olivier Guévremont <guevremont.o@gmail.com> Co-authored-by: hepap <47506601+hepap@users.noreply.github.com> Former-commit-id: 26d8692
…1191) Description The example is a simple case of particle discharging from a plate that compares the computational performance and the angle of repose of simulations using the Adaptive Sparse Contacts, the dynamic Load Balancing and both of them from a baseline simulation. The simulations have 52000 particles and run under 45 minutes each. This is the first example describing the ASC method. Co-authored-by: Olivier Guévremont <guevremont.o@gmail.com> Co-authored-by: hepap <47506601+hepap@users.noreply.github.com> Former-commit-id: 26d8692
Description
The example is a simple case of particle discharging from a plate that compares the computational performance and the angle of repose of simulations using the Adaptive Sparse Contacts, the dynamic Load Balancing and both of them from a baseline simulation.
The simulations have 52000 particles and run under 45 minutes each.
This is the first example describing the ASC method.
Miscellaneous (will be removed when merged)
I will have to rebase to master after the example of Cleo is merged because of the conflicts in the .rst DEM exemples page.
Checklist (will be removed when merged)
See this page for more information about the pull request process.
Code related list:
Pull request related list: