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
DM-14204: Modernize python in ip_* packages #84
Conversation
4880360
to
d083bcf
Compare
import unittest | ||
|
||
from builtins import range | ||
from past.builtins import basestring |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As you are doing more than future, you may consider changing the commit message to something about dropping py2 support and syntax, but it is just a suggestion
|
||
This task has no debug variables | ||
|
||
\section ip_diffim_imageDecorrelation_DecorrelateALKernelSpatialTask_Example Example of using DecorrelateALKernelSpatialTask | ||
@section ip_diffim_imageDecorrelation_DecorrelateALKernelSpatialTask_Example Example of using DecorrelateALKernelSpatialTask # noqa E501 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should there be two spaces before Example?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not make any difference. However, in responding to this I see another error: the # noqa 501
is going to show up as part of the title so will fix that (in two files) by shortening the line and removing the # noqa
. While I'm fixing that I'll make it one space in the files I fix.
Doxygen and pep8 don't get along very well.
d083bcf
to
ae53a1c
Compare
Remove use of future and __future__
ae53a1c
to
10ea640
Compare
I don't know how to fix or hide the ## E266 warnings for the `##` comment blocks.
d30ae36
to
49aad9d
Compare
No description provided.