Skip to content

Commit

Permalink
BUG: Fix default for post_tfm_kwargs, fixes #697 (#710)
Browse files Browse the repository at this point in the history
* BUG: fix default for post_tfm_kwargs, fixes Inconsistent syllable error rate between vak eval and predict #697

---------

Co-authored-by: zz367 <zz367@cornell.edu>
  • Loading branch information
zhileiz1992 and zhileiz367 committed Oct 2, 2023
1 parent bb410ed commit a783922
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/vak/config/eval.py
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ class EvalConfig:
post_tfm_kwargs = attr.ib(
validator=validators.optional(are_valid_post_tfm_kwargs),
converter=converters.optional(convert_post_tfm_kwargs),
default={}, # empty dict so we can pass into transform with **kwargs expansion
default=None, # empty dict so we can pass into transform with **kwargs expansion
)

# optional, data loader
Expand Down
2 changes: 1 addition & 1 deletion src/vak/config/learncurve.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,5 +74,5 @@ class LearncurveConfig(TrainConfig):
post_tfm_kwargs = attr.ib(
validator=validators.optional(are_valid_post_tfm_kwargs),
converter=converters.optional(convert_post_tfm_kwargs),
default={}, # empty dict so we can pass into transform with **kwargs expansion
default=None, # empty dict so we can pass into transform with **kwargs expansion
)

0 comments on commit a783922

Please sign in to comment.