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

Fix bug in alignment tests and remove rootclang warnings. #292

Merged
merged 1 commit into from Jan 24, 2018

Conversation

Projects
None yet
2 participants
@MarkusFrankATcernch
Contributor

MarkusFrankATcernch commented Jan 24, 2018

BEGINRELEASENOTES

  • Fix bug in Alignment test:

    • A long standing bug was fixed for the test t_AlignDet_Telescope_align_new.
    • It was assumed that the DetElements were ordered in memory, which is not true (probably due to memory randomization).
    • Moved to path dependent ordering. Test now OK.
    • Closes #282
  • Removed a Bunch of rootcling warnings

    • Apparently rootcling now expects a class with the name of the dictionary header file.
    • To satify this some dummy anonymous classes were added.
    • Closes #290

ENDRELEASENOTES

@MarkusFrankATcernch MarkusFrankATcernch merged commit 59cc956 into AIDASoft:master Jan 24, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@MarkusFrankATcernch MarkusFrankATcernch changed the title from Remove rootclang warnings. to Fix bug in Alignment Tests and remove rootclang warnings. Jan 24, 2018

@MarkusFrankATcernch MarkusFrankATcernch changed the title from Fix bug in Alignment Tests and remove rootclang warnings. to Fix bug in alignment tests and remove rootclang warnings. Jan 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment