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.

Sign up for GitHub

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

Jump to bottom

Moving idft and dft into Transform from Visibility #8

Merged

samaloney merged 1 commit into TCDSolar:master from pgabor:transformation

Aug 12, 2017

Contributor

pgabor commented Aug 7, 2017

No description provided.


          Moving idft and dft into Transform from Visibility

8981b51

pep8speaks commented Aug 7, 2017

Hello @pgabor! Thanks for submitting the PR.

In the file xrayvision/Transform.py, following are the PEP8 issues :

Line 25:80: E501 line too long (94 > 79 characters)
Line 48:80: E501 line too long (87 > 79 characters)
Line 55:80: E501 line too long (97 > 79 characters)
Line 80:80: E501 line too long (86 > 79 characters)

In the file xrayvision/Visibillity.py, following are the PEP8 issues :

Line 63:80: E501 line too long (91 > 79 characters)
Line 80:80: E501 line too long (98 > 79 characters)
Line 92:80: E501 line too long (87 > 79 characters)
Line 129:80: E501 line too long (86 > 79 characters)
Line 163:80: E501 line too long (82 > 79 characters)

samaloney approved these changes

View reviewed changes

samaloney merged commit e4f66f3 into TCDSolar:master

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

Labels

None yet