Channel name addition to plot_response_curves() #20
Replies: 3 comments 4 replies
-
Hey @FedericoRaimondi ! Thanks for opening up this one. Channel names are given at This is given that the same model object can be trained multiple times with different data and so it is consistent with the model for any plot (I know other plots do allow to pass the names). As the model object serves as a center piece for all the library. We need to see if we allow all plots to independently pass the names or if we stick with what the model object contains and remove it from the plots that allow it. Or just have both options, maybe the latter seems like the easiest for most people to find it for their usage. |
Beta Was this translation helpful? Give feedback.
-
thanks, indeed it worked. yes, I missed that as I was thinking to the other plots. |
Beta Was this translation helpful? Give feedback.
-
Hi @pabloduque0, Can we also add a point on the curve to indicate what our mean spending is to make it more insightful? and we know whether we are spending close saturation levels visually? Like the one below. I don't know if this is something that is already planned. Thanks! |
Beta Was this translation helpful? Give feedback.
-
Not sure if this is already in your plans, but I'm adding it here as a feature request:
In
plot_response_curves()
we miss the argument to add the channel name, that would be really nice from a visualization point of view.Thanks for your effort!
Beta Was this translation helpful? Give feedback.
All reactions