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: GraphDatasets v0.1.0 #29788

Merged
merged 1 commit into from
Mar 15, 2021

Conversation

JuliaRegistrator
Copy link
Contributor

UUID: bf13de61-383b-4610-8d2c-e75fac0df62b
Repo: https://github.com/simonschoelly/GraphDatasets.jl.git
Tree: 15e575145b50e84529443c797311e9ed19afc460

Registrator tree SHA: e934b8c55381f28735124f23e8f7e96d09b20416
JuliaRegistrator referenced this pull request in simonschoelly/GraphDatasets.jl Feb 10, 2021
* MUTAG and Mutagenicity
* NCI1 and NCI109
* PTCDatasets
* PROTEINS datasets
* Add fingerprint dataset
@JuliaRegistrator JuliaRegistrator temporarily deployed to stopwatch February 10, 2021 11:40 Inactive
@github-actions
Copy link
Contributor

Your new package pull request does not meet the guidelines for auto-merging. Please make sure that you have read the General registry README. The following guidelines were not met:

  • Package name similar to 1 existing package.
  1. Similar to GraphMLDatasets. Damerau-Levenshtein distance 2 is at or below cutoff of 2.

Note that the guidelines are only required for the pull request to be merged automatically. However, it is strongly recommended to follow them, since otherwise the pull request needs to be manually reviewed and merged by a human.

After you have fixed the AutoMerge issues, simple retrigger Registrator, which will automatically update this pull request. You do not need to change the version number in your Project.toml file (unless of course the AutoMerge issue is that you skipped a version number, in which case you should change the version number).

If you do not want to fix the AutoMerge issues, please post a comment explaining why you would like this pull request to be manually merged.

Since you are registering a new package, please make sure that you have also read the package naming guidelines: https://julialang.github.io/Pkg.jl/dev/creating-packages/#Package-naming-guidelines-1


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.

@simonschoelly
Copy link
Contributor

[noblock]
@yuehhua The name of this package would be similar to your GraphMLDatasets package. Do you think these packages could coexist or would you prefer if I come up with a more different name like NetworkDatasets, GraphData (I have no had a convincing idea yet)

@yuehhua
Copy link
Contributor

yuehhua commented Feb 11, 2021

[noblock]
I checked the paper you reference. Maybe you can use TUDatasets as the paper's title or a different name like NetworkDatasets is also OK. However, if you will, it is welcome to contribute GraphMLDatasets.

@DilumAluthge
Copy link
Member

[noblock]

@simonschoelly Do you want me to manually merge this PR? Or do you want me to close it?

@DilumAluthge
Copy link
Member

[noblock]

Bump @simonschoelly.

Do you want me to manually merge this PR? Or do you want me to close it?

@simonschoelly
Copy link
Contributor

Sorry for the delay. We disscused this now on slack and it is ok to keep the name as it is. So could you merge this manually?

@DilumAluthge DilumAluthge merged commit 19707d7 into master Mar 15, 2021
@DilumAluthge DilumAluthge deleted the registrator/graphdatasets/bf13de61/v0.1.0 branch March 15, 2021 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants