Skip to content

Comments

PID: Merge split tasks into one configurable#5475

Merged
jgrosseo merged 1 commit intoAliceO2Group:devfrom
njacazio:nj-o2PID-full-tables
Feb 15, 2021
Merged

PID: Merge split tasks into one configurable#5475
jgrosseo merged 1 commit intoAliceO2Group:devfrom
njacazio:nj-o2PID-full-tables

Conversation

@njacazio
Copy link
Collaborator

This PR creates split tables for only required particle hypothesis, in a single task.
In addition:
- This commit create all split tables for TPC and TOF PID response in the split tasks
- The filling of the split PID tables is done based on the configuration recieved by the task
- Empty tables are sent for mass hypotheses that are not requested
- The creation of several tasks is now avoided because of the large memory consumption overhead that was shown in the past
- Uniformized PID tasks

@njacazio
Copy link
Collaborator Author

@jgrosseo as we discussed on Friday, this implements the configurable PID tables in a single task

- This commit create all split tables for TPC and TOF PID response in the split tasks
- The filling of the split PID tables is done based on the configuration recieved by the task
- Empty tables are sent for mass hypotheses that are not requested
- The creation of several tasks is now avoided because of the large memory consumption overhead that was shown in the past
- Uniformized PID tasks
@njacazio njacazio force-pushed the nj-o2PID-full-tables branch from 0021321 to 7dd3a17 Compare February 15, 2021 17:17
@jgrosseo
Copy link
Collaborator

Unrelated. Merging.

@jgrosseo jgrosseo merged commit 6f41c51 into AliceO2Group:dev Feb 15, 2021
EmilGorm pushed a commit to EmilGorm/AliceO2 that referenced this pull request Nov 22, 2021
- This commit create all split tables for TPC and TOF PID response in the split tasks
- The filling of the split PID tables is done based on the configuration recieved by the task
- Empty tables are sent for mass hypotheses that are not requested
- The creation of several tasks is now avoided because of the large memory consumption overhead that was shown in the past
- Uniformized PID tasks
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