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.

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

Bug fixes #38

Merged
merged 9 commits into from
Mar 27, 2018
Merged

Bug fixes #38

merged 9 commits into from
Mar 27, 2018

Conversation

samaloney
Copy link
Member

No description provided.

@pep8speaks
Copy link

Hello @samaloney! Thanks for submitting the PR.

Line 166:101: E501 line too long (147 > 100 characters)
Line 170:101: E501 line too long (172 > 100 characters)
Line 304:101: E501 line too long (147 > 100 characters)
Line 381:101: E501 line too long (147 > 100 characters)

Line 280:5: E265 block comment should start with '# '
Line 281:13: E231 missing whitespace after ','
Line 295:24: E231 missing whitespace after ','
Line 295:31: E231 missing whitespace after ','
Line 316:34: E231 missing whitespace after ','

@samaloney samaloney merged commit b051b96 into TCDSolar:master Mar 27, 2018
samaloney pushed a commit to samaloney/xrayvision that referenced this pull request Mar 28, 2018
Bug fixes (TCDSolar#38)
92a8d3b 
* Fix TCDSolar#29 add unit decorator to init method

* Fix TCDSolar#30 sign change on u, v coords on import

* Fix lint issues on long lines in docstrings

* Change normalisation of idft to no of visibilities from number of pixels

* Temporary fix for to_map for RHESSI

* Update .rtd-environment.yml so doc built work on RTD
@samaloney samaloney deleted the bug-fixes branch March 28, 2018 10:19
samaloney pushed a commit that referenced this pull request Jun 7, 2018
Doc updates (#39)
2c94c99 
* Fix #29 add unit decorator to init method

* Fix #30 sign change on u, v coords on import

* Fix lint issues on long lines in docstrings

* Change normalisation of idft to no of visibilities from number of pixels

* Temporary fix for to_map for RHESSI

* Bug fixes (#38)

* Fix #29 add unit decorator to init method

* Fix #30 sign change on u, v coords on import

* Fix lint issues on long lines in docstrings

* Change normalisation of idft to no of visibilities from number of pixels

* Temporary fix for to_map for RHESSI

* Update .rtd-environment.yml so doc built work on RTD

* Update readme

* Update tutorial

* Add RHESSI time parsing

* Travis Issues
samaloney pushed a commit that referenced this pull request Jun 17, 2018
Bug fixes and doc updates (#40)
e227549 
* Fix lint issues on long lines in docstrings

* Fix various bugs

* Bug fixes (#38)

* Fix #29 add unit decorator to init method

* Fix #30 sign change on u, v coords on import

* Fix lint issues on long lines in docstrings

* Change normalisation of idft to no of visibilities from number of pixels

* Temporary fix for to_map for RHESSI

* Update .rtd-environment.yml so doc build works on RTD

* Update tutorial and readme

* Update tutorial add images

* Add RHESSI time parsing

* More travis updates

* Update README fix badges

* Bug fixes and documentation updates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@samaloney @pep8speaks