search

syedalimohsinbukhari / pyMultiFit

A library to fit fit the data with multiple fitters.
MIT License
0 stars 0 forks source link

[ENH] `amplitude` issue with distributions #6

Closed syedalimohsinbukhari closed 1 month ago

syedalimohsinbukhari commented 1 month ago

It seems that the distributions are lacking the parameter amplitude for use, have to fix it.

syedalimohsinbukhari commented 1 month ago

Fixed it with gaussian, logNormal and laplace distributions see 9f212ca

syedalimohsinbukhari commented 1 month ago

The standard distributions do not have an amplitude factor, it should be accounted for in an extra class function, otherwise the implementations are correct.

syedalimohsinbukhari commented 1 month ago

done in ab0c9b1bcc8abbea83ca562c3a8fdbf29e32f454