Skip to content

Commit

Permalink
improve comment
Browse files Browse the repository at this point in the history
  • Loading branch information
lxvm committed Nov 3, 2023
1 parent 1efb0ee commit aa15339
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Integrals.jl
Original file line number Diff line number Diff line change
Expand Up @@ -135,9 +135,9 @@ function __solvebp_call(prob::IntegralProblem, alg::VEGAS, sensealg, domain, p;
y = similar(prob.f.integrand_prototype,
size(prob.f.integrand_prototype)[begin:(end - 1)]...,
prob.f.max_batch)
# MonteCarloIntegration v0.0.x passes batches as columns of a matrix
# MonteCarloIntegration v0.0.x passes points as rows of a matrix
# MonteCarloIntegration v0.1 passes batches as a vector of views of
# a matrix with batches as rows of a matrix
# a matrix with points as columns of a matrix
# see https://github.com/ranjanan/MonteCarloIntegration.jl/issues/16
# This is an ugly hack that is compatible with both
f = x -> (prob.f(y, eltype(x) <: SubArray ? parent(first(x)) : x', p); vec(y))

Check warning on line 143 in src/Integrals.jl

View check run for this annotation

Codecov / codecov/patch

src/Integrals.jl#L143

Added line #L143 was not covered by tests
Expand Down

0 comments on commit aa15339

Please sign in to comment.