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

Implement Morris surface tension model #584

Open
wants to merge 485 commits into
base: main
Choose a base branch
from

Conversation

svchb
Copy link
Collaborator

@svchb svchb commented Jul 29, 2024

Depends on #539
Depends on #604
Depends on #605
Depends on #606
Depends on #607
Depends on #608
Depends on #609
Depends on #599
super seeds #523

@svchb svchb marked this pull request as draft July 29, 2024 12:54
@svchb svchb added the enhancement New feature or request label Jul 29, 2024
Copy link

codecov bot commented Aug 1, 2024

Codecov Report

Attention: Patch coverage is 69.14894% with 87 lines in your changes missing coverage. Please review.

Project coverage is 69.81%. Comparing base (2540f08) to head (19f2e88).

Files with missing lines Patch % Lines
src/schemes/fluid/surface_tension.jl 60.71% 22 Missing ⚠️
src/visualization/write2vtk.jl 0.00% 18 Missing ⚠️
src/schemes/fluid/fluid.jl 28.57% 10 Missing ⚠️
...rc/schemes/fluid/entropically_damped_sph/system.jl 20.00% 8 Missing ⚠️
src/schemes/fluid/surface_normal_sph.jl 88.73% 8 Missing ⚠️
src/general/semidiscretization.jl 72.72% 6 Missing ⚠️
src/preprocessing/particle_packing/system.jl 0.00% 6 Missing ⚠️
...rc/schemes/fluid/weakly_compressible_sph/system.jl 70.58% 5 Missing ⚠️
src/general/system.jl 66.66% 2 Missing ⚠️
src/schemes/boundary/system.jl 90.47% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #584      +/-   ##
==========================================
- Coverage   70.53%   69.81%   -0.72%     
==========================================
  Files          96       96              
  Lines        5976     6139     +163     
==========================================
+ Hits         4215     4286      +71     
- Misses       1761     1853      +92     
Flag Coverage Δ
unit 69.81% <69.14%> (-0.72%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

svchb added a commit to svchb/TrixiParticles.jlOpen that referenced this pull request Sep 27, 2024
svchb added a commit to svchb/TrixiParticles.jlOpen that referenced this pull request Sep 27, 2024
@svchb svchb mentioned this pull request Oct 7, 2024
svchb added 12 commits October 10, 2024 15:56
Conflicts:
	examples/fluid/dam_break_oil_film_2d.jl
	src/schemes/fluid/weakly_compressible_sph/density_diffusion.jl
	src/schemes/fluid/weakly_compressible_sph/rhs.jl
	src/schemes/fluid/weakly_compressible_sph/state_equations.jl
	src/schemes/fluid/weakly_compressible_sph/system.jl
	src/visualization/write2vtk.jl
@svchb svchb requested a review from efaulhaber January 28, 2025 15:18
@svchb
Copy link
Collaborator Author

svchb commented Jan 28, 2025

@efaulhaber Still was not able to find the allocations that occur in "sphere_surface_tension_2d.jl"

@efaulhaber still the case

LasNikas pushed a commit that referenced this pull request Jan 30, 2025
* fix merge

* fix merge

* fix merge

* fix merge

* fix merge

* more merge fixes

* format

* fix

* fix

* fix

* fix

* fix

* more fixes

* fix

* fix

* fix

* more fixes

* fix

* fix

* format

* running

* update

* update

* optimization

* update

* update

* update

* update

* format

* up

* update

* remove unused code

* switch to support radius

* update

* fix doc

* add adhesion coefficient

* update_broken

* update

* format

* add to examples

* fix tests

* fix merge

* fix typos

* add basic test

* format

* rename

* format

* rename container system

* reduce example run time

* update news and readme

* format

* fix test

* reduce run time

* format

* fix

* correct merge

* update doc test

* revert

* fix tests

* fix

* fix

* review comments

* fix

* fix

* correct some stuff

* init with empty initial condition

* review update

* rename function

* remove unnecessary if

* docs

* fix doc

* revert one change

* fix typo

* update

* try to avoid allocs

* fix mem allocs

* review update

* update docs

* update

* review

* fix

* format

* make examples smaller

* reduce resolution

* review

* rename

* use trixi_include

* format

* use trixi_include

* update

* update

* review update

* format

* fix test errors

* update

* fix

* remove invalid surface normals

* add

* fix

* update

* update

* update

* fix

* update

* update

* update

* remove unused

* remove unused

* format

* fix tests

* fix

* fix

* fix naming

* format

* fix test

* set test up for 1.11

* fix test

* rename to number_density

* back merge

* fix

* format

* fix

* fix tests

* back merge from #584

* move additonal changes from #584

* add test

* add test

* format

* Increase errors for 1.11

* Fix invalidations

* Fix tests

* Update ci.yml

* revert

* Update ci.yml

* Update test/validation/validation.jl

Co-authored-by: Erik Faulhaber <[email protected]>

* format

* Update news and set to 0.2.4

* update

* fix MD format

* Update NEWS.md

* some fixes

* add docs

* review fixes

* remove independent setting of smoothing_kernel and smoothing_length

* remove calls to surface_normal method smoothing kernel

* format

* update news

* typo

* Update NEWS.md

* forgot some renames

* fix doc tests

* fix docs

* fix tests

* format

* Update NEWS.md

* fix tests

* Update NEWS.md

* fix

* fix

* fix

* fix

* review fixes

* review comments

* fix test

* typo

* review updates

* review update

* add boundary system

* update

* update

* format

* remove color since it is unused in this PR

* review comment

* format

* merge error

* review comment

* fix

* remove surface_normal_method()

* merge error

* function was renamed in PN

* format

* fix test

* fix error

* fix

* review comments

* review comments

* review comments

* format

* merge

* format

* add comment

* remove ideal_neighbor()

* format

* typo

* add missing docs

* only initialize necessary values

* fix test

* fix examples

---------

Co-authored-by: Erik Faulhaber <[email protected]>
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