-
Notifications
You must be signed in to change notification settings - Fork 25
Description
Hello,
I'd like to express my gratitude for your efforts in creating this valuable tool.
I have a few suggestions that I believe could enhance the user experience and improve the documentation:
-
Improved Variable Naming in
get_results()
:
Theget_results()
function returns a wealth of useful information. However, the use of abbreviations such asse
,t_stat
,p_t
,bic
, orrss
might be challenging for users from diverse fields. See: this Software Engineering Stack Exchange post. -
Example Usage for
get_results()
andget_params()
in Documentation:
Whileget_results()
andget_params()
are excellent methods, the documentation lacks examples illustrating their usage. Adding clear examples, possibly with simple mathematical models like the ones below, would significantly benefit users:$$ y = const + α1⋅x $$ $$ y = const - β1+ α2⋅(x − breakpoint1) $$ -
Introduce a Function for Returning a List of Fits:
It would be valuable to introduce a new function that returns a list of fits, such as[fit_1, breakpoint1, fit_2, breakpoint2]
. Each fit (e.g.,fit_1
) could be a callable function that acceptsxs
and returns correspondingys
based on the fit.
Thank you for considering these suggestions. I believe these enhancements will contribute to the overall usability and understanding of your tool.
I already forked the repository and fixed the Variable Naming in get_results()
. I can request a merge. However Wanted to share my thoughts here first.