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

Addons required by XMM Wolter optics integration inside axionlib #235

Merged
merged 43 commits into from
Jul 6, 2022

Conversation

jgalan
Copy link
Member

@jgalan jgalan commented May 30, 2022

jgalan Large: 629

It adds few add-ons required by axionlib optics implementation.

  • TRestCombinedMask implements now a combination of TRestPatternMask. It has been fully documented.
  • TRestTools::GetColumnFromTable method added.
  • TRestPhysics::GetConeNormal, TRestPhysics::GetVectorReflection and TRestPhysics::GetVectorsAngle implemented.
  • TRestStringHelper::Double/IntegerToString now accepts an additional optional argument with the number format.
  • restRoot now includes automatically TRestSystemOfUnits so that it recognizes the units constructor.

Also masks have been placed at a dedicated framework/masks directory.

jgalan added 28 commits May 30, 2022 21:42
@jgalan jgalan marked this pull request as ready for review June 15, 2022 08:24
@jgalan jgalan marked this pull request as draft June 15, 2022 08:24
@jgalan jgalan marked this pull request as ready for review June 26, 2022 14:46
@jgalan jgalan requested a review from nkx111 June 26, 2022 14:46
@jgalan jgalan requested a review from juanangp June 26, 2022 14:47
@jgalan jgalan requested review from lobis and a team June 26, 2022 14:47
@jgalan jgalan merged commit 95abbaf into master Jul 6, 2022
@jgalan jgalan deleted the xmm_optics branch July 6, 2022 09:39
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