Pull Request Review Events

johnwparent reviewed a spack/spack pull request

I’m confused how this hasn’t hit more folks considering normpath converts // -> / which should break every http based url. …

View Review

alalazo reviewed a spack/spack pull request

None

View Review

Copilot reviewed a spack/spack pull request

## Pull Request Overview…

View Review

trws reviewed a flux-framework/flux-sched pull request

I went over, built and tested this. It seems like a good change to me. @milroy any concerns?…

View Review

garlick reviewed a flux-framework/flux-core pull request

LGTM!…

View Review

JustinPrivitera reviewed a visit-dav/visit pull request

None

View Review

tomstitt reviewed a LLNL/RAJA pull request

None

View Review

adayton1 reviewed a LLNL/CHAI pull request

None

View Review

thartland reviewed a LLNL/hiop pull request

Great addition! I have tested it on another application code. In a separate PR I will add the MPIEvaluator wrapper of the EvaluationManager and utilize it for various tasks in a BO loop….

View Review

doutriaux1 reviewed a LLNL/merlin pull request

None

View Review

davidbeckingsale reviewed a LLNL/blt pull request

None

View Review

dtaller reviewed a LLNL/CARE pull request

None

View Review

dtaller reviewed a LLNL/CHAI pull request

None

View Review