Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Samplers order parameters #4285

Merged
merged 13 commits into from
Dec 5, 2023
Prev Previous commit
Next Next commit
Cleaned commented code
  • Loading branch information
MaggotHATE committed Dec 1, 2023
commit 7601a49130bc18540edb86ca9550647d6283fbec
7 changes: 0 additions & 7 deletions common/sampling.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -225,13 +225,6 @@ llama_token llama_sampling_sample(
// temperature sampling
size_t min_keep = std::max(1, params.n_probs);

// llama_sample_top_k (ctx_main, &cur_p, top_k, min_keep);
// llama_sample_tail_free(ctx_main, &cur_p, tfs_z, min_keep);
// llama_sample_typical (ctx_main, &cur_p, typical_p, min_keep);
// llama_sample_top_p (ctx_main, &cur_p, top_p, min_keep);
// llama_sample_min_p (ctx_main, &cur_p, min_p, min_keep);
// llama_sample_temp (ctx_main, &cur_p, temp);

for (auto s : samplers_sequence){
switch (s){
case 'k':{
Expand Down