-
Notifications
You must be signed in to change notification settings - Fork 17
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
Disable modelselection for some cases #58
Comments
@karlnapf The major problem still is using modelselection in web-demos on large dataset and various combination arising. |
I agree, there should maybe be a timeout function? Any ideas how to restrict larger datasets? |
hmm I could set up client side timeouts but they won't help afaik and we have no idea about server stuff this is going to run on. xval can be done but in similar way, exposing potential big computations through the interface will lead to problems, no? about restricting datasets, maybe just check num of samples (say < 100 for demo purposes) ?? |
yes you are right, exposing potantial big computation through this interface is critical, but i think we can restrict things in exacly that way, only allow for small number of samples. If there are more points, an error pops up saying "Model selection only supported for less than N samples due to computational costs) |
That is a good idea in principle, but we currently dont have ressources to think about this. If you wanna come up with something, feel free, but you would be on your own. |
No description provided.
The text was updated successfully, but these errors were encountered: