Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jul 12 2021 17:27
    hybras opened #219
  • May 29 2021 03:48
    digantamisra98 closed #212
  • Mar 08 2021 15:15
    AtheMathmo closed #218
  • Mar 08 2021 15:15
    AtheMathmo commented #218
  • Feb 23 2021 15:24
    asad-awadia opened #218
  • Dec 20 2020 12:15
    yutiansut opened #217
  • Jul 10 2020 17:17
    alfaevc opened #216
  • Jul 10 2020 17:13
    alfaevc opened #215
  • Jul 10 2020 17:10
    alfaevc opened #214
  • Jan 29 2020 23:01
    sixilli synchronize #213
  • Jan 29 2020 02:39
    sixilli edited #213
  • Jan 29 2020 02:29
    sixilli synchronize #213
  • Jan 29 2020 01:39
    sixilli opened #213
  • Jan 21 2020 16:00
    DenialAdams synchronize #205
  • Jan 17 2020 07:19
    digantamisra98 opened #212
  • Jan 12 2020 10:58
    LukasKalbertodt commented #208
  • Dec 26 2019 11:08
    flip111 opened #211
  • Dec 25 2019 20:09
    flip111 opened #210
  • Dec 18 2019 02:47
    AtheMathmo commented #206
  • Dec 18 2019 02:44
    AtheMathmo closed #209
James Sutton
@zeryx
my serialization method was only done that way as a quick hack, rustc-serialize or serde is far superior
essentially just making all the structs rustcencodable and rustcdecodable will do it, super simple with rustc-serialize
James Sutton
@zeryx
heh I'm new with PRs @AtheMathmo, I feel like lots of those changes shouldn't have been actually within the PR, but I'm assuming when doing fork pull reqs it always goes to the master?
James Lucas
@AtheMathmo
You should be able to specify which branch from and to. In this case the PR is master -> master
James Sutton
@zeryx
good comments / suggestions though, I'll fix that up tonight
James Lucas
@AtheMathmo
My GitFu is pretty weak, but you should be able to make a new branch and rebase master, then make the PR from the new branch onto my master
that way you can keep whichever changes you want out of the PR
James Sutton
@zeryx
nice, yeah I'm good at regular ol commits & merges, PR's are a new thing for me (I don't work in the backend team ofc heh)
rad I'll do that
James Lucas
@AtheMathmo
thanks :) however you're comfortable doing it is best, like I say I'm fairly clueless with a lot of it...
I've started to get through my backlog of non-rust things a little more. Hopefully that means I'll have some time to really dig into the project again soon :)
James Sutton
@zeryx
:smile: yeah AtheMathmo/rusty-machine#155 isn't blocking me at a grand scale, just blocking the clustering tool from working
we're clustering just to make KNN faster over a huge dataset so it's not 100% necessary at the moment
hierarchical clustering lets you do neat things like partition datasets when searching for recommendations/etc
James Lucas
@AtheMathmo

@andrewcsmith have you had a chance to see my comments on #155 ?

I'm taking another look now to try and find the regression

Zack M. Davis
@zackmdavis
@AtheMathmo dunno if you already saw, but it looks like someone scooped you on integrating with LAPACK: https://github.com/masonium/linxal
James Lucas
@AtheMathmo
ha I did see! I'm glad someone did it - truly it's not been a hugely motivating goal for me personally and I haven't really got a good idea of how it should look in rulinalg.
Sean Martin
@phasedchirp
@AtheMathmo re: #145 After trying out some tests and benchmarks for the QR-based lin_reg training method I had, this implementation at least is probably not worth the speed trade-off (the cases where it makes a noticeable difference in parameters aren't super-realistic and it's much slower).
Sean Martin
@phasedchirp
Although it should be noted that some large portion of the slow-down is likely due to my implementation, since the theoretical differences are not nearly so large.
James Lucas
@AtheMathmo
Hey @phasedchirp sorry that I missed this message.
It's possible that your approach has some stability advantages that might be worth looking into. I'm not too sure personally and would need to do some reading I imagine
Sean Martin
@phasedchirp
@AtheMathmo it has some minor stability advantages in terms of the stability of the solution informal tests, but only in the really artificial case that I tried.
James Lucas
@AtheMathmo
I'll try to find some time to take a look through the code this weekend. We might be able to close the performance gap a little
James Sutton
@zeryx
hey @AtheMathmo sorry about the PR being on hold forever, we got pretty swamped for the past few months at work and changed gears, will look at closing it sometime this/next week.
James Lucas
@AtheMathmo
@zeryx - no worries at all! I've also had a couple very busy months and had to put a lot of things on hold myself. Take your time :)
John
@onFireForGod_gitlab
I am training a svm, and I get pretty horribly results. Below are the results of actual and predicted. The actual is on the bottom.
Screen Shot 2019-03-19 at 10.34.08 AM.pdf
I was thinking to use GP
any suggestions?
this is a time series data.
John
@onFireForGod_gitlab
Also how can models be saved?
YMoukeddar
@YMoukeddar
Hello all, i have question, it's kinda of a blocker for me
i want to use NB classifier
but the problem is that i have a huge set, 2.3M document and 300k feature, it's stored in a sparse matrix, but the classifier only accepts dense matrix
Alexandre Janniaux
@alexandre-janniaux
Hi :)