Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

ArnoldiMethod conversion #1008

Merged
merged 10 commits into from
Sep 9, 2018
Merged

ArnoldiMethod conversion #1008

merged 10 commits into from
Sep 9, 2018

Conversation

jpfairbanks
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Sep 9, 2018

Codecov Report

Merging #1008 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1008      +/-   ##
==========================================
+ Coverage   99.85%   99.85%   +<.01%     
==========================================
  Files          79       80       +1     
  Lines        2677     2689      +12     
==========================================
+ Hits         2673     2685      +12     
  Misses          4        4

@jpfairbanks jpfairbanks changed the title [wip]: ArnoldiMethod conversion ArnoldiMethod conversion Sep 9, 2018
@jpfairbanks
Copy link
Contributor Author

This is ready for review. The only things to hold up over would be https://github.com/haampie/ArnoldiMethod.jl/issues/69 which would move our eigs code into ArnoldiMethod.jl so we don't have to export eigs.

@sbromberger
Copy link
Owner

@jpfairbanks it's now passing tests. Ship it, or is it just luck that normalized_cut is working? :)

@sbromberger sbromberger merged commit 6ee63a9 into master Sep 9, 2018
@sbromberger sbromberger deleted the jpf/iram branch September 9, 2018 23:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants