Skip to content

Conversation

fmassa
Copy link
Member

@fmassa fmassa commented Mar 24, 2020

No description provided.

@fmassa fmassa requested a review from seemethere March 24, 2020 18:12
description: "version number of release binary; by default, build a nightly"
type: string
default: ""
default: "0.5.1"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why just a bump in patch version? Is there only bug fixes within this release?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There will be no major new features in this release, so I think it might be better to make it a minor release.

But given that we haven't had a 1.0 release of torchvision, we have been using the patch numbering for minor releases. At some point we should decide when to cut the 1.0.0 release and do proper MAJOR.MINOR.PATCH versioning though

@seemethere
Copy link
Member

seemethere commented Mar 24, 2020

Oh right, the rc1 for 1.5.0 got incorrectly marked so that the no package actually exists with +cu101

This should be remedied in pytorch/pytorch#35309, going to see if we can fix this in RC2

@vincentqb vincentqb mentioned this pull request Mar 24, 2020
1 task
@peterjc123
Copy link
Contributor

Do we need to do the release for Windows?

@fmassa
Copy link
Member Author

fmassa commented Mar 30, 2020

@peterjc123 yes, we need to do a release for Windows as well, I just saw that you opened a PR for it, thanks!

@peterjc123
Copy link
Contributor

@fmassa Which branch should be used?

@codecov-io
Copy link

codecov-io commented Mar 31, 2020

Codecov Report

Merging #2011 into v0.5.1 will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           v0.5.1   #2011   +/-   ##
======================================
  Coverage    0.48%   0.48%           
======================================
  Files          92      92           
  Lines        7407    7407           
  Branches     1128    1128           
======================================
  Hits           36      36           
  Misses       7358    7358           
  Partials       13      13           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b68adcf...efa3cc5. Read the comment docs.

@seemethere seemethere self-requested a review March 31, 2020 04:41
@seemethere seemethere dismissed their stale review March 31, 2020 04:53

committed my change

@fmassa
Copy link
Member Author

fmassa commented Apr 1, 2020

@peterjc123 we are going to be using branch v0.5.1, although the release will be 0.6.0, which I agree it's confusing. Let me know if you think creating a new branch named v0.6.0 would be better

@peterjc123
Copy link
Contributor

@fmassa No problem. I think it's okay. BTW, please tell me when do you need the release.

@fmassa
Copy link
Member Author

fmassa commented Apr 1, 2020

@peterjc123 thanks! I'll let you know, I'm currently fixing a bug and I'll let you know when everything is ready

@fmassa fmassa closed this Jul 8, 2020
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.

4 participants