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

Uses the array data structure for ParticleField and pullbacks for AD #13

Open
wants to merge 263 commits into
base: master
Choose a base branch
from
Open

Uses the array data structure for ParticleField and pullbacks for AD #13

wants to merge 263 commits into from

Conversation

cibinjoseph
Copy link
Member

This PR is to review the changes made and make sure everything looks good. This may not need to be merged into master immediately.

rymanderson and others added 30 commits November 9, 2023 09:35
@rymanderson
prep new benchmark
189b009
@rymanderson
passing unit tests
40f856f
@rymanderson
make expansion_order consistent and prep benchmark
b31c96f
@rymanderson
prep benchmark with fmm
1caa44d
@rymanderson
specify julia-1.6
78bf00b
@rymanderson
allow changing pfield.Uinf by using less restrictive type
56d5d34
@rymanderson
bug fix
c106932
@rymanderson
remove duplicate eltype definition
69bcba8
@rymanderson
add nonzero_sigma parameter
2c8f93d
@rymanderson
change nonzero_sigma default =false
8b6271e
@rymanderson
initialize particle field with custom float type
d9b28e7
@rymanderson
merge conflict on workstation
1101eb6
@rymanderson
ForwardDiff compatibility
c163a52
@rymanderson
add custom_UJ function
ceeaac6
@rymanderson
working with panels
ad44787
@rymanderson
fix viscous and SFS syntax error
df9de1a
@rymanderson
update fields explicitly in remove_particle
4fdbf47
@rymanderson
toggle saving particle field
2ec569a
@rymanderson
update fmm syntax
73c6d13
@rymanderson
add custom_UJ function
07da72c
@rymanderson
working with panels
712650f
@rymanderson
fix viscous and SFS syntax error
99ef2f7
@rymanderson
update fmm syntax
4acd689
@rymanderson
merge conflict
9b355c8
@cibinjoseph
Remove unused tests
29360c1
@cibinjoseph
Take out vol
0ee4bef
@cibinjoseph
Remove field sigma
03f8065
@cibinjoseph
Comment out field vol
710eb00
@cibinjoseph
Remove field circulation
859bfba
@cibinjoseph
Remove field Gamma
27415bc
rymanderson and others added 30 commits October 16, 2024 09:09
@rymanderson
add relative error option to fmm
99f8f2e
@cibinjoseph
Single gpu streams kernel
b21015f
@cibinjoseph
Add .case to end of folder name
0e6c9d4
@cibinjoseph
Add .case files to ignore
56b0caa
@cibinjoseph
Remove trial kernel
6038fb2
@cibinjoseph
Add check_shared_memory()
59c5839
@cibinjoseph
Add multiple streams multiple gpu kernel
e73c936
@rymanderson
update FastMultipole gpu syntax
65a3a3f
@cibinjoseph
Correct error in shared memory check function
2fb4c5b
@cibinjoseph
Change to in-place operation for interaction
9670101
Remove debugging override
f7be9ad
Correct igpu to istream
fa4e4e7
@cibinjoseph
Changes to gpu kernel
b7d22a2
@rymanderson
update derivativesSwitch
f4d4075
@cibinjoseph
Add nearfield_device!() for gpu direct computation
0bdfede
@cibinjoseph
Add gpu kernel changes from Cibin
b2122c4
@cibinjoseph
Correct sort function name
a6ba8de
@cibinjoseph
Correct mistake in expand_source_indices()
4914d85
@cibinjoseph
Add documentation for expand_source_indices()
7b69593
@cibinjoseph
Add multiple gpus and multiple streams to nearfield_device!()
0fbb5a4
Add gpu kernel for multiple gpus and streams
da56a2b
@cibinjoseph
Add multithreading for UJ_direct() over targets
bb63c7c
@cibinjoseph
Use shared memory function check
9ad9c0c
@cibinjoseph
Correct obsolete function add_SFS
13c7c0f
@cibinjoseph
Remove warmup_gpu temporarily
f7d2fc8
@cibinjoseph
Add fully_direct case
f154a74
@cibinjoseph
Use UJ_d as a separate output to prevent redundat copying
253c6ff
@cibinjoseph
Benchmark leapfrog case
6406846
Use 512 as max threads per block
ead8e76
Correct max threads per block
ecedbc5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rymanderson rymanderson rymanderson left review comments

@Eric-P-Green Eric-P-Green Awaiting requested review from Eric-P-Green

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.
4 participants
@cibinjoseph @rymanderson @Eric-P-Green @BTV25