moose
moose copied to clipboard
idaholab
Parallel active learning in Monte Carlo simulations with Gaussian Process
To do :
- [ ] Error checking and tests
- [x] Enum options for active learning acquisition functions
- [ ] Documentation
closes #22280
Please do not review this yet. I need to make a few more additions/changes. (wondering whether I can add labels...)
You cant add labels sorry. Which label were you thinking of adding?
Maybe "do not review" or "do not merge"? But if I cannot remove them, I'd prefer not having them.
You would not be able to remove them but also we would do it for you and doubt it would become an issue
You can also turn your PR to a draft if you want
You would not be able to remove them but also we would do it for you and doubt it would become an issue
You can also turn your PR to a draft if you want
Draft is a great idea!
Job Documentation on 5f25b5b wanted to post the following:
View the site here
This comment will be updated on new commits.
@zachmprince @grmnptr This is ready for review. I am only fixing some minor doc issues.
Job Precheck on d4faaf6 wanted to post the following:
Your code requires style changes.
A patch was auto generated and copied here
You can directly apply the patch by running, in the top level of your repository:
curl -s https://mooseframework.inl.gov/docs/PRs/22282/clang_format/style.patch | git apply -v
Alternatively, with your repository up to date and in the top level of your repository:
git clang-format 88ca6d4e7b9e91717f598187ebe561e1e18f9581
Job Coverage on 5f25b5b wanted to post the following:
Framework coverage
| e40a32 | #22282 5f25b5 | ||||
|---|---|---|---|---|---|
| Total | Total | +/- | New | ||
| Rate | 84.62% | 84.62% | -0.00% | - | |
| Hits | 81970 | 81966 | -4 | 0 | |
| Misses | 14897 | 14901 | +4 | 0 | |
Modules coverage
Stochastic tools
| e40a32 | #22282 5f25b5 | ||||
|---|---|---|---|---|---|
| Total | Total | +/- | New | ||
| Rate | 91.17% | 91.29% | +0.11% | 95.11% | |
| Hits | 6073 | 6243 | +170 | 175 | |
| Misses | 588 | 596 | +8 | 9 | |
Full coverage reports
Reports
-
framework -
chemical_reactions -
combined -
contact -
electromagnetics -
external_petsc_solver -
fluid_properties -
fsi -
functional_expansion_tools -
geochemistry -
heat_conduction -
level_set -
misc -
navier_stokes -
optimization -
peridynamics -
phase_field -
porous_flow -
ray_tracing -
rdg -
reactor -
richards -
scalar_transport -
solid_properties -
stochastic_tools -
tensor_mechanics -
thermal_hydraulics -
xfem
This comment will be updated on new commits.
Probably just a network error causing the test harness to crash after. It seems invalidating it fixed it
Thanks, @zachmprince for the comments. For those comments, I had issues or questions, I will directly responded to the specific comment.
Job Precheck on f6f08ba wanted to post the following:
Your code requires style changes.
A patch was auto generated and copied here
You can directly apply the patch by running, in the top level of your repository:
curl -s https://mooseframework.inl.gov/docs/PRs/22282/clang_format/style.patch | git apply -v
Alternatively, with your repository up to date and in the top level of your repository:
git clang-format af38e9f0b6976df8d0a90b1c6b339bef1dbc9b00