Skip to content

Feature/hkpilot#24

Closed
guiguem wants to merge 15 commits intoHKDAQ:masterfrom
guiguem:feature/hkpilot
Closed

Feature/hkpilot#24
guiguem wants to merge 15 commits intoHKDAQ:masterfrom
guiguem:feature/hkpilot

Conversation

@guiguem
Copy link
Contributor

@guiguem guiguem commented Jun 7, 2023

No description provided.

JostMigenda and others added 15 commits December 22, 2020 22:19
To avoid confusion, since fNEff2 is unrelated to fDarkRate2 or fNPMTs2.
Also includes changes to whitespace and comments.
as suggested by @gpronost
* Add function to overwrite neighbour tree + quiet mode to minimize debug

* Replace Quiet mode by negative verbosity

* Replace if ( fVerbosity ) by if ( fVerbosity > 0 )

* Replace if ( fVerbosity ) by if ( fVerbosity > 0 )

* Remove Overwrite function, fix missing verbosity flag

* Update WCSimFLOWER.h
* Add function to overwrite neighbour tree + quiet mode to minimize debug

* Replace Quiet mode by negative verbosity

* Replace if ( fVerbosity ) by if ( fVerbosity > 0 )

* Replace if ( fVerbosity ) by if ( fVerbosity > 0 )

* Remove Overwrite function, fix missing verbosity flag

* Update WCSimFLOWER.h

* Add configurations used for MC Production

* Correct Neighbour distance for Hybrid 40pc

* Remove debug
* Update mPMT numbers after PMT mask applied

* Add full support for PMT mask

* One nPMT_nomask was missing

* Check PMT Mask was not correctly done (need TubeID)

* Debug GetNearestNeighbours with Verbose is set

* Solve issue in GetNearestNeighbours
* Add ROOT File close() in order to prevent some (rare) cases of ROOT File corruption

* Add case for hybrid geometry in CorrectEnergy function
@JostMigenda JostMigenda changed the base branch from master to JostMigenda/mPMT February 26, 2024 15:58
@JostMigenda JostMigenda changed the base branch from JostMigenda/mPMT to master February 26, 2024 16:59
@JostMigenda
Copy link
Collaborator

Merged the CI/Docker changes in #26; the mPMT-related changes are covered by #14.

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