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

Getting website ready for 0.11 release #916

Merged
merged 9 commits into from Feb 13, 2016

Conversation

Projects
None yet
2 participants
@Garyfallidis
Member

Garyfallidis commented Feb 12, 2016

Hi this PR fixes crucial issue #906 and minor issue #900. Also it adds the dki tutorial back in the index.

@@ -1,4 +1,4 @@
.. _release0.10:
.. _release0.11:

This comment has been minimized.

@arokem

arokem Feb 12, 2016

Member

Good catch!

This comment has been minimized.

@Garyfallidis

Garyfallidis Feb 12, 2016

Member

Hey after fixed this for some reason the link on 0.10 in the index.rst stopped linking.
I am now rerunning the website to double check.
Finally, we can compile the website and do other work simultaneously!! 🎱

plt.close('all')
print(script)
execfile(script, namespace)
del namespace

This comment has been minimized.

@arokem

arokem Feb 12, 2016

Member

Ah yes. That's bonkers...

@@ -83,6 +86,8 @@ def show_both_bundles(bundles, colors=None, show=False, fname=None):
**After bundle registration**.
As you can see the two cingulum bundles are well aligned although they contain many streamlines of different length and shape.

This comment has been minimized.

@arokem

arokem Feb 12, 2016

Member

PEP8: long line...

@@ -22,7 +23,9 @@
"""
An important step before running the registration is to resample the streamlines
so that they both have the same number of points per streamline. Here we will
use 20 points.
use 20 points. This step is not optional. Inputing streamlines with different

This comment has been minimized.

@arokem

arokem Feb 12, 2016

Member

Typo: "Inputing" => "Inputting"

@arokem

This comment has been minimized.

Member

arokem commented Feb 13, 2016

👍

@arokem

This comment has been minimized.

Member

arokem commented Feb 13, 2016

This is all on the docs side, so I am not going to wait for Travis and go ahead with the merge.

arokem added a commit that referenced this pull request Feb 13, 2016

Merge pull request #916 from Garyfallidis/website_for_0.11_release
Getting website ready for 0.11 release

@arokem arokem merged commit b181a13 into nipy:master Feb 13, 2016

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@Garyfallidis

This comment has been minimized.

Member

Garyfallidis commented on 55166f7 Nov 28, 2016

I can replicate the issue. Let me try it and will get back to you.

This comment has been minimized.

Member

arokem replied Nov 28, 2016

This comment has been minimized.

Member

Garyfallidis replied Nov 28, 2016

yes, thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment