robotology / idyntree

Multibody Dynamics Library designed for Free Floating Robots
BSD 3-Clause "New" or "Revised" License
177 stars 67 forks source link

Preparing for 201x release #309

Closed traversaro closed 7 years ago

traversaro commented 7 years ago

YARP/iCub are preparing for a release for the 15th of June, and iDynTree should merge devel in master and do a new release as well.

General Notification: Please avoid regenerating matlab bindings in master after https://github.com/robotology/idyntree/pull/305 as been merged, as it drastically changes the way in which we are handling the bindings.

Things to discuss:

traversaro commented 7 years ago

cc @francesco-romano @DanielePucci @diegoferigo @fjandrad @pattacini @nunoguedelha @S-Dafarra @gabrielenava

traversaro commented 7 years ago

I think we should merge all open PRs before the release.

traversaro commented 7 years ago

For the version number, I think we can sync with the icub-main one, and use v1.8.0 from the next release.

francesco-romano commented 7 years ago

Honestly, I would prefer to have our own version number...

traversaro commented 7 years ago

I see no advantage in having our own version number, but I am open to other suggestions.

nunoguedelha commented 7 years ago

What I've seen in Release Engineering, and it seemed to be a good practice, is having a version number for each component / library and an "Integration" version number for the superbuild, connecting all the components/libraries together. This allows to handle dependencies while letting each component follow its own rhythm of releases. Of course we can, and probably should use the same format, and agree on common versioning convention, if it's not already done.

traversaro commented 7 years ago

For the version number, I think @diegoferigo should decide, just because I don't want to decide on this. : )

traversaro commented 7 years ago

Once we decided on a version number, we can roll out the ChangeLog and finally release the new version.

diegoferigo commented 7 years ago

I don't think this decision should be mine, however, this is a recap of the current status of our software:

As you see, we have no policy at all. With today's release, idyntree will be even more standalone than ever, and in my humble opinion it doesn't have much sense for the time being aligning it randomly to any project. If in the future a more strict versioning will be enforced for our ecosystem, we will adapt accordingly. Said this, @traversaro if you think we reached a decent API stability, we can bump to 1.0, otherwise, if we are close and with the next release we plan to freeze or at least to minimize API changes, 0.9 could work. Third option, postponing this discussion and go with 0.5. Opinions?

traversaro commented 7 years ago

I don't think this decision should be mine

I think it is a good exercise, if @DanielePucci agrees. : )

Regarding the current status:

Anyhow, no problem in having our own release naming scheme, probably we will need at least to keep the odd/even stability style.

traversaro commented 7 years ago

For reference, this is the discussion in which I argued for all icub repos to share the same version number: https://github.com/robotology/icub-main/issues/339 .

traversaro commented 7 years ago

After f2f meeting, we agreed on a version number of 0.8 for this version of iDynTree.

DanielePucci commented 7 years ago

I do agree :)

diegoferigo commented 7 years ago

@traversaro Do you have a changelog draft (at least in your mind)?

traversaro commented 7 years ago

I started a document following YARP style in https://github.com/robotology/idyntree/blob/devel/doc/releases/v0_8.md . For the 0.8 is quite simple, but I hope that in the future we can maintain it for each PR (i.e. each PR should document its changes).

diegoferigo commented 7 years ago

each PR should document its changes @traversaro

I always agree for distributing the effort in a long timespan

traversaro commented 7 years ago

There are several important class that are under heavy modifications such as InverseKinematics and BerdyHelper . I propose that we wait some of these modifications before releasing the new version of iDynTree .

diegoferigo commented 7 years ago

I agree with you, also considering that we lost the yarp freeze train for this time.

traversaro commented 7 years ago

We need some of the new classes handling conversion from and to DH chains in WB-Toolbox, so I think it is time to finally release the latest version of iDynTree devel in master.

I think we should properly mark the parts of the library that we expect to change in the documentation as such, to avoid more freedom in releasing iDynTree .

traversaro commented 7 years ago

I added a few notes for the experimental classes in https://github.com/robotology/idyntree/pull/376 , once we merge it we can relase iDynTree 0.8 .

traversaro commented 7 years ago

iDynTree 0.8 released in https://github.com/robotology/idyntree/pull/377 .