Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

[FOCAL-98] Optimizing the hit processing in FOCAL#13297

Merged
mfasDa merged 1 commit intoAliceO2Group:devAliceO2Group/AliceO2:devfrom
hahassan7:devhahassan7/AliceO2:devCopy head branch name to clipboard
Jul 19, 2024
Merged

[FOCAL-98] Optimizing the hit processing in FOCAL#13297
mfasDa merged 1 commit intoAliceO2Group:devAliceO2Group/AliceO2:devfrom
hahassan7:devhahassan7/AliceO2:devCopy head branch name to clipboard

Conversation

@hahassan7
Copy link
Contributor

Optimizing the hit processing in the FOCAL detector by using hash ID and inverse std::unordered_map instead of the linear for the hit which can be exhausting.

@github-actions
Copy link
Contributor

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass3
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0

@alibuild
Copy link
Collaborator

Error while checking build/O2/fullCI for 11cd542 at 2024-07-14 11:36:

## sw/BUILD/O2-latest/log
c++: error: unrecognized command-line option '--rtlib=compiler-rt'
c++: error: unrecognized command-line option '--rtlib=compiler-rt'

Full log here.

Copy link
Collaborator

@mfasDa mfasDa left a comment

Choose a reason for hiding this comment

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

Only small naming change, otherwise ready to merge

mfasDa
mfasDa previously approved these changes Jul 19, 2024
Copy link
Collaborator

@mfasDa mfasDa left a comment

Choose a reason for hiding this comment

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

Looks good

Use std::unordered_map with hit key based on hit coordinates in order to have fast hash-based hit search.
@mfasDa mfasDa merged commit 66ae06b into AliceO2Group:dev Jul 19, 2024
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.