[
https://issues.apache.org/jira/browse/MAHOUT-220?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12795551#action_12795551
]
Ted Dunning commented on MAHOUT-220:
------------------------------------
{quote}
FWIW, I'd say stuff that converts text, etc. to our internal representations
belongs in the Utils module, where all the "helper" classes are.
{quote}
That is what I would have liked to do. We have some problems with dependencies.
Utils depends on core which depends on math. But the classifier stuff depends
on vectorization and is in core. That means that either the classifier stuff
has to be moved to utils (or a new module before utils) or vectorization moved
down. Math seems like a nice home for vectorization, but some kinds need
random numbers and RandomUtils is in core.
Bleah.
My final answer is to put the vectorization in core, partly because I am too
lazy to re-imagine the entire module structure. This does raise the question
of whether our modules are serving our needs or we are serving theirs.
> Mahout Bayes Code cleanup
> -------------------------
>
> Key: MAHOUT-220
> URL: https://issues.apache.org/jira/browse/MAHOUT-220
> Project: Mahout
> Issue Type: Improvement
> Components: Classification
> Affects Versions: 0.3
> Reporter: Robin Anil
> Assignee: Robin Anil
> Fix For: 0.3
>
> Attachments: MAHOUT-BAYES.patch, MAHOUT-BAYES.patch
>
>
> Following isabel's checkstyle, I am adding a whole slew of code cleanup with
> the following exceptions
> 1. Line length used is 120 instead of 80.
> 2. static final log is kept as is. not LOG.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
