acts-project / acts

Experiment-independent toolkit for (charged) particle track reconstruction in (high energy) physics experiments implemented in modern C++
https://acts.readthedocs.io
Mozilla Public License 2.0
104 stars 168 forks source link

Debug Information in Acts Seeder #944

Closed osbornjd closed 5 days ago

osbornjd commented 3 years ago

The current implementation of the Acts seeding algorithm has no debug or verbose statements that could be turned on with the Acts logging functionality. It would be useful, for e.g. debugging when no seeds are found, if there were some options for understanding what the seeding algorithm is doing with the input that is provided to it, e.g. with some logging statements.

stale[bot] commented 3 years ago

This issue/PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 3 years ago

This issue/PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

AJPfleger commented 6 months ago

As discussed in today's developers meeting, @osbornjd might have a quick look on the current status and then we will decide on how to proceed with this.

AJPfleger commented 6 months ago

It seems, the Seeder still lacks a lot of debugging print outs. Let's keep this PR open for now and have the next person working on the seeder pick it up again.

General outcome from the discussion: Add more logging-statement, that help to better understand, what currently is/was happening. If this becomes performance relevant, we could also think about a compiler-flag, that removes loggers below a certain level.

github-actions[bot] commented 4 months ago

This issue/PR has been automatically marked as stale because it has not had recent activity. The stale label will be removed if any interaction occurs.