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

Script to compile only the tools you need #325

Open
wants to merge 5 commits into
base: Application
Choose a base branch
from

Conversation

marvlad
Copy link
Contributor

@marvlad marvlad commented Dec 16, 2024

Describe your changes

I am adding a bash script in script/ dir to compile in a ToolAnalysis/build_[name_here] directory only the needed tools. This speeds up the compilation time from 40 mins to ~3 minutes depending the tool. Additionally, the .gitignore was modify so the build_* dir doesn't interfere later.

The script after chmod +x ToolAnalysisMyTools.sh in scrip can be run it as scrip/./ToolAnalysisMyTools.sh ConfigfileName or scrip./ToolAnalysisMyTools.sh "ConfigfileName1,ConfigfileName2,..." or scrip./ToolAnalysisMyTools.sh ConfigfileName clean. Allowing single Tool, or multiple tools.

Checklist before submitting your PR

  • This PR implements a single change (one new/modified Tool, or a set of changes to implement one new/modified feature)
  • This PR alters the minimum number of files to affect this change
  • If this PR includes a new Tool, a README and minimal demonstration ToolChain is provided
  • If a new Tool/ToolChain requires model or configuration files, their paths are not hard-coded, and means of generating those files is described in the readme, with examples provided on /pnfs/annie/persistent
  • For every new usage, there is a reason the data must be on the heap
  • For every new there is a delete, unless I explicitly know why (e.g. ROOT or a BoostStore takes ownership)

Additional Material

Attach any validation or demonstration files here. You may also link to relavant docdb articles.

@marvlad
Copy link
Contributor Author

marvlad commented Dec 16, 2024

Adding a clarification.

How to use it:
scrip/./ToolAnalysisMyTools.sh configfileName or
scrip./ToolAnalysisMyTools.sh "configfileName1,configfileName2,..." or
scrip./ToolAnalysisMyTools.sh configfileName1 clean

Adding more information
there was (") missing
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.

1 participant