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 upgrades motivated by TRestAxionSolarFlux implementations #184

Merged
merged 8 commits into from
May 9, 2022

Conversation

jgalan
Copy link
Member

@jgalan jgalan commented Apr 21, 2022

jgalan Ok: 73

Adds few methods that are needed at rest-for-physics/axionlib new implementations

  • TRestTools::GetFileNameRoot to get the filename without extension and without path.

  • TRestTools::TransposeTable it changes rows by columns in a std::vector<std::vector table.

  • TRestMetadata will now search for the RML in case it is not found at the path we have given in the constructor. For example, if now I do TRestAxionSolarFlux("fluxes.rml") it will get the fluxes.rml file found at the data path. This is handy so that input data and definitions are found in a single place, so there is no need to have the RML copied somewhere else.
    https://github.com/rest-for-physics/axionlib-data/tree/master/solarFlux

@jgalan jgalan changed the title Jgalan solar flux Few TRestTools upgrades motivated by TRestAxionSolarFlux implementations Apr 21, 2022
@jgalan jgalan requested a review from a team May 5, 2022 06:33
@jgalan jgalan requested review from juanangp, lobis and nkx111 May 5, 2022 06:33
@jgalan jgalan marked this pull request as ready for review May 5, 2022 06:33
jgalan and others added 2 commits May 6, 2022 09:35
Co-authored-by: Juan Antonio García <80903717+juanangp@users.noreply.github.com>
@jgalan
Copy link
Member Author

jgalan commented May 9, 2022

Plz, aprove

@jgalan jgalan requested a review from a team May 9, 2022 13:11
@jgalan jgalan merged commit 88432f5 into master May 9, 2022
@jgalan jgalan deleted the jgalan_solar_flux branch May 9, 2022 16:41
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.

None yet

4 participants