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

Keep up with coffea histogram changes #35

Open
areinsvo opened this issue Aug 5, 2019 · 0 comments
Open

Keep up with coffea histogram changes #35

areinsvo opened this issue Aug 5, 2019 · 0 comments

Comments

@areinsvo
Copy link
Contributor

areinsvo commented Aug 5, 2019

In coffea 0.6.7, the usage of project() and group() was changed and our plotting code is no longer compatible. Our code needs to be updated. The current plotting code should still work with coffea 0.6.6 but breaks in 0.6.7.

From the cms-coffea-users egroup:
Just an FYI, coffea 0.6.7 introduced two backwards-incompatible changes to the histogram interface:

  • Change Hist.group() signature to be consistent with Hist.rebin()
    To upgrade: transpose the two arguments: Hist.group(new_axis, old_axis, ...) to Hist.group(old_axis, new_axis, ...)
    (per Different interfaces for hist.rebin() and hist.group() scikit-hep/coffea#90 )
  • Rename hist.project() to hist.integrate()
    Currently hist.project() will perform the same action as before, but emits a FutureWarning that it will eventually change behavior. In the meantime, just replace calls to project() with calls to integrate(). The signature is identical.
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