-
Notifications
You must be signed in to change notification settings - Fork 59
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
Normalize variabels #451
Comments
@bonh thanks for your post. Could you perhaps share some more details? Are you talking about the It would be great if you could share some code! |
I use The function is quite complex and I'm not able to share it (yet). I'll try to find a MWE. What I noticed was, that in the nonscaled problem, the values chosen by |
At this exact moment I have no time to take a detailed look. However, I from a quick look I am led to believe that the problem is that we're not using the adaptive/adaptive/learner/learnerND.py Lines 110 to 169 in d2c8041
This should be a relatively easy fix. @bonh, unrelated to this issue, how is your experience with sampling a 5D space? Does Adaptive produce good results, better results than random sampling or uniform sampling? Personally, I have not even tried running real simulations beyond 3D, always thinking that "the curse of dimensionally" would bite me. |
That'd explain my observations, thanks! I Just started sampling a 5D space, before that it was 3D, too. My target is to train a surrogate approximating my complex, costly function. However, the function is not that costly that I cannot sample 4000 points in a reasonable time. My guess is, that I would get similar results with different sampling procedures because I'm filling the parameter space very well. So I didn't do a detailed analysis but I think that I require about 30 % less samples with |
I get the feeling that normalizing the variables to be around 1 greatly improves the sampling. Specifically, it prevents the "point is inside the hull" error. Might be worth adding to the tutorial somewhere?
The text was updated successfully, but these errors were encountered: