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

DOC: TensorModel.__init__ docstring. #1080

Merged
merged 1 commit into from Jun 23, 2016

Conversation

Projects
None yet
4 participants
@arokem
Member

arokem commented Jun 17, 2016

No description provided.

@arokem arokem referenced this pull request Jun 17, 2016

Merged

NF: Free water tensor model #835

@coveralls

This comment has been minimized.

coveralls commented Jun 17, 2016

Coverage Status

Coverage remained the same at 82.88% when pulling cf9b33a on tensormodel-docstring into 82573c7 on master.

@codecov-io

This comment has been minimized.

codecov-io commented Jun 17, 2016

Current coverage is 80.81%

Merging #1080 into master will not change coverage

@@             master      #1080   diff @@
==========================================
  Files           200        200          
  Lines         22985      22985          
  Methods           0          0          
  Messages          0          0          
  Branches       2309       2309          
==========================================
  Hits          18576      18576          
  Misses         3933       3933          
  Partials        476        476          

Powered by Codecov. Last updated by 82573c7...cf9b33a

@jchoude

This comment has been minimized.

Contributor

jchoude commented Jun 20, 2016

Seems good to me. Can merge in a few days if no one says anything.

@jchoude

This comment has been minimized.

Contributor

jchoude commented Jun 22, 2016

No one wants to chime in? I know this is a benign change.

@arokem

This comment has been minimized.

Member

arokem commented Jun 23, 2016

🚢

@jchoude jchoude merged commit 2836d60 into master Jun 23, 2016

5 checks passed

codecov/patch 100% of diff hit (target 80.81%)
Details
codecov/project 80.81% (+0.00%) compared to 82573c7
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 82.88%
Details
@jchoude

This comment has been minimized.

Contributor

jchoude commented Jun 23, 2016

Thanks @arokem (and yeah! for the first PR that I merge)

@arokem

This comment has been minimized.

Member

arokem commented Jun 23, 2016

Congratulations! Here's to many more!
On Jun 22, 2016 7:02 PM, "Jean-Christophe Houde" notifications@github.com
wrote:

Thanks @arokem https://github.com/arokem (and yeah! for the first PR
that I merge)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1080 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAHPNqsmGxuvXFUst6l4NCzj2Cbiz1Ymks5qOekygaJpZM4I4isb
.

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