Skip to content

Commit f9823f8

Browse files
committed
enable density computation with adaptive step size solvers
1 parent adefa7b commit f9823f8

File tree

2 files changed

+0
-25
lines changed

2 files changed

+0
-25
lines changed

bayesflow/networks/diffusion_model/diffusion_model.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -413,12 +413,6 @@ def _forward(
413413
raise ValueError("Stochastic methods are not supported for forward integration.")
414414

415415
if density:
416-
if integrate_kwargs["steps"] == "adaptive":
417-
logging.warning(
418-
"Using adaptive integration for density estimation can lead to "
419-
"problems with autodiff. Switching to 200 fixed steps instead."
420-
)
421-
integrate_kwargs["steps"] = 200
422416

423417
def deltas(time, xz):
424418
v, trace = self._velocity_trace(xz, time=time, conditions=conditions, training=training)
@@ -467,12 +461,6 @@ def _inverse(
467461
if density:
468462
if integrate_kwargs["method"] in STOCHASTIC_METHODS:
469463
raise ValueError("Stochastic methods are not supported for density computation.")
470-
if integrate_kwargs["steps"] == "adaptive":
471-
logging.warning(
472-
"Using adaptive integration for density estimation can lead to "
473-
"problems with autodiff. Switching to 200 fixed steps instead."
474-
)
475-
integrate_kwargs["steps"] = 200
476464

477465
def deltas(time, xz):
478466
v, trace = self._velocity_trace(xz, time=time, conditions=conditions, training=training)

bayesflow/networks/flow_matching/flow_matching.py

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import logging
21
from collections.abc import Sequence
32

43
import keras
@@ -239,12 +238,6 @@ def _forward(
239238
) -> Tensor | tuple[Tensor, Tensor]:
240239
integrate_kwargs = self.integrate_kwargs | kwargs
241240
if density:
242-
if integrate_kwargs["steps"] == "adaptive":
243-
logging.warning(
244-
"Using adaptive integration for density estimation can lead to "
245-
"problems with autodiff. Switching to 200 fixed steps instead."
246-
)
247-
integrate_kwargs["steps"] = 200
248241

249242
def deltas(time, xz):
250243
v, trace = self._velocity_trace(xz, time=time, conditions=conditions, training=training)
@@ -273,12 +266,6 @@ def _inverse(
273266
) -> Tensor | tuple[Tensor, Tensor]:
274267
integrate_kwargs = self.integrate_kwargs | kwargs
275268
if density:
276-
if integrate_kwargs["steps"] == "adaptive":
277-
logging.warning(
278-
"Using adaptive integration for density estimation can lead to "
279-
"problems with autodiff. Switching to 200 fixed steps instead."
280-
)
281-
integrate_kwargs["steps"] = 200
282269

283270
def deltas(time, xz):
284271
v, trace = self._velocity_trace(xz, time=time, conditions=conditions, training=training)

0 commit comments

Comments
 (0)