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

Added some docs for TS's options #66

Merged
merged 1 commit into from
Apr 26, 2016
Merged

Added some docs for TS's options #66

merged 1 commit into from
Apr 26, 2016

Conversation

kshyatt
Copy link
Contributor

@kshyatt kshyatt commented Apr 25, 2016

No description provided.

@codecov-io
Copy link

codecov-io commented Apr 25, 2016

Current coverage is 7.23%

Merging #66 into master will increase coverage by +<.01%

@@           master       #66   diff @@
=======================================
  Files          22        22          
  Lines       21803     21803          
  Methods         0         0          
  Branches        0         0          
=======================================
  Hits         1577      1577          
  Misses      20226     20226          
  Partials        0         0          

Sunburst

Powered by Codecov. Last updated by 4e34587

@JaredCrean2
Copy link
Contributor

I'm not sure we should be documenting PETSc options as part of the julia wrappers. Having 2 sets of documentation (Petsc's and ours) makes inconsistencies much more likely.

@kshyatt
Copy link
Contributor Author

kshyatt commented Apr 26, 2016

I added this mostly for consistency with KSP, since we have a big opts table there. I can shrink it down with a link to the relevant PETSc webpage?

@JaredCrean2
Copy link
Contributor

Sounds good

@kshyatt
Copy link
Contributor Author

kshyatt commented Apr 26, 2016

Better?

@JaredCrean2
Copy link
Contributor

Build failure looks unrelated. Merging.

@JaredCrean2 JaredCrean2 merged commit 1ad5c5c into master Apr 26, 2016
@kshyatt kshyatt deleted the ksh/docts branch April 26, 2016 22:43
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.

None yet

3 participants