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

separate brushfire-tree from brushfire-training #51

Open
avibryant opened this issue May 18, 2015 · 2 comments
Open

separate brushfire-tree from brushfire-training #51

avibryant opened this issue May 18, 2015 · 2 comments

Comments

@avibryant
Copy link
Contributor

Some applications, like model servers, will not need the training code. It would be nice if they could depend on a smaller subset of the code. Likely, this would include:

  • Tree
  • Predicate
  • Voter
  • the serialization Injections
  • Dispatched

but hopefully would not include:

  • Instance
  • Splitter
  • Error
  • Sampler
@tixxit
Copy link
Contributor

tixxit commented May 18, 2015

Something to think about while doing this, is adding a case class Forest(trees: Map[Int, Tree[K, V, T]) in brushfire-training while we're at it. A lot of helper methods could be added to this to make the creation of new types of Trainers easier.

@avibryant
Copy link
Contributor Author

Forest is probably useful in brushfire-trees too, right? Then I guess we could have an implicit ForestTrainingHelpers...

avibryant pushed a commit that referenced this issue Apr 28, 2016
)

Factor out trees and serialization to support evaluation without training
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants