Skip to content

Its-gpu: add tracklet finding and cell finding on GPU (HIP+NVIDIA) - #9289

Merged
davidrohr merged 30 commits into
AliceO2Group:devfrom
mconcas:its-gpu-tracking
Jul 29, 2022
Merged

Its-gpu: add tracklet finding and cell finding on GPU (HIP+NVIDIA)#9289
davidrohr merged 30 commits into
AliceO2Group:devfrom
mconcas:its-gpu-tracking

Conversation

@mconcas

@mconcas mconcas commented Jun 29, 2022

Copy link
Copy Markdown
Collaborator

To do before merging:

  • Fix: [ERROR] Exception caught: std::bad_alloc: hipErrorOutOfMemory: hipErrorOutOfMemory error running on HIP
  • Fix: eternally stuck at thrust::unique() call in hip -> does not occur on MI50
  • Remove the unneeded adaptations to CPU code
  • Validate MC labels creation
  • Thrust duplicate removal
  • Add Cell finding on GPU
  • Fix Cell finding on HIP
  • Refactorize passing parameters to kernels
  • Fix full gpu tracking

@mconcas mconcas changed the title (WIP) its-gpu: port updated ITS tracklet finder on GPU (WIP) its-gpu: port updated ITS tracklet finder to GPU Jun 29, 2022
@mconcas
mconcas force-pushed the its-gpu-tracking branch from 2f06af0 to 47357d6 Compare July 3, 2022 09:59
@mconcas mconcas changed the title (WIP) its-gpu: port updated ITS tracklet finder to GPU (WIP) its-gpu: add tracklet finding and cell finding on GPU (HIP+NVIDIA) Jul 3, 2022
@mconcas
mconcas force-pushed the its-gpu-tracking branch 4 times, most recently from 123d619 to b38dd07 Compare July 8, 2022 20:18
@mconcas mconcas changed the title (WIP) its-gpu: add tracklet finding and cell finding on GPU (HIP+NVIDIA) Its-gpu: add tracklet finding and cell finding on GPU (HIP+NVIDIA) Jul 8, 2022
@mconcas
mconcas marked this pull request as ready for review July 8, 2022 20:20
@mconcas

mconcas commented Jul 8, 2022

Copy link
Copy Markdown
Collaborator Author

@davidrohr: this is finally ready for review

@davidrohr

Copy link
Copy Markdown
Collaborator

ok, will have a look, but I'll postpone merging until after the EPN SW update on Wednesday, in order not to pull in more large changes.

@mconcas

mconcas commented Jul 9, 2022

Copy link
Copy Markdown
Collaborator Author

ok, will have a look, but I'll postpone merging until after the EPN SW update on Wednesday, in order not to pull in more large changes.

Sure, I'd also wait for #9374 as per @shahor02 request.

@mconcas
mconcas requested a review from a team as a code owner July 9, 2022 09:23
@mconcas
mconcas force-pushed the its-gpu-tracking branch from 165d8d5 to eb0209d Compare July 10, 2022 17:17
@mconcas
mconcas force-pushed the its-gpu-tracking branch 5 times, most recently from 0a0e3b9 to 4e61bc8 Compare July 17, 2022 17:57
@mconcas

mconcas commented Jul 18, 2022

Copy link
Copy Markdown
Collaborator Author

Hi @TimoWilken, some tests seem not to have no space left on device, just for your knowledge.

@mconcas
mconcas force-pushed the its-gpu-tracking branch 2 times, most recently from 3e508e0 to 118536f Compare July 23, 2022 13:59
@davidrohr

Copy link
Copy Markdown
Collaborator

@mconcas : I merged the other PR. Could you rebase this one again :)

@mconcas

mconcas commented Jul 26, 2022

Copy link
Copy Markdown
Collaborator Author

@mconcas : I merged the other PR. Could you rebase this one again :)

thanks, yep I'm doing it

@mconcas
mconcas force-pushed the its-gpu-tracking branch from dc77e2c to f07cc5b Compare July 28, 2022 15:50
@mconcas

mconcas commented Jul 28, 2022

Copy link
Copy Markdown
Collaborator Author

@davidrohr ok, this has been rebased. It can be merged upon green CI.

CC-ing @mpuccio

@davidrohr

Copy link
Copy Markdown
Collaborator

Compiles and runs fine with me, will merge later today.

@davidrohr
davidrohr merged commit b716b77 into AliceO2Group:dev Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants