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

does hill_sat go through (0, 0) #917

Closed
dirknbr opened this issue Aug 7, 2024 · 7 comments · Fixed by #920 or #925
Closed

does hill_sat go through (0, 0) #917

dirknbr opened this issue Aug 7, 2024 · 7 comments · Fixed by #920 or #925
Labels

Comments

@dirknbr
Copy link

dirknbr commented Aug 7, 2024

Is it possible that your hill_saturation function doesn't pass through (0, 0), which would be necessary since we assume 0 sales if a single channel is 0

Screenshot from 2024-08-07 16-36-12

@juanitorduz
Copy link
Collaborator

Thanks for reporting this @dirknbr ! We have a fix in #920

@dirknbr
Copy link
Author

dirknbr commented Aug 12, 2024

but does this fix mean that sigma is still the aymptote for all positive values of lambda and beta?

@juanitorduz
Copy link
Collaborator

juanitorduz commented Aug 12, 2024

but does this fix mean that sigma is still the aymptote for all positive values of lambda and beta?

Actually, it's going to be sigma minus the offset. Let me reopen this issue and see if we can bring back the original parametrization. Maybe @dirknbr you wanna give it a go? :)

@juanitorduz juanitorduz reopened this Aug 12, 2024
@dirknbr
Copy link
Author

dirknbr commented Aug 12, 2024

why not use equation 4 of https://static.googleusercontent.com/media/research.google.com/en//pubs/archive/46001.pdf

@juanitorduz
Copy link
Collaborator

Ok! Seems the way to go. I am unsure about the current parameterization via exponentials.

@juanitorduz
Copy link
Collaborator

@juanitorduz
Copy link
Collaborator

We added the original implementation of the paper (with the corresponding reference)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants