Skip to content

Conversation

@jjacob
Copy link
Contributor

@jjacob jjacob commented Aug 9, 2014

...g bash and python scripts for 7 TeV running for samples recently made available.

@jjacob jjacob changed the title Updating cross-sections of processes for both 7 TeV and 8 TeV; updati… n... [WIP] Updating cross-sections of processes for both 7 TeV and 8 TeV; updati… n... Aug 11, 2014
@jjacob
Copy link
Contributor Author

jjacob commented Aug 14, 2014

Just to tidy up some of the pending Analysis Software issues, I believe this commit will fix issues numbers #106 and #89.

README.md Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is DICE-specific, I wouldn't put it here (or make it conditional)
as well as sourcing thisroot.sh on DICE

This recipe is not just for DICE — cmake is useful for compiling on macs, where CMSSW is unavailable.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can check if the hostname command includes dice.priv (or just dice).
Definitely make it clear in the documentation. You can also mention the minimal cmake version (2.6.0 according to CMakeLists.txt).

jjacob added a commit to jjacob/AnalysisSoftware that referenced this pull request Aug 15, 2014
@jjacob
Copy link
Contributor Author

jjacob commented Aug 15, 2014

Changes made according to comments.

@jjacob jjacob changed the title [WIP] Updating cross-sections of processes for both 7 TeV and 8 TeV; updati… n... Updating cross-sections of processes for both 7 TeV and 8 TeV; updati… n... Aug 18, 2014
@jjacob
Copy link
Contributor Author

jjacob commented Aug 18, 2014

[WIP] ---> ready to merge now after running on v11 nTuples.

kreczko added a commit that referenced this pull request Aug 20, 2014
Updating cross-sections of processes for both 7 TeV and 8 TeV; updati… n...
@kreczko kreczko merged commit fd0c1d9 into BristolTopGroup:master Aug 20, 2014
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