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

Jump to bottom

Xiangyu/revise bidirectional buffer #658

Merged
merged 299 commits into from
Sep 16, 2024
Merged

Xiangyu/revise bidirectional buffer #658

merged 299 commits into from
Sep 16, 2024

Conversation

Xiangyu-Hu
Copy link
Owner

@Xiangyu-Hu Xiangyu-Hu commented Sep 15, 2024 •
edited
Loading

  1. introduced body part index so that one can identify different buffer regions.
  2. do particle deletion within the bi-directional buffer class with checking on body part index.
  3. add comment that we need first inject for all buffers then do deletion for them.
  4. found the mixing between injection and deletion in parallel is dangerous.

@Xiangyu-Hu
move opt to cmake
7d7e515
@Xiangyu-Hu
full version cmake options
bb77ff5
@Xiangyu-Hu
incremental change
e9ce48b
@Xiangyu-Hu
move options
cfacace
@Xiangyu-Hu
revert back
454d1cb
@Xiangyu-Hu
only test sycl cases
0c970b4
@Xiangyu-Hu
use apt install
ddb2b7f
@Xiangyu-Hu
list the libs
39da6f9
@Xiangyu-Hu
add opencl runtime
b7f57c6
@Xiangyu-Hu
check if runtime installed
598ae86
@Xiangyu-Hu
set environment again
a623b74
@Xiangyu-Hu
folder name tests_sycl
eeb56f2
@Xiangyu-Hu
set sycl env for each step
edd78a7
@Xiangyu-Hu
put back to ci pull request
ddef538
@Xiangyu-Hu
Merge branch 'debug/fan' into xiangyu/sycl_redo
70002bb
@Xiangyu-Hu
Merge branch 'master' into xiangyu/sycl_index_type
5d15f9c
@Xiangyu-Hu
all compiled
b4cf7c6
@Xiangyu-Hu
introduce type Index
30a5421
@Xiangyu-Hu
Merge branch 'xiangyu/sycl_redo' into xiangyu/sycl_index_type
2db61bc
@Xiangyu-Hu
Merge branch 'refactory/use_triangle_mesh_distace_library' into xiang…
a48f09d 
…yu/sycl_index_type
@Xiangyu-Hu
prepare execution classes
479a3b3
@Xiangyu-Hu
Use SizeT
76af971
@Xiangyu-Hu
Merge branch 'fix/generalize_2_3d' into xiangyu/sycl_index_type
efb7b88
@Xiangyu-Hu
merge change
f2a0785
@Xiangyu-Hu
small change
69ffbb7
@Xiangyu-Hu
Merge branch 'fix/generalize_2_3d' into xiangyu/sycl_index_type
4d22bf6
@Xiangyu-Hu
Merge branch 'refactory/size_t_registered_too' into xiangyu/solve_mer…
9dce48f 
…ge_issue
@Xiangyu-Hu
briefly use UnsignedInt for IDs
a931087
@Xiangyu-Hu
begin
c67fa3f
@Xiangyu-Hu
Merge branch 'xiangyu/sycl_redo' into xiangyu/split_particle_sort
2c0b58b
@Xiangyu-Hu
a bug on particle reload
174dd44
@Xiangyu-Hu
bug on add variable to sort
339b36c
@Xiangyu-Hu
mesh cell list do not have member of particle position
d42ff8d
@Xiangyu-Hu
comment
4342d75
@Xiangyu-Hu
begin
4a75bcf
@Xiangyu-Hu
base variable changed to entity
9fa45c7
@Xiangyu-Hu
change file layout
abad4ee
@Xiangyu-Hu
master and stable branch
29504ed
@Xiangyu-Hu
Merge branch 'master' into xiangyu/sycl_redo
7815f9a
@Xiangyu-Hu
to check a bug
7261d32
@Xiangyu-Hu
to check locally
89345bc
@Xiangyu-Hu
bug is the that update cell linked list is forgotten
2057100
@Xiangyu-Hu
Merge branch 'xiangyu/body_part_index' into xiangyu/revise_bidirectio…
165d596 
…nal_buffer
@Xiangyu-Hu
to check locally
e7b25d8
@Xiangyu-Hu
cmake preset ignored
108a4c4
@Xiangyu-Hu
naming
7c1d141
@Xiangyu-Hu
change other cases
b8e3204
@Xiangyu-Hu
to text
4f60ccf
@Xiangyu-Hu
to test locally
6c08cb6
@Xiangyu-Hu
to check locally
0a52e43
@Xiangyu-Hu
bug in mix poiseuille flow found, the inflow velocity define in oppos…
1483181 
…ite direction
@Xiangyu-Hu
first do injection for all buffers, then do deletion for all buffers
0f76628
@Xiangyu-Hu
Merge branch 'xiangyu/revise_bidirectional_buffer' of https://github.…
c89d1e9 
…com/Xiangyu-Hu/SPHinXsys into xiangyu/revise_bidirectional_buffer
@Xiangyu-Hu
passed the test cases on pressure boundary condition
18320c9
@Xiangyu-Hu Xiangyu-Hu merged commit 29a4c04 into master Sep 16, 2024
7 checks passed
@Xiangyu-Hu Xiangyu-Hu deleted the xiangyu/revise_bidirectional_buffer branch September 16, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FengWang3119 FengWang3119 FengWang3119 approved these changes

@ShuoguoZhangTUM ShuoguoZhangTUM Awaiting requested review from ShuoguoZhangTUM

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Xiangyu-Hu @FengWang3119