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.

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

fix: save analysis data #41

Merged
merged 6 commits into from
May 13, 2019
Merged

fix: save analysis data #41

merged 6 commits into from
May 13, 2019

Conversation

njzjz
Copy link
Owner

@njzjz njzjz commented May 13, 2019

fix #40

@pep8speaks
Copy link
Contributor

pep8speaks commented May 13, 2019

Hello @njzjz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 26:80: E501 line too long (109 > 79 characters)
Line 28:80: E501 line too long (127 > 79 characters)
Line 31:80: E501 line too long (157 > 79 characters)

Line 58:80: E501 line too long (106 > 79 characters)
Line 71:80: E501 line too long (83 > 79 characters)

Comment last updated at 2019-05-13 12:16:19 UTC

@njzjz njzjz self-assigned this May 13, 2019
@conventional-release-bot
Copy link

v1.0.20

🐛 Bug Fixes

  • save analysis data (29e9c70) by Jinzhe Zeng

v1.0.19...v1.0.20

@codecov
Copy link

codecov bot commented May 13, 2019

Codecov Report

Merging #41 into master will increase coverage by 14.62%.
The diff coverage is 85.41%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #41       +/-   ##
===========================================
+ Coverage   57.21%   71.84%   +14.62%     
===========================================
  Files           5        5               
  Lines         194      206       +12     
===========================================
+ Hits          111      148       +37     
+ Misses         83       58       -25
Impacted Files Coverage Δ
gaussianrunner/tests/test_gaussianrunner.py 100% <100%> (ø) ⬆️
gaussianrunner/runner.py 82.5% <100%> (+1.67%) ⬆️
gaussianrunner/analyst.py 81.15% <80%> (+39.62%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 64b4f58...5c0461b. Read the comment docs.

@coveralls
Copy link

coveralls commented May 13, 2019

Coverage Status

Coverage increased (+14.6%) to 71.845% when pulling 5c0461b on analyze into 64b4f58 on master.

@njzjz njzjz merged commit ed54550 into master May 13, 2019
@delete-merged-branch delete-merged-branch bot deleted the analyze branch May 13, 2019 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

store useful dicts instead of log files
4 participants