Repository navigation
[16.0][IMP] stock_average_daily_sale: enable to choose the strategies used in config - #547
Conversation
|
Hi @jbaudoux, |
OCA-git-bot
added
series:16.0
mod:stock_average_daily_sale
e26c23e to
06f80ab
Compare
jbaudoux
left a comment
jbaudoux
left a comment
There was a problem hiding this comment.
As you clean the terminology, can you drop all occurrence of sale and replace by consumption. This module doesn't measure sales
| <field name="exclude_weekends" /> | ||
| </group> | ||
| </group> | ||
| <group name="strategies" string="Strategies"> |
There was a problem hiding this comment.
| <group name="strategies" string="Strategies"> | |
| <group name="strategies" string="Reorder point strategies"> |
| string="Average Daily Usage" | ||
| > | ||
| <div colspan="2" class="text-muted fst-italic mb-2"> | ||
| Target quantity = (Number of Days × Average Daily Quantity) + Safety Buffer. |
There was a problem hiding this comment.
| Target quantity = (Number of Days × Average Daily Quantity) + Safety Buffer. | |
| Min = (Average Daily Quantity x Number of Days) + Safety |
There was a problem hiding this comment.
And add also doc for
Safety = Daily Standard Deviation * Safety Factor * sqrt(nbr days in the analyzed period)
| </group> | ||
| <group name="average_sales_strategy" string="Average Sales"> | ||
| <div colspan="2" class="text-muted fst-italic mb-2"> | ||
| Target quantity = Number of Sales to Cover × Average Quantity per Sale. |
There was a problem hiding this comment.
| Target quantity = Number of Sales to Cover × Average Quantity per Sale. | |
| Min = Average Consumption Quantity x Factor |
There was a problem hiding this comment.
Why would you use the term "Consumption" here? This does not make sense to me? IMO "Number of sales" was clearer since we are indeed computing the number of SO's... What do you have in mind with this terminology?
There was a problem hiding this comment.
It's nothing about sales: Collect moves from the config location to outside that location
| </group> | ||
| <group name="max_sales_strategy" string="Max Sales"> | ||
| <div colspan="2" class="text-muted fst-italic mb-2"> | ||
| Target quantity = Max Quantity in one Sale. |
There was a problem hiding this comment.
You need to add a factor here as well
| Target quantity = Max Quantity in one Sale. | |
| Min = Max Consumption Quantity x Factor |
06f80ab to
cbd7089
Compare
Based on: #545 (to be merged first)
cc @jbaudoux