Skip to content

Conversation

pandeyhs
Copy link
Contributor

@pandeyhs pandeyhs commented Sep 26, 2017

Example python scripts are being added, as monitored by #111.

@henryiii
Copy link
Member

henryiii commented Sep 26, 2017

@pandeyhs , I've merged master into the branch, please pull.

@codecov
Copy link

codecov bot commented Sep 26, 2017

Codecov Report

Merging #120 into master will decrease coverage by 0.76%.
The diff coverage is 66.09%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #120      +/-   ##
==========================================
- Coverage   69.34%   68.58%   -0.77%     
==========================================
  Files          34       34              
  Lines        1233     1254      +21     
  Branches      165      162       -3     
==========================================
+ Hits          855      860       +5     
- Misses        378      394      +16
Impacted Files Coverage Δ
include/goofit/BinnedDataSet.h 100% <ø> (ø) ⬆️
include/goofit/fitting/FitManagerMinuit2.h 100% <ø> (ø) ⬆️
include/goofit/UnbinnedDataSet.h 100% <ø> (ø) ⬆️
include/goofit/fitting/Params.h 100% <ø> (ø) ⬆️
include/goofit/GlobalCudaDefines.h 100% <ø> (ø) ⬆️
include/goofit/PDFs/basic/ExpPdf.h 100% <ø> (ø) ⬆️
include/goofit/PDFs/basic/GaussianPdf.h 100% <ø> (ø) ⬆️
src/goofit/Abort.cc 4.76% <0%> (ø) ⬆️
src/goofit/FCN.cc 63.15% <100%> (ø) ⬆️
src/goofit/FitManagerMinuit2.cc 63.88% <100%> (ø) ⬆️
... and 17 more

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 79d8762...83f8761. Read the comment docs.

@henryiii henryiii added this to the V 2.1 milestone Oct 21, 2017
@henryiii
Copy link
Member

henryiii commented Nov 4, 2017

About to merge.

@henryiii henryiii merged commit 3121d83 into master Nov 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants