-
Notifications
You must be signed in to change notification settings - Fork 213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Training two samples take ~480 seconds #574
Comments
RMTC seems to scale poorly with the I am not familiar with the RMTC model and its implementation, maybe @hwangjt could give some insights on this topic? |
Ok, at the suggestion of @relf , I have created a pytest benchmark that reproduces this issue. First, the here is the runtime as a function of the number of samples; linear scaling: Now I fix the number of samples at 500, dimension at 3, and then sweep the From this figure it is unclear whether the scaling is quadratic or exponential, but a log-log scale reveals it is a power law: The scaling with dimension is clearly exponential, and perhaps a little worse than that: Code to reproduce:
Command to reproduce:
|
The documentation for RMTC uses `num_elements=20`. In Issue SMTorg#574, it is shown that the training time is exponential in `num_elements`. Hence using copy-paste from the example can blow up the compute time. Reduce `num_elements` in the example so that users are less likely to hit this exponential behavior.
Thanks for the thorough analysis! |
* Document example with less expensive parameters The documentation for RMTC uses `num_elements=20`. In Issue #574, it is shown that the training time is exponential in `num_elements`. Hence using copy-paste from the example can blow up the compute time. Reduce `num_elements` in the example so that users are less likely to hit this exponential behavior. * Change the example Python source accordingly --------- Co-authored-by: relf <[email protected]>
Code to reproduce:
Result:
Snakeviz profile screenshotted:
Profile is attached (I had to add a superfluous
.txt
extension to get github to allow the upload).rmtc.profile.txt
The text was updated successfully, but these errors were encountered: