Closed ledell closed 7 years ago
Hi, great to hear that it was useful! I only just saw this issue (have not really been monitoring GitHub in my last job) and presume you have it all sorted now.
@ellisp We added some of your code to H2O here. I think there was a bug in it that got fixed as well. Thanks again!
Thank, Peter. All sorted. I ended up not using the NZIS SURF data in the conference presentation, but the acknowledgements are in my code in case I ever change my mind.
Ross
On 25 July 2017 at 07:46, Peter Ellis notifications@github.com wrote:
Hi, great to hear that it was useful! I only just saw this issue (have not really been monitoring GitHub in my last job) and presume you have it all sorted now.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ellisp/ellisp.github.io/issues/40#issuecomment-317564429, or mute the thread https://github.com/notifications/unsubscribe-auth/AFKJGy90CZG9xKMiW2XmDfzG52NNZtxXks5sRRCYgaJpZM4OHUfA .
Hi Peter, We have had some complaints that
h2o::as.h2o()
is very slow for sparse matrices and so we came across your awesome blog post on the topic. We are planning to implement your solution (unless you wanted to submit a PR directly).Thanks! -Erin