ContrastiveSampler lacks a seed param, while the docstring states it has one #1481
Labels
type:Bug
Something isn't working
Comments
|
Yep, just a docs issue. Will fix up. |
mattdangerw
added a commit
to mattdangerw/keras-hub
that referenced
this issue
Mar 5, 2024
mattdangerw
added a commit
that referenced
this issue
Mar 5, 2024
abuelnasr0
pushed a commit
to abuelnasr0/keras-nlp
that referenced
this issue
Apr 2, 2024
Tusespifump1o
added a commit
to Tusespifump1o/keras-nlp
that referenced
this issue
Aug 26, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Contrastive samples docs here:
https://keras.io/api/keras_nlp/samplers/contrastive_sampler/
The function signature does not have a "seed" arg:
keras_nlp.samplers.ContrastiveSampler(k=5, alpha=0.6, **kwargs)But the docstring states:
seed: int. The random seed. Defaults to None.
Probably just a docstring issue. Since this samples is a variant of BeamSampler, it probably does not need a seed.
The text was updated successfully, but these errors were encountered: