feat(gpu): add 1_1 classical pbs params for specialized version

[guillermo-oyarzun]

closes: please link all relevant issues

PR content/description

Add a new set of params to match the specialised classical on GPU on gaussian. We don't want to replace the current ones, cause that would affect the CPU

Check-list:

• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
• Relevant issues are marked as resolved/closed, related issues are linked in the description
• Check for breaking changes (including serialization changes) and add them to commit message following the conventional commit specification

---

This change is 

[agnesLeroy]

Thank you very much @guillermo-oyarzun! Some comments, also the core crypto benchmarks with 1_1 parameters should use this parameter set for GPU 🙏

[guillermo-oyarzun]

Thank you very much @guillermo-oyarzun! Some comments, also the core crypto benchmarks with 1_1 parameters should use this parameter set for GPU 🙏

I think in the way i did it it should also be using them cause i changed the bench_params alias, i will double check

[agnesLeroy]

Thank you very much @guillermo-oyarzun! Some comments, also the core crypto benchmarks with 1_1 parameters should use this parameter set for GPU 🙏

I think in the way i did it it should also be using them cause i changed the bench_params alias, i will double check

Ah indeed you're right, looks good!

[agnesLeroy]

@IceTDrinker am I forgetting something here in the review? Is there anything else to do regarding parameters?

[IceTDrinker]

checking

[IceTDrinker]

The thing missing is updating the vec all in the root of the v1_5 parameters so that security checks e.g. can run

That's why I would prefer an automated generation, but I know it's a tall order...

@IceTDrinker reviewed 5 of 5 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @agnesLeroy and @mayeul-zama)

[IceTDrinker]

you also may want to add it to the meta/gpu module ?

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @agnesLeroy and @mayeul-zama)

[guillermo-oyarzun]

The thing missing is updating the vec all in the root of the v1_5 parameters so that security checks e.g. can run

That's why I would prefer an automated generation, but I know it's a tall order...

@IceTDrinker reviewed 5 of 5 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @agnesLeroy and @mayeul-zama)

Ok i did it, not sure that the fact that now we have 141 elements in the vec can affect in something (considering before we always had 140)

[guillermo-oyarzun]

you also may want to add it to the meta/gpu module ?

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @agnesLeroy and @mayeul-zama)

We commented and decided not to add them there, since we don't have a param set specifically for GPU for classical params, i would be strange to have only 1 param with a different naming.

[IceTDrinker]

The thing missing is updating the vec all in the root of the v1_5 parameters so that security checks e.g. can run
That's why I would prefer an automated generation, but I know it's a tall order...

@IceTDrinker reviewed 5 of 5 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @agnesLeroy and @mayeul-zama)

Ok i did it, not sure that the fact that now we have 141 elements in the vec can affect in something (considering before we always had 140)

doesn't matter as long as you updated the the array size

[IceTDrinker]

you also may want to add it to the meta/gpu module ?

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @agnesLeroy and @mayeul-zama)

We commented and decided not to add them there, since we don't have a param set specifically for GPU for classical params, i would be strange to have only 1 param with a different naming.

would likely be beneficial to have if you want the "parmater configurator" to be able to provide it to users

[IceTDrinker]

parameter check in progress to make sure the addition to the all vec worked https://github.com/zama-ai/tfhe-rs/actions/runs/18875125723

[IceTDrinker]

glwe looks ok: https://github.com/zama-ai/tfhe-rs/actions/runs/18875125723/job/53862829711#step:7:32
lwe too: https://github.com/zama-ai/tfhe-rs/actions/runs/18875125723/job/53862829711#step:7:264

[IceTDrinker]

Looks good ! and security run to check things is green, thanks !