Skip to content
This repository has been archived by the owner on Dec 7, 2018. It is now read-only.

Wrapped tex strings to 80-character limit #19

Merged
merged 6 commits into from
Apr 17, 2018

Conversation

duncanmmacleod
Copy link
Member

This PR appeases pep8 some more by wrapping TeX strings to the 80-character limit. Also included are a few tex bug fixes, and some other pep8 style changes.

This PR should not change the functionality of any code, in any way.

@duncanmmacleod duncanmmacleod added this to the 0.1.0 milestone Apr 17, 2018
@duncanmmacleod duncanmmacleod added documentation Changes to code documentation pr:bug-fix Backwards-compatible change that does not add new functionality (patch version bump) labels Apr 17, 2018
@coveralls
Copy link

coveralls commented Apr 17, 2018

Pull Request Test Coverage Report for Build 89

  • 2 of 4 (50.0%) changed or added relevant lines in 2 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage remained the same at 40.441%

Changes Missing Coverage Covered Lines Changed/Added Lines %
gwin/likelihood.py 1 2 50.0%
gwin/results/scatter_histograms.py 1 2 50.0%
Files with Coverage Reduction New Missed Lines %
gwin/workflow.py 1 0.0%
Totals Coverage Status
Change from base Build 86: 0.0%
Covered Lines: 899
Relevant Lines: 2223

💛 - Coveralls

@cdcapano
Copy link
Collaborator

@duncanmmacleod Thanks for doing this... do you have an example output page?

@duncanmmacleod
Copy link
Member Author

https://ldas-jobs.ligo.caltech.edu/~duncan.macleod/gwin/19/html/ (requires LIGO.ORG authentication)

Copy link
Collaborator

@cdcapano cdcapano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for figuring out the line continuation with tex.

@cdcapano cdcapano merged commit 0d50dca into gwastro:master Apr 17, 2018
@duncanmmacleod duncanmmacleod deleted the tex-linebreaks branch April 17, 2018 13:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Changes to code documentation pr:bug-fix Backwards-compatible change that does not add new functionality (patch version bump)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants