add fig_kwargs

[Borda]

I found quite convenient to add extra figure arguments

[pep8speaks]

Hello @Borda, Thank you for updating !

• In the file dipy/viz/regtools.py:

Line 52:81: E501 line too long (93 > 80 characters)

Comment last updated at 2020-07-22 17:45:09 UTC

[codecov]

Codecov Report

Merging #2168 into master will not change coverage.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##           master    #2168   +/-   ##
=======================================
  Coverage   91.37%   91.37%           
=======================================
  Files         251      251           
  Lines       32449    32449           
  Branches     3408     3408           
=======================================
  Hits        29651    29651           
  Misses       2057     2057           
  Partials      741      741           

Impacted Files	Coverage Δ
dipy/viz/regtools.py	31.49% <33.33%> (ø)

[skoudoro]

Thank you for this @Borda!

Looks fine to me. Can you just fix the pep8 above and it is ready to be merged.
Thank you

[Borda]

Looks fine to me. Can you just fix the pep8 above and it is ready to be merged.

it looks like some unrelated issue...
E ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1076)

[skoudoro]

Yes, I saw that, there is an issue for it (#2172)

[Borda]

Yes, I saw that, there is an issue for it (#2172)

so I will wait till it is resolved... :]

[Borda]

@skoudoro rebased on actual master 🦝

[skoudoro]

Master has been fixed @Borda, you can rebase, and finalize your PR. Sorry for the delay

[Borda]

@skoudoro done :]

[skoudoro]

Thank you @Borda!