snuspl / nemo

Nemo: A flexible data processing system
https://snuspl.github.io/nemo/
Apache License 2.0
22 stars 6 forks source link

[Onyx-748,749] Fix ALS and related passes/tests #750

Closed johnyangk closed 6 years ago

johnyangk commented 6 years ago

Before: Only check if the source is from reserved or not After: Also check if the dependency is O2O, and change the ALS example to exploit the Pado technique

Closes #748 Closes #749

johnyangk commented 6 years ago

Apparently, the tests depend on both the app and the pass, so I've changed the PR to address both issues at once.

This is the ALS DAG after the change. Computations are placed on transient/reserved resources in an 'alternating' manner. 😄 http://34.215.106.107/onyx-dag-out/6bfba8e264414f4f89fce8e3c6535506.svg