mboehm7 commented on pull request #1550: URL: https://github.com/apache/systemds/pull/1550#issuecomment-1059836817
This looks generally good to me @Shafaq-Siddiqi and having this hierarchical seed management is very nice. However, could you please create a new PR that only contains your new commit. When I tried rebasing this PR, your "forced push" made the commit appear somewhere in the middle which could potentially mess up the history. Once this is in, I would then help run a few experiments to see how to improve the runtime of this cleaning pipeline enumeration. Thanks. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
