Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add environment for the VTXdigi test #14

Closed
wants to merge 1 commit into from

Conversation

jmcarcell
Copy link
Member

This is an example of what it has to be done for fixing issues with CTest not finding VTXdigitizer, for example. It's copied and adapted from k4FWCore, so probably all the paths are not needed. After doing this it runs but I get a different error later:

ToolSvc.SimG4Sa...   INFO Hits from readout "VTXIBCollection" will be saved in the collection "VTXIB_simTrackerHits".
ToolSvc.SimG4Sa...   INFO Hits from readout "VTXOBCollection" will be saved in the collection "VTXOB_simTrackerHits".
ToolSvc.SimG4Sa...   INFO Hits from readout "VTXDCollection" will be saved in the collection "VTXD_simTrackerHits".
ToolSvc.SimG4Sa...  ERROR Readout  'readoutName':'SiWrBCollection' not found! Please check tool configuration.  Exiting...
ToolSvc             ERROR Error initializing tool 'ToolSvc.SimG4SaveTrackerHitsSiWrB'
SimG4Alg            ERROR Unable to retrieve the output saving tool PublicToolHandle('SimG4SaveTrackerHits/SimG4SaveTrackerHitsSiWrB')
EventLoopMgr        ERROR Unable to initialize Algorithm: SimG4Alg
ServiceManager      ERROR Unable to initialize Service: EventLoopMgr
ApplicationMgr      ERROR Application Manager Terminated with error code 1

Consider having all the tests under /test instead of having them each in a different folder; the current approach means the environment will have to be copied in each subfolder while it could be in a single place.

@BrieucF
Copy link
Collaborator

BrieucF commented Feb 7, 2024

Thanks Juan! Yes I will also fix the subsequent expected error.

@armin-ilg
Copy link
Collaborator

The vertex code is using key4hep/k4geo#314 where the Silicon wrapper is integrated as well in IDEA, hence it's failing. Shall I comment out all silicon wrapper references?

@BrieucF
Copy link
Collaborator

BrieucF commented Feb 7, 2024

No worries Armin, I'll do this together with the other test fixing

@BrieucF
Copy link
Collaborator

BrieucF commented Feb 9, 2024

Closing, superseded by https://github.com/BrieucF/k4RecTracker/pull/15 . Thanks for the help @jmcarcell !

@BrieucF BrieucF closed this Feb 9, 2024
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.

3 participants