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

Incorporate type fix from https://github.com/AIDASoft/DD4hep/pull/1172. Improve checksum debugging possibilities. #1201

Merged
merged 5 commits into from
Dec 14, 2023

Conversation

MarkusFrankATcernch
Copy link
Contributor

BEGINRELEASENOTES

  • Incorporate type fix from DetectorChecksum.cpp: fix a typo #1172
  • Propagate polish setting from ROOT surfaces to Geant4
  • Improve debugging capabilities of detector checksums by improved dumping possibilities.
    ENDRELEASENOTES

Copy link

github-actions bot commented Dec 13, 2023

Test Results

       6 files         6 suites   6h 49m 53s ⏱️
   356 tests    356 ✔️ 0 💤 0
1 058 runs  1 058 ✔️ 0 💤 0

Results for commit 3cbc7dd.

♻️ This comment has been updated with latest results.

@andresailer
Copy link
Member

The g4units is probably systematic. I fixed it by changing the stepper in the config
https://github.com/AIDASoft/DD4hep/pull/1196/files#diff-c10dfc19972ca561e312795821c6b5f9e062b72a2edf60a414ae66fe5f0145b6

But I am wondering if some property is not correctly converted unit wise.

@MarkusFrankATcernch
Copy link
Contributor Author

@andresailer Hmm. Possibly. I checked my changes and I think they should not interfere with the changes you made before. If you merge your PR, I could update and rerun the tests.....

@MarkusFrankATcernch MarkusFrankATcernch enabled auto-merge (rebase) December 14, 2023 13:20
@MarkusFrankATcernch MarkusFrankATcernch merged commit 3a6b9ff into AIDASoft:master Dec 14, 2023
13 checks passed
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