dfdx / Boltzmann.jl

Restricted Boltzmann Machines in Julia
Other
67 stars 27 forks source link

Register for Julia 1.0? #51

Closed cossio closed 5 years ago

cossio commented 5 years ago

Why is the package not registered? Does it work on Julia 1.0? Thanks.

dfdx commented 5 years ago

I think when the code here was updated for Julia 1.0 there's was no clear guidance on migrating to new package infrastructure (e.g. adding Package.toml instead of REQUIRE) so I decided to postpone registration. So far I still don't see clear guidance, but combing both - Project.toml and REQUIRE - seems to be de facto approach. I'll try to update Boltzmann.jl and register a new version.

If you need it right now, you can always install the master branch using:

] add Boltzmann#master
rofinn commented 5 years ago

Yes, this code was updated, but at the time you could only tag packages through METADATA. Since METADATA is no longer synchronized with the main registry METADATA.jl isn't accepting PRs for 1.0 and we shouldn't need the REQUIRE anymore. The recommended method now is to use Registrator.jl and related tools. If you enable the recommended bot on this repo I can add a Project.toml.

dfdx commented 5 years ago

Thanks @rofinn for letting me know about Registrator.jl - I missed the point when we switched to it. I've enabled the bot for Boltzmann.jl, so feel free to add Project.toml. Otherwise, I'll take a look at it myself on the weekend.

dfdx commented 5 years ago

@JuliaRegistrator register()

JuliaRegistrator commented 5 years ago

Registration pull request created: JuliaRegistries/General/511

After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.

This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:

git tag -a v0.7.0 -m "<description of version>" 234bd18a650adcda25b40e9691d510ae09816b50
git push origin v0.7.0
dfdx commented 5 years ago

Version v0.7 is now registered, thanks @rofinn for your help!