Baunsgaard opened a new pull request, #2158:
URL: https://github.com/apache/systemds/pull/2158

   This commit increases the default estimate of frame size. Previously, frames 
were estimated similarly to Matrices. The wrong estimate leads to problems on 
frames of > Integer rows. To improve it, this commit defaults to 8 character 
strings on all cells. In an unread matrix.
   
   Since there is no way of knowing if the input Frame contains longer strings, 
it is still a subpar estimate. However,
   it is an improvement overestimating everything as a dense double Matrix.
   
   (The change happened because I encountered very incorrect estimates in 
BEWARE)


-- 
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.

To unsubscribe, e-mail: dev-unsubscr...@systemds.apache.org

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

Reply via email to