-
Notifications
You must be signed in to change notification settings - Fork 465
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
New package: PYTHIA8 v0.1.0 #115979
New package: PYTHIA8 v0.1.0 #115979
Conversation
JuliaRegistrator
commented
Sep 25, 2024
- Registering package: PYTHIA8
- Repository: https://github.com/JuliaHEP/PYTHIA8.jl
- Created by: @peremato
- Version: v0.1.0
- Commit: 9b576775e56d11536594fe4fef97bea1f717e9d7
- Reviewed by: @peremato
- Reference: JuliaHEP/PYTHIA8.jl@9b57677#commitcomment-147197461
- Description: Pythia8 Julia interface
- Release notes:
UUID: ab9774ad-7527-4370-92ef-86b02700ecc9 Repo: https://github.com/JuliaHEP/PYTHIA8.jl.git Tree: 7aaa008e4fc205ccc185f7d0a7e9a8d683273c3b Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human. 1. New package registrationPlease make sure that you have read the package naming guidelines. 2. AutoMerge Guidelines which are not met ❌
3. Needs action: here's what to do next
If you need help fixing the AutoMerge issues, or want your pull request to be manually merged instead, please post a comment explaining what you need help with or why you would like this pull request to be manually merged. Then, send a message to the 4. To pause or stop registrationIf you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text Tip: You can edit blocking comments to add |
[noblock] @goerz Can you please unblock the request with this new name. Thanks! |
[noblock] So yeah, unfortunately, this still caught the "not all letters are upper-case" check, which I can't override. But it should be fine. You just have to go back on Slack in three days and ask for a maintainer to merge it manually. I don't see this PR as particularly controversial, so it's just a matter of getting someone's attention |
Thanks! |
[noblock] Why is this called PYTHIA8.jl, that is, why is the version number included in the package name? If a version 9 of the upstream package is released, should we expect a PYTHIA9.jl? That doesn't seem ideal. Otherwise I think this can be merged. |
[noblock] very good question, for some reason HEP has settled to this pattern: |
[noblock] PYTHIA6 was written in FORTRAN, PYTHIA8 is in C++. If ever there is a PYTHIA9 will be written on a different language, perhaps in Julia 😄, so we do not expect a PYTHIA9 in the immediate future. |
Alright, I take it that these different "versions" are more like entire rewrites, in which case this makes sense. I'll go ahead and merge this. |