[FEAT] What if - pricing in retail scenario #340
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
|
View / edit / reply to this conversation on ReviewNB mergenthaler commented on 2024-05-06T21:49:22Z Maybe we can include a brief motivation or intro section. Something like:
elephaint commented on 2024-05-07T11:48:10Z Added a sentence |
|
View / edit / reply to this conversation on ReviewNB mergenthaler commented on 2024-05-06T21:49:23Z Now that our docu is getting very comprehensive, here would be a good place to link to your guide on exogenous variables. elephaint commented on 2024-05-07T11:55:28Z Added a callout-tip |
|
View / edit / reply to this conversation on ReviewNB mergenthaler commented on 2024-05-06T21:49:24Z This is great. Here I would relate this idea to the concept of elasticity. Maybe something like:
Elasticity measures how one variable responds to changes in another, commonly used to determine how price changes affect demand or supply. High elasticity indicates a significant response to small price changes, while low elasticity shows minimal response. This concept aids in setting pricing strategies and assessing the impact of economic policies on market dynamics.
Another helpful addition would be to include a comment on the underling assumptions. Maybe something like:
elephaint commented on 2024-05-07T12:10:27Z Added a call-out note for elasticity reference and on assumptions. |
|
View / edit / reply to this conversation on ReviewNB mergenthaler commented on 2024-05-06T21:49:24Z It would be interesting to draw some conclusions. Somethih like:
elephaint commented on 2024-05-07T12:17:37Z Added conclusion to the text above the plot |
|
Added a sentence View entire conversation on ReviewNB |
|
Added a callout-tip View entire conversation on ReviewNB |
|
Added a call-out note for elasticity reference and on assumptions. View entire conversation on ReviewNB |
|
Added conclusion to the text above the plot View entire conversation on ReviewNB |
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
|
Thanks for the changes @elephaint! This is ready for your review @AzulGarza. |
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
|
View / edit / reply to this conversation on ReviewNB AzulGarza commented on 2024-05-08T21:54:01Z let's make colab changes in #349 |
AzulGarza
requested changes
May 8, 2024
There was a problem hiding this comment.
thanks @elephaint! i'm so sorry about asking for this change, but could you remove the colab part and wait to merge all colab changes in #349.
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
…ixtla into feature/what-if-pricing
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
AzulGarza
approved these changes
May 10, 2024
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
Experiment ResultsExperiment 1: air-passengersDescription:
Results:
Plot:
Experiment 2: air-passengersDescription:
Results:
Plot:
Experiment 3: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 4: electricity-multiple-seriesDescription:
Results:
Plot:
Experiment 5: electricity-multiple-seriesDescription:
Results:
Plot:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds simple use case for evaluating different pricing scenarios when forecasting product demand for a set of products in retail.