search

zavtech / morpheus-core

The foundational library of the Morpheus data science framework
Apache License 2.0
238 stars 22 forks source link

Switch from trove to fastutil #54

Open Zavster opened 7 years ago

benmccann commented 6 years ago

@Zavster I'd love to see this PR make its way in after all the work that's been done on it already! :-)

benmccann commented 6 years ago

@Zavster any reason not to merge this?

Zavster commented 6 years ago

Hey Ben, I wanted to assess using Goldman Sachs collection as a possible alternative, but have been working on other parts of Morpheus lately. I will definitely adopt either GS collections or Fastutil, hopefully before year end.

https://github.com/goldmansachs/gs-collections

benmccann commented 6 years ago

Any thoughts on this one?

benmccann commented 6 years ago

@Zavster think you might still merge this?

Zavster commented 6 years ago

Hey Ben, I am sitting out a non-compete which ends in early September. I will resume Morpheus development in anger then....

benmccann commented 6 years ago

Good luck with the new gig!

I've been contributing quite a bit to Tablesaw in the meantime. I'm not quite sure what it'd look like, but it'd be cool to join forces on the libraries somehow