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

[PR Tracking] for Tau Algorithm, Jet Algorithm, and Alg Updates (circa Mid Sept 2015) #41

Closed
mulhearn opened this issue Sep 22, 2015 · 6 comments

Comments

@mulhearn
Copy link
Member

76X PR cms-sw#11253 is already merged (since 7_6_0_pre5)
75X PR cms-sw#11437 is merged.
74X PR cms-sw#11435 is merged.
So far only the minimum prerequisites commits from #43 needed to compile cms_sw#11253 were included in the 74X PR. I still worry that stuff is missing in 74X but will follow up in that dedicated issue.

@mulhearn mulhearn changed the title PRs for Tau Algorithm, Jet Algorithm, and Alg Updates (circa Mid Sept 2015) [PR Tracking] for Tau Algorithm, Jet Algorithm, and Alg Updates (circa Mid Sept 2015) Sep 22, 2015
@mulhearn
Copy link
Member Author

@apana you are interested in this.

@mulhearn
Copy link
Member Author

After cherry-picking into 74X, the following compile error... some prereq missing... investigating.
error: 'isoTauLut' was not declared in this scope
unsigned lutAddress = isoTauLut->lutAddress(tauEt,jetEt);

@mulhearn
Copy link
Member Author

This one isn't in 74X...
bbcafca

This was part of the wholesale 75X alg updates that weren't backported to 74X, tracking this issue here #43

@mulhearn
Copy link
Member Author

Included in #67

@mulhearn
Copy link
Member Author

David agrees to include in 74X, so no longer urgent issue.

@mulhearn mulhearn removed the URGENT label Oct 22, 2015
@mulhearn
Copy link
Member Author

All done.

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

No branches or pull requests

1 participant