IContextParamsExtensions.ToLlamaContextParams overwrites passed SeqMax value

[Ed-Pavlov]

I'm not experienced with all this AI stuff, so don't know, if it a bug or not.

https://github.com/SciSharp/LLamaSharp/blob/master/LLama/Extensions/IContextParamsExtensions.cs
In the method IContextParamsExtensions.ToLlamaContextParams passed SeqMax value is stored at first
result.n_seq_max = @params.SeqMax; - ln. 28

but then is overriden by hardcoded logic
result.n_seq_max = Math.Min(Math.Max(10U, result.n_ctx / 8U), 64U /*0x40*/); - ln. 58

Because of that it's impossible to set sequences number as desired.

Should I create a bug, or it's supposed to be this way?

[martindevans]

That definitely looks like a bug!

[Ed-Pavlov]

Thanks. I've created a pull request.