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

Few TRestTools/TRestPhysics/TRestStringHelper upgrades motivated by TRestAxionOpticsMirror #175

Merged
merged 21 commits into from
Apr 22, 2022

Conversation

jgalan
Copy link
Member

@jgalan jgalan commented Apr 4, 2022

jgalan Large: 299

Few tools added and/or reviewed which are required by recent implementation at rest-for-physics/axionlib#22

  • TRestTools::ExportBinaryTable(fname,data) has been added. It will write a binary file with name fname with the contents of std::vector<std::vector <double/float/int>> &data.

  • TRestTools::GetMin/MaxValueFromTable now operates over the full table if no column is specified.

  • TRestStringHelper::StringToElements prototyping has been simplified, and method is now overloaded. Usage of new implemented method StringToElements( string in, "[", ",", "]" ) where the method will find the first string coincidence of "[" and extract the elements from the given format.

  • TRestStringHelper::DoubleToString it takes a double number and returns it as a string.

  • TRestPhysics::GetConeVectorIntersection added to find the intersection of a straight with a cone.

  • TRestTools::GetFileNameExtension added to facilitate retrieving the extension of a filename.

  • TRestTools::IsBinaryFile and TRestTools::GetBinaryFileColumns methods added.

  • TRestTools::ReadBinaryTable will attempt to read the number of columns from N000i/f/d filename extension format.

@rest-for-physics/core_dev

@jgalan jgalan requested review from KonradAltenmueller and a team April 4, 2022 09:08
@jgalan jgalan changed the title Jgalan mirror Methods added or reviewed to be used in recent axionlib implementation. Apr 4, 2022
@jgalan jgalan requested review from jovoy and ruzarmen April 4, 2022 09:10
@jgalan jgalan marked this pull request as draft April 5, 2022 10:17
@jgalan jgalan marked this pull request as ready for review April 8, 2022 08:44
@jgalan jgalan requested a review from a team April 8, 2022 09:54
@jgalan jgalan requested review from nkx111 and removed request for jovoy and ruzarmen April 13, 2022 15:15
@jgalan jgalan changed the title Methods added or reviewed to be used in recent axionlib implementation. Few TRestTools/TRestPhysics/TRestStringHelper upgrades motivated by TRestAxionOpticsMirror Apr 21, 2022
@jgalan jgalan merged commit 1627c07 into master Apr 22, 2022
@lobis lobis deleted the jgalan_mirror branch June 30, 2022 09:25
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