[GitHub] [systemml] phaniarnab opened a new pull request #897: [MINOR]New partial rewrites and bugfixes

[GitBox <gi...@apache.org>]

phaniarnab opened a new pull request #897:
URL: https://github.com/apache/systemml/pull/897


   This patch contains
   - two new partial rewrites which are specialization of existing
   rewrites,
   - bugfixes and optimizations in partial rewrites,
   - lineage tracing for removeEmpty,
   - a new test class to test algorithms and builtins with reuse,
   - extension of lineage tracing of list objects.
   Note that lineage doesn't work with most of the list handling methods
   today. Due to that the current generalized grid search builtin is far
   from working with lineage framework.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [systemml] mboehm7 commented on pull request #897: [MINOR]New partial rewrites and bugfixes

[GitBox <gi...@apache.org>]

mboehm7 commented on pull request #897:
URL: https://github.com/apache/systemml/pull/897#issuecomment-622088512


   LGTM - thanks @phaniarnab for the various improvements. During the merge, I just fixed the debug logging in `LineageRewriteReuse` to use the proper logging infrastructure.
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org