[
https://issues.apache.org/jira/browse/MAHOUT-165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12779068#action_12779068
]
Jake Mannix commented on MAHOUT-165:
------------------------------------
bq. Should all code be marked deprecated until it has unit tests with a comment
to say why it is deprecated? That would give us a clear visual signal that the
lifeguard has left the pool.
+1
Sounds like a good plan.
So if there's going to be another submodule, for colt, I'm not sure if I'm
qualified to do that - I'm not a maven expert, so writing a new pom I could
(and should, and will!) learn how to do, but it would slow down this process.
Is it ok if I post a patch which has the org.apache.mahout.colt.* code just
living inside of core/source/main/java, and then someone else can help out by
refactoring my patch to have the right build / maven setup? Any volunteers for
that?
> Using better primitives hash for sparse vector for performance gains
> --------------------------------------------------------------------
>
> Key: MAHOUT-165
> URL: https://issues.apache.org/jira/browse/MAHOUT-165
> Project: Mahout
> Issue Type: Improvement
> Components: Matrix
> Affects Versions: 0.2
> Reporter: Shashikant Kore
> Assignee: Grant Ingersoll
> Fix For: 0.3
>
> Attachments: colt.jar, mahout-165-trove.patch,
> MAHOUT-165-updated.patch, mahout-165.patch, MAHOUT-165.patch, mahout-165.patch
>
>
> In SparseVector, we need primitives hash map for index and values. The
> present implementation of this hash map is not as efficient as some of the
> other implementations in non-Apache projects.
> In an experiment, I found that, for get/set operations, the primitive hash of
> Colt performance an order of magnitude better than OrderedIntDoubleMapping.
> For iteration it is 2x slower, though.
> Using Colt in Sparsevector improved performance of canopy generation. For an
> experimental dataset, the current implementation takes 50 minutes. Using
> Colt, reduces this duration to 19-20 minutes. That's 60% reduction in the
> delay.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
