Baunsgaard commented on pull request #1200:
URL: https://github.com/apache/systemds/pull/1200#issuecomment-799840317


   Before these changes the parsing of MNIST took ~1.2 sec because of 
inefficient parsing of strings and NaN special handling.
   
   Unfortunately this PRs implementation is not as fast as SystemML 1.0 which 
is able to parse MNIST
   in 0.8 sec on average (i did verify with repeated executions of various 
kinds). I did try to use exactly the old code but in our
   current system this only perform to 1 sec.
   
   
   Current:
   
![image](https://user-images.githubusercontent.com/9947148/111236090-5c2d8900-85f2-11eb-8f81-e7774f88bf54.png)
   
   
![image](https://user-images.githubusercontent.com/9947148/111236076-55067b00-85f2-11eb-933e-b205596ef490.png)
   
   Old:
   
![image](https://user-images.githubusercontent.com/9947148/111236104-62bc0080-85f2-11eb-9f65-08c7537fb0f7.png)
   
   
![image](https://user-images.githubusercontent.com/9947148/111236055-4a4be600-85f2-11eb-994a-0d952fff0c45.png)
   


----------------------------------------------------------------
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:
[email protected]


Reply via email to