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.

Sign up for GitHub

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

Update restart files of application tests for p4est 2.3.6 #1181

Merged
merged 14 commits into from
Jul 1, 2024
Merged

Update restart files of application tests for p4est 2.3.6 #1181

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
7e6f707
Update restart files in cfd-dem
acdaigneault Jun 28, 2024
fccba1d
Remove useless dem restart in cmake file
acdaigneault Jun 28, 2024
fc0755b
Update the lethe-fluid-vans restart files
acdaigneault Jun 28, 2024
cca24f4
Update poiseuille restart files
acdaigneault Jun 28, 2024
5fff0c3
Add a restart file generator for sharp
acdaigneault Jun 28, 2024
256a151
Update lethe-particles tests
acdaigneault Jun 28, 2024
69c2c16
Change log
acdaigneault Jun 28, 2024
1b4a5d0
Indent
acdaigneault Jun 28, 2024
7f7c874
Attempt to unable test with deal.ii v9.5
acdaigneault Jun 29, 2024
6a58c6c
Fix typo
acdaigneault Jun 29, 2024
739152b
Use yaml ifs
acdaigneault Jun 29, 2024
502c4b8
Change dp in dem generator of restart fluidized bed file
acdaigneault Jun 30, 2024
3660833
Change position of initial particles in some test
acdaigneault Jun 30, 2024
fdfc87f
Remove the statistics
acdaigneault Jun 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE/pull_request_template_bug-fix.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ See [this page](https://chaos-polymtl.github.io/lethe/documentation/contributing
Code related list:
- [ ] All in-code documentation related to this PR is up to date (Doxygen format)
- [ ] Lethe documentation is up to date
- [ ] Fix has unit test(s) (preferred) or application test(s)
- [ ] Fix has unit test(s) (preferred) or application test(s), and restart files are in the generator folder
- [ ] The branch is rebased onto master
- [ ] Changelog (CHANGELOG.md) is up to date
- [ ] Code is indented with indent-all and .prm files (examples and tests) with prm-indent
Expand Down
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE/pull_request_template_new-feature.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ See [this page](https://chaos-polymtl.github.io/lethe/documentation/contributing
Code related list:
- [ ] All in-code documentation related to this PR is up to date (Doxygen format)
- [ ] Lethe documentation is up to date
- [ ] New feature has unit test(s) (preferred) or application test(s)
- [ ] New feature has unit test(s) (preferred) or application test(s), and restart files are in the generator folder
- [ ] The branch is rebased onto master
- [ ] Changelog (CHANGELOG.md) is up to date
- [ ] Code is indented with indent-all and .prm files (examples and tests) with prm-indent
Expand Down
7 changes: 7 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,13 @@
All notable changes to the Lethe project will be documented in this file.
The format is based on [Keep a Changelog](http://keepachangelog.com/).

## [Master] - 2024-06-16

### Changed

- MINOR Some application-test restart files have been updated using p4est 2.3.6. Some test results have changed for lethe-fluid-particles and lethe-fluid-vans, since the DEM solver has slightly changed since the previous restart files generation. [#1181](https://github.com/chaos-polymtl/lethe/pull/1181)
acdaigneault marked this conversation as resolved.
Show resolved Hide resolved


## [Master] - 2024-06-16

### Changed
Expand Down
7 changes: 0 additions & 7 deletions applications_tests/lethe-fluid-particles/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,6 @@ file(COPY particle_sedimentation_files/dem.triangulation.info DESTINATION "${CMA
file(COPY particle_sedimentation_files/dem.triangulation_fixed.data DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY particle_sedimentation_files/dem.triangulation_variable.data DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/particle_sedimentation.${_build_type}/mpirun=1/")

acdaigneault marked this conversation as resolved.
Show resolved Hide resolved
file(COPY restart_particle_sedimentation_files/dem.particles DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/dem.pvdhandler DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/dem.simulationcontrol DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/dem.triangulation DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/dem.triangulation.info DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/dem.triangulation_fixed.data DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/dem.triangulation_variable.data DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/case.particles DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/case.pvdhandler DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
file(COPY restart_particle_sedimentation_files/case_particles.pvdhandler DESTINATION "${CMAKE_CURRENT_BINARY_DIR}/restart_particle_sedimentation.${_build_type}/mpirun=1/")
Expand Down
42 changes: 21 additions & 21 deletions applications_tests/lethe-fluid-particles/adaptive_sparse_contacts.mpirun=1.output
View file
Open in desktop
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does the new p4est version cause the changes in the application_tests? Or did the tests have been changed change?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

In this case I'm not sure, but I don't think so. A few things have changed in the DEM solver that causes tiny differences. The mobility status can change in one iteration.
I build this test and it was pretty hard to get all the status with a very small number of particles. The test is probably very sensitive to the initial condition. Or the generation files in the repo were not the good ones.

Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,11 @@ DEM contact search at dem step 0
DEM contact search at dem step 1
Finished 50 DEM iterations
---------------------------------------------------------------
Global continuity equation error: 3.13678e-09 s^-1
Max local continuity error: 6.13917e-08 s^-1
Global continuity equation error: 2.803e-09 s^-1
Max local continuity error: 7.07295e-08 s^-1

**********************************************************************************
Transient iteration: 2 Time: 0.002 Time step: 0.001 CFL: 7.47157e-05
Transient iteration: 2 Time: 0.002 Time step: 0.001 CFL: 7.48857e-05
**********************************************************************************
--------------
Void Fraction
Expand All @@ -50,11 +50,11 @@ DEM
DEM contact search at dem step 1
Finished 50 DEM iterations
---------------------------------------------------------------
Global continuity equation error: -7.56756e-09 s^-1
Max local continuity error: 3.10857e-06 s^-1
Global continuity equation error: -1.26904e-09 s^-1
Max local continuity error: 7.28588e-07 s^-1

**********************************************************************************
Transient iteration: 3 Time: 0.003 Time step: 0.001 CFL: 0.000162479
Transient iteration: 3 Time: 0.003 Time step: 0.001 CFL: 0.000149645
**********************************************************************************
--------------
Void Fraction
Expand All @@ -68,11 +68,11 @@ DEM
DEM contact search at dem step 1
Finished 50 DEM iterations
---------------------------------------------------------------
Global continuity equation error: 1.94709e-09 s^-1
Max local continuity error: 5.08953e-07 s^-1
Global continuity equation error: 3.9841e-09 s^-1
Max local continuity error: 1.05602e-06 s^-1

**********************************************************************************
Transient iteration: 4 Time: 0.004 Time step: 0.001 CFL: 0.000224045
Transient iteration: 4 Time: 0.004 Time step: 0.001 CFL: 0.000224447
**********************************************************************************
--------------
Void Fraction
Expand All @@ -86,12 +86,12 @@ DEM
DEM contact search at dem step 1
Finished 50 DEM iterations
---------------------------------------------------------------
Global continuity equation error: 7.6613e-09 s^-1
Max local continuity error: 5.17523e-07 s^-1
Global continuity equation error: 1.16336e-09 s^-1
Max local continuity error: 1.41713e-06 s^-1

*********************************************************************************
Transient iteration: 5 Time: 0.005 Time step: 0.001 CFL: 0.00029829
*********************************************************************************
**********************************************************************************
Transient iteration: 5 Time: 0.005 Time step: 0.001 CFL: 0.000299344
**********************************************************************************
--------------
Void Fraction
--------------
Expand All @@ -107,7 +107,7 @@ print_from_processor_0
0 1
0
1 8
3 3 3 3 3 3 3 3
4 4 4 4 4 4 4 4


[deal.II intermediate Patch<3,3>]
Expand All @@ -117,7 +117,7 @@ print_from_processor_0
1 1
0
1 8
2 2 2 2 2 2 2 2
4 4 4 4 4 4 4 4


[deal.II intermediate Patch<3,3>]
Expand All @@ -127,7 +127,7 @@ print_from_processor_0
2 1
0
1 8
2 2 2 2 2 2 2 2
4 4 4 4 4 4 4 4


[deal.II intermediate Patch<3,3>]
Expand All @@ -137,7 +137,7 @@ print_from_processor_0
3 1
0
1 8
2 2 2 2 2 2 2 2
4 4 4 4 4 4 4 4


[deal.II intermediate Patch<3,3>]
Expand All @@ -147,7 +147,7 @@ print_from_processor_0
4 1
0
1 8
3 3 3 3 3 3 3 3
4 4 4 4 4 4 4 4


[deal.II intermediate Patch<3,3>]
Expand Down Expand Up @@ -188,5 +188,5 @@ DEM
DEM contact search at dem step 1
Finished 50 DEM iterations
---------------------------------------------------------------
Global continuity equation error: 6.76463e-10 s^-1
Max local continuity error: 6.79928e-07 s^-1
Global continuity equation error: -8.07686e-10 s^-1
Max local continuity error: 1.81029e-06 s^-1
2 changes: 1 addition & 1 deletion applications_tests/lethe-fluid-particles/adaptive_sparse_contacts_files/dem.particles
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
0 0 350 74 350
0 0 350 78 350
2 changes: 1 addition & 1 deletion applications_tests/lethe-fluid-particles/adaptive_sparse_contacts_files/dem.pvdhandler
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
0
Time File
Time File
Binary file modified applications_tests/lethe-fluid-particles/adaptive_sparse_contacts_files/dem.triangulation
View file
Open in desktop
Binary file not shown.
Binary file modified ...s_tests/lethe-fluid-particles/adaptive_sparse_contacts_files/dem.triangulation_fixed.data
View file
Open in desktop
Binary file not shown.
Binary file modified ...ests/lethe-fluid-particles/adaptive_sparse_contacts_files/dem.triangulation_variable.data
View file
Open in desktop
Binary file not shown.
Loading
Loading