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

Movement issue39 update flux function interfaces #56

Conversation

KathrinTessella
Copy link
Contributor

Updated the tests and documentation accordingly. Perhaps we want a better description for the 'Arguments' in the documentation?

@KathrinTessella
Copy link
Contributor Author

@nick: any suggestion how to improve the argument lists in the documentation? Perhaps that could be done together with issue #52 ?

@goldingn
Copy link
Member

Yup, I think that's a good idea - a few words to describe the role of each parameter

@codecov-io
Copy link

Current coverage is 69.89%

Merging #56 into master will increase coverage by +0.39% as of b8ee14e

@@            master     #56   diff @@
======================================
  Files            1       1       
  Stmts          469     475     +6
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit            326     332     +6
  Partial          0       0       
  Missed         143     143       

Review entire Coverage Diff as of b8ee14e

Powered by Codecov. Updated on successful CI builds.

KathrinTessella added a commit that referenced this pull request Oct 21, 2015
…lux-defs

Movement issue39: update flux function interfaces to allow to update the various parameters separately.
@KathrinTessella KathrinTessella merged commit 5f6c3ec into SEEG-Oxford:master Oct 21, 2015
@KathrinTessella KathrinTessella deleted the movement-issue39-update-flux-defs branch October 21, 2015 07:12
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