[MRG] Release 0.6 by rflamary · Pull Request #91 · PythonOT/POT · GitHub
Skip to content

[MRG] Release 0.6#91

Merged
rflamary merged 16 commits into
masterfrom
v1.0
Sep 10, 2019
Merged

[MRG] Release 0.6#91
rflamary merged 16 commits into
masterfrom
v1.0

Conversation

@rflamary

@rflamary rflamary commented Jul 5, 2019

Copy link
Copy Markdown
Collaborator

This PR is a preparation for release 0.6 of POT

After all this work (and use of POT by me and other people) it feels as we reached a stable and nice list of tools around OT.

What I will do it this PR:

  • Change version to 0.6
  • Add detailed release text in RELEASES.md
  • Add build of wheels for manylinux using cibuildwheel (not on travis but locally before uploading to PIP)
  • Remove beta flag
  • Update Pypi classifiers
  • Add Python 3.7 to travis and remove 3.4

@hichamjanati

hichamjanati commented Jul 23, 2019

Copy link
Copy Markdown
Contributor

@ngayraud

Copy link
Copy Markdown
Contributor

I completely agree with @hichamjanati . Especially on the naming issues: there are so many variables with names like a, b, etc... which should change into meaningful variable names - including the filenames (eg da could be named DomainAdaptation) A lot of little issues here and there need to be addressed. For example, in the da code, the fit() functions take Xs, ys, Xt, yt as input. Very often, one does not have (or need) ys and yt. The resulting code looks like this:
...fit(Xs, None, Xt). The order should change into (Xs, Xt, ys, yt).

A coding spring would surely address all of that, I'd be happy to attend if I can!

@rflamary

Copy link
Copy Markdown
Collaborator Author

Hello,

I'm OK to call it release 0.6 but we need a proper release shortly (we are beginning to be very late on pip and conda) and a lot of the new stuff is only available on source. My idea behind 1.0 was that similarly to recent scipy i think we have achieved an important milestone and it might be the time to release a stable version that has been used in the wild. we could then start work on 2.0 that will introduce some API changes.

I also agree that POT could do with some work on the API and a cleanup of parameter names. But those would come with a proper deprecation schedule and I proper discussions between all contributors (and users).

@hichamjanati I agree with most of your wish list but i will not put some API changes in the next release. The last release is already quite old and I will do a merge/release in the next weeks. I think the change you propose require some time to discuss and test before release because it will break a lot of things.

Meeting together for a code sprint is a very nice idea. I will think about how to organize this (we can also imagine a virtual meeting to limit transport).

@rflamary rflamary changed the title [WIP] Preparing Release 1.0 [WIP] Preparing Release 1.0 (or 0.6 ?) Aug 15, 2019
@rflamary rflamary changed the title [WIP] Preparing Release 1.0 (or 0.6 ?) [WIP] Preparing Release 0.6 Sep 9, 2019
@rflamary rflamary changed the title [WIP] Preparing Release 0.6 [MRG] Release 0.6 Sep 9, 2019
@rflamary

rflamary commented Sep 9, 2019

Copy link
Copy Markdown
Collaborator Author

Hello I have built wheels for all linux version (Py 2.7,3.4,3.5,3.6,3.7 for 32 and 64 bits). I will upload them on release so that it will not be necessary to compile POT from source when installing from Pypi.

We have a nice version 0.6 with the last unbalanced merge. The last release on pypi was nearly one year ago so I will go ahead with merging and releasing shortly this new version of POT if there is no objections.

I have seen a lot of comments (and upvotes) about API updates and I'm open to discussions. It's something that we can do with proper deprecation releases and so on but we need to discuss it all together. Maybe we should open a thread on the POT slack ?

@ncourty

ncourty commented Sep 9, 2019 via email

Copy link
Copy Markdown
Collaborator

@agramfort

agramfort commented Sep 9, 2019 via email

Copy link
Copy Markdown
Collaborator

@rflamary rflamary merged commit 5e70a77 into master Sep 10, 2019
@rflamary rflamary deleted the v1.0 branch October 17, 2019 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants