Skip to content

Add FilterTracks processor to filter tracks.#10

Merged
gianelle merged 3 commits intoMuonColliderSoft:masterfrom
kkrizka:FilterTracks
Feb 28, 2023
Merged

Add FilterTracks processor to filter tracks.#10
gianelle merged 3 commits intoMuonColliderSoft:masterfrom
kkrizka:FilterTracks

Conversation

@kkrizka
Copy link
Copy Markdown

@kkrizka kkrizka commented Dec 7, 2022

Originally written by @njbruhwiler.

A processor that filters a track collection. The input and output collections are set via InputTrackCollectionName and OutputTrackCollectionName parameters.

The available track properties to filter on are:

  • BarrelOnly: Keep tracks with only barrel hits.
  • NHitsTotal: Minimum number of hits on track
  • NHitsVertex: Minimum number of hits on vertex detector
  • NHitsInner: Minimum number of hits on inner tracker
  • NHitsOuter: Minimum number of hits on outer tracker
  • MinPt: Minimum transverse momentum
  • Chi2Spatial: Spatial chi squared

gianelle added a commit that referenced this pull request Jan 25, 2023
Merge remote-tracking branch 'karolrepo/FilterTracks' into v02-08-MC
@gianelle
Copy link
Copy Markdown

gianelle commented Jan 26, 2023

Hi,
is it possible to use a more generic approach likes the one used in the RefitFinal, which accepts cuts for hits using something like:
< detID >, < detI D>,... < lower threshold >

For example:

  • to require n hits on vertex you writes: 1,2 n
  • if you want n hits on inner + vertex barrel writes: 1,3,4 n

A second request is if it is possible to put the "hits requests" in OR condition instead of AND, this because you can
always create AND condition calling the filter more times.

With these two changes we can remove the filter from the RefitFinal processor, using this one instead (which seems a
more correct approach) .

Ale

@gianelle gianelle merged commit 618722f into MuonColliderSoft:master Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants