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

New package: KmeansBenchmarks v0.1.0 #123879

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JuliaRegistrator
Copy link
Contributor

UUID: 86303ed7-7ab5-4b67-a024-c8ce05cc70f4
Repo: https://github.com/szcf-weiya/KmeansBenchmarks.jl.git
Tree: d86c7abbf1177812e39a465b98106d6876ef110d

Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
Copy link
Contributor

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 registration

Please make sure that you have read the package naming guidelines.

2. AutoMerge Guidelines which are not met ❌

  • There is no compat entry for julia.
  • No licenses detected in the package's top-level folder. An OSI-approved license is required.

3. Needs action: here's what to do next

  1. Please try to update your package to conform to these guidelines. The General registry's README has an FAQ that can help figure out how to do so.
  2. After you have fixed the AutoMerge issues, simply retrigger Registrator, the same way you did in the initial registration. This will automatically update this pull request. You do not need to change the version number in your Project.toml file (unless the AutoMerge issue is that you skipped a version number).

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 #pkg-registration channel in the public Julia Slack for better visibility.

4. To pause or stop registration

If 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 [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

@gdalle
Copy link
Contributor

gdalle commented Jan 28, 2025

Hi, congrats on the package! Does it need to be registered though?

@JuliaTagBot JuliaTagBot added the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Jan 28, 2025
@szcf-weiya
Copy link

Hi @gdalle, thanks for your comment. I thought the package could be easily accessed and installed by other users once registered, and I want to provide a clear and simple template for benchmarking if one wants to contribute to the package. Actually, before I decided to register, I checked that there are also some benchmarks on other particular aspects, such as this one https://github.com/fcdimitr/LFRBenchmarkGraphs.jl

@gdalle
Copy link
Contributor

gdalle commented Jan 28, 2025

Installation is also very easy for unregistered packages based on the GitHub URL. Given that yours is not really meant to be reused in other projects, and given that it is rather short (which doesn't make it trivial!), I'm not sure registration brings all that much. But I'd love to hear other opinions!

@szcf-weiya
Copy link

@gdalle thanks for your comment. I will keep it unregistered for some while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants