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

Migrate PyJet dependency to FastJet #158

Closed
jacanchaplais opened this issue Oct 3, 2023 · 0 comments · Fixed by #159
Closed

Migrate PyJet dependency to FastJet #158

jacanchaplais opened this issue Oct 3, 2023 · 0 comments · Fixed by #159
Assignees
Labels
enhancement New feature or request

Comments

@jacanchaplais
Copy link
Owner

I noticed an ImportError when trying to import graphicle, due to a missing symbol from a C library. I am unsure whether this is to do with an instability in the latest release of the C libraries, or due to PyJet's dependencies no longer keeping up to date, but it seems now is a good time to migrate to FastJet. This will add dependencies of awkward under the hood, but the same functionality can be achieved.

Specifically, using the fastjet.ClusterSequence constructor, and calling the .constituent_index() method achieves the same thing as the structured array approach, with the index inserted in as a field before clustering. The only difference appears to be that these clusters are not sorted by $p_T$, so make sure to do this before yielding the results out.

@jacanchaplais jacanchaplais added the enhancement New feature or request label Oct 3, 2023
@jacanchaplais jacanchaplais self-assigned this Oct 3, 2023
jacanchaplais added a commit that referenced this issue Oct 3, 2023
@jacanchaplais jacanchaplais linked a pull request Oct 3, 2023 that will close this issue
jacanchaplais added a commit that referenced this issue Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant