-
Notifications
You must be signed in to change notification settings - Fork 0
Add a simple getting started notebook #129
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
am-kaiser
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very good job. I have a couple of comments but many of them are just small changes.
| # 1. Runs the simulator `N_SIMULATIONS_PER_POINT` times at each training point `x` | ||
| # 2. Fits the Gumbel distribution to those samples to estimate `loc(x)` and `scale(x)` | ||
| # 3. Provides these fitted parameters to the GP | ||
| # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make_experiment does not actually run anything. It is just a wrapper to simplify the steps you mention in later stages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it better now?
| # 4. Extracts the **maximum** response in each period | ||
| # 5. Estimates the **median** of the resulting extreme response distribution | ||
| # 6. Repeats under different draws from the GP posterior to capture model uncertainty | ||
| # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sounds a bit like MarginalCDFExtrapolation is basically brute force just with the surrogate model. That is not quite correct (GPBruteForce is doing that more or less). The procedure for MarginalCDFExtrapolation is:
- from the GP get the posterior distributions at the specified env. points
- draw samples from posterior distributions
- define Gumbel distribution using loc and scale
- using inverse CDF we can then calculate QoI for each posterior sample
This is just an explanation for you but maybe a bit too detailed for such a tutorial. While the batching is done and has some effects. It is maybe not super important at this level of abstraction
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the explanation. I tried to simplify the text for this tutorial.
|
I have addressed all the review comments. Please take another look! Thanks! |
Add getting started notebook for Axtreme (a simplification of the basic example).