Skip to content

Commit 14ef30f

Browse files
authored
Fix typo
L22: Setting `learn_theta=true` (before was something else)
1 parent 6ef85f6 commit 14ef30f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/models/sequence/ss/s4_simple/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,10 @@ By default, the kernel ignores the initial state (fusing `b` and `c`), and only
2020
You can play with these parameters in the training run:
2121
* Adding `use_initial=true` will add a learnable initial state, and learn the `b` and `c` parameters separately.
2222
* Setting `learn_theta=true` will make the `theta` parameters learnable (we usually see a decrease in performance of about 3 points from this).
23-
* Setting `leran_a=false` will make the `a` parameters not learnable. We don't see much of a performance degradation on CIFAR in this case, which speaks to the utility of the Chebyshev initialization!
23+
* Setting `learn_a=false` will make the `a` parameters not learnable. We don't see much of a performance degradation on CIFAR in this case, which speaks to the utility of the Chebyshev initialization!
2424
* Setting `zero_order_hold=false` will switch from Zero-Order Hold to left-end-point quadrature. Additionally setting `trap_rule=true` will switch to the trapezoid rule (when `zero_order_hold` is set to `false`).
2525

2626
## Other Files
2727

2828
There are two other files in this folder, `s4_wrapper.py` and `utils.py`.
29-
They contain some standard wrapper classes and utils for integrating into the state spaces code base.
29+
They contain some standard wrapper classes and utils for integrating into the state spaces code base.

0 commit comments

Comments
 (0)