hivemall-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From myui <...@git.apache.org>
Subject [GitHub] incubator-hivemall pull request #121: [WIP][HIVEMALL-151] Support Matrix con...
Date Thu, 05 Oct 2017 11:31:52 GMT
GitHub user myui opened a pull request:

    https://github.com/apache/incubator-hivemall/pull/121

    [WIP][HIVEMALL-151] Support Matrix conversion from DoK to CSR/CSC matrix

    ## What changes were proposed in this pull request?
    
    - Support Matrix conversion from DoK to CSR/CSC matrix
    - Introduce FloatMatrix (dumped supported Java version from Java 7 to Java 8!)
    - Revise SLIM implementation to use CSRMatrix converted from DoKMatrix for [this issue](https://github.com/apache/incubator-hivemall/pull/117#pullrequestreview-65179017)
    
    ## What type of PR is it?
    
    Improvement,  Refactoring
    
    ## What is the Jira issue?
    
    https://issues.apache.org/jira/browse/HIVEMALL-151
    
    ## How was this patch tested?
    
    unit tests, manual tests
    
    ## Checklist
    
    (Please remove this section if not needed; check `x` for YES, blank for NO)
    
    - [x] Did you apply source code formatter, i.e., `mvn formatter:format`, for your commit?
    - [ ] Did you run system tests on Hive (or Spark)?


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/myui/incubator-hivemall HIVEMALL-151

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-hivemall/pull/121.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #121
    
----
commit 6240b09780de0a8bc79316cbe39ec19275a6d043
Author: Makoto Yui <myui@apache.org>
Date:   2017-10-05T11:27:19Z

    Supported to_csr/csc in DokMatrix, added FloatMatrix, and bumped up supported Java version
from 1.7 to 1.8

----


---

Mime
View raw message