Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REL: 1.5.0 #3174

Merged
merged 35 commits into from
Mar 31, 2020
Merged

REL: 1.5.0 #3174

merged 35 commits into from
Mar 31, 2020

Conversation

effigies
Copy link
Member

@effigies effigies commented Feb 23, 2020

Summary

Prep for new feature release 1.5.0, targeting release on Monday, February 24.

Given that we just released 1.4.2 about a week ago, I'm inclined to hold this one off for any feature PRs that would like to shoot for inclusion.

Require merge or postponement decision on all issues/PRs in https://github.com/nipy/nipype/milestone/38:

Will try review the open PRs and see if anything is close enough to push on, tomorrow.

Release checklist

Uncredited authors

The following authors have contributed, but not added themselves to the .zenodo.json file. If you would like to be an author on Zenodo releases, please add yourself or comment with your preferred publication name, affiliation and ORCID. If you would like to stop being spammed whenever I'm the one doing releases, let me know, and I'll add you to a blacklist.

No entry to sort: cdla (@cdla)
No entry to sort: Gio Piantoni (@gpiantoni)
No entry to sort: Victor Férat (@vferat)
No entry to sort: Niklas Förster (@niklasfoe)
No entry to sort: Adam Kimbler (@adamkimbler)
No entry to sort: Kirstie Whitaker (@KirstieJane)
No entry to sort: Pablo Polosecki (@polosecki)
No entry to sort: Ami Tsuchida
No entry to sort: Daniel Brenner (@brennerd11)
No entry to sort: Isaiah Norton (@ihnorton)
No entry to sort: Kevin Sitek (@sitek)
No entry to sort: Luke Bloy (@bloyl)
No entry to sort: Martin Luessi (@mluessi)
No entry to sort: steve (@steve19922)
No entry to sort: Charl Linssen (@turingbirds)
No entry to sort: Félix C. Morency (@fmorency)
No entry to sort: Jonathan R. Williford (@williford)
No entry to sort: Michiel Cottaar (@MichielCottaar)
No entry to sort: Regina Kim (@reginakim)
No entry to sort: Valentin Haenel (@esc)
No entry to sort: Xu Wang
No entry to sort: maedoc (@maedoc)

I am unable to find GitHub handles for Ami Tsuchida or Xu Wang.

Apologies also to anybody who may have

Acknowledgment

  • (Mandatory) I acknowledge that this contribution will be available under the Apache 2 license.

@codecov
Copy link

codecov bot commented Feb 23, 2020

Codecov Report

Merging #3174 into master will decrease coverage by 0.01%.
The diff coverage is 62.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3174      +/-   ##
==========================================
- Coverage   65.07%   65.06%   -0.02%     
==========================================
  Files         301      301              
  Lines       39624    39624              
  Branches     5247     5247              
==========================================
- Hits        25786    25781       -5     
- Misses      12771    12776       +5     
  Partials     1067     1067
Flag Coverage Δ
#unittests 65.06% <62.5%> (-0.02%) ⬇️
Impacted Files Coverage Δ
nipype/interfaces/base/traits_extension.py 88.23% <ø> (ø) ⬆️
nipype/pipeline/engine/workflows.py 69.12% <0%> (ø) ⬆️
nipype/interfaces/io.py 56.41% <0%> (ø) ⬆️
nipype/info.py 92.3% <100%> (ø) ⬆️
nipype/interfaces/freesurfer/utils.py 63.25% <100%> (ø) ⬆️
nipype/utils/filemanip.py 72.87% <0%> (-1.18%) ⬇️

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 c37fcfb...47de76e. Read the comment docs.

@akimbler
Copy link
Contributor

Hey,

Sorry for the delay in submitting this.

{
"affiliation": "Florida International University",
"name": "Kimbler, Adam",
"orcid": "0000-0001-5885-9596"
}

@effigies effigies added this to the 1.5.0 milestone Feb 24, 2020
@effigies effigies force-pushed the rel/1.5.0 branch 2 times, most recently from be4e46d to 84d48f4 Compare February 24, 2020 02:50
@effigies
Copy link
Member Author

@miykael @djarecka Any chance one of you could have a look at https://circleci.com/workflow-run/6eba71d4-5cb8-4711-b95a-83d7393dd71c?

For reference, here's the diff between that branch and master: miykael/nipype_tutorial@master...effigies:rel/1.4.1

@effigies
Copy link
Member Author

Bump for @miykael and @djarecka, in case this got lost in your notifications last night.

@oesteban
Copy link
Contributor

It could be interesting to push #3180 into 1.5.0, wdyt @effigies ?

@effigies
Copy link
Member Author

Sure. I don't see much prospect of fixing the tutorials soon, so the release is held up for now. Squeeze in as you like.

@williford
Copy link
Contributor

williford commented Feb 29, 2020 via email

@oesteban
Copy link
Contributor

While we figure out the notebooks problem, could we maybe push an RC?

@effigies
Copy link
Member Author

Sure.

@effigies
Copy link
Member Author

@oesteban @satra I'm planning on releasing 1.5.0rc1 from this branch, once the feedstock build is fixed. Or, if we don't care about getting release candidates on conda, I can just go ahead...

@oesteban
Copy link
Contributor

I'm fine either way. I don't think we really need this RC other than in Pypi

@effigies
Copy link
Member Author

Sorry, should have updated. Rerendering the feedstock worked. 1.5.0rc1 is on PyPI and conda-forge.

sulantha2006 and others added 22 commits March 20, 2020 17:37
FIX: Use PackageInfo to get NiftyReg version
TEST: Mock terminal output before testing changing default value
ENH: Add reverse-ordered transform lists to ants.Registration outputs
@effigies effigies merged commit 47de76e into master Mar 31, 2020
@effigies effigies mentioned this pull request Mar 31, 2020
10 tasks
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.

6 participants