MPI.jl icon indicating copy to clipboard operation
MPI.jl copied to clipboard

Release version 1.0?

Open eschnett opened this issue 3 years ago • 7 comments

Recently, version 0.19 was released. Since the major version number is 0, Julia's package manager flags this as incompatible with 0.18, as if this was a major release. I don't think that's a correct description: This was a backward compatible change that introduced a new feature, i.e. truly a minor release.

Should we release MPI 1.0? The MPI standard is stable, the APIs provided here are stable, and adding new functionality will easily remain possible.

eschnett avatar Jul 30 '21 02:07 eschnett

Recently, version 0.19 was released. Since the major version number is 0, Julia's package manager flags this as incompatible with 0.18, as if this was a major release. I don't think that's a correct description: This was a backward compatible change that introduced a new feature, i.e. truly a minor release.

I deprecated MPI.Init_thread, so it wasn't quite backward compatible.

I guess the question is if there are any more breaking changes we should make in the near future? The only one I had in mind was #479, but i'd appreciate some input on that before I do anything more.

simonbyrne avatar Jul 30 '21 18:07 simonbyrne

Oh the other thing is whether Put and Get should have ! suffixes?

simonbyrne avatar Jul 30 '21 22:07 simonbyrne

I guess Put shouldn't, but Get should, since it'll modify its argument?

eschnett avatar Jul 31 '21 00:07 eschnett

Put modifies an argument on another process though.

simonbyrne avatar Jul 31 '21 00:07 simonbyrne

Julia uses put! to put values into a channel. I concede.

eschnett avatar Jul 31 '21 00:07 eschnett

It's purely a matter of personal preference, but I'd give the current version (which just introduced another breaking change) some time to mature for a while. If there are no breaking changes in the next 1-2 months, then I see no reason not to bump to v1.

sloede avatar Aug 12 '21 13:08 sloede

I've created a milestone and added the issues/PRs I thought should be included. If there are any other API breaks you would like to see, either chime in here or open an issue.

simonbyrne avatar Aug 27 '21 03:08 simonbyrne