Rate of Finished Goods is not calculating correctly in case there is a process Loss Involved

[JitendraSAW]

Information about bug

We are in a Paint Manufacturing Industry.

1. We have a use case where out of Total Qty- 97, out of Which 1 Qty is consider as a Process Loss (Scrap),
2. When we record the Manufacture Entry of Qty-97, 1 Qty taken as a Scrap, Hence 96 is the actual Qty which is Actual Yield Qty
3. But the Valuation of the above Qty of FG Item is showing wrong- It shows wrong Valuation- It is showing wrong valuation =13521.26/97=139.3944, wheras it should show- 13521.26/96=140.8465, since the actual yield qty is 96 whereas it is considering even the Process Loss Qty also which should not be as this qty is totally loss for me.

Module

manufacturing

Version

Installation method

FrappeCloud

Relevant log output / Stack trace / Full Error Message.

No response

[goapaints]

Please also note that Process Loss should not have a TARGET WAREHOUSE.

[rohitwaghchaure]

@JitendraSAW

We are adding process loss in the inventory to match up the "For Quantity" and sum of FG item's quantity in the manufacture stock entry. If we keep the FG item quantity less than "For Quantity" then system throws the error that "The finished product quantity and For Quantity cannot be different".

I understand your point that the Process Loss should not be the part of Inventory. To fix this problem I am suggesting a solution that if the Item has Is Process Loss flag then that item won't be added in the Inventory. If the item has the flag Is Scrap Item then item will add the item in the Inventory. If the item as both Is Process Loss and Is Scrap Item then also system will add that item in the stock.

Process Loss condition

1. If the Item has checkbox "Is Process Loss" -> Don't add Item in the Inventory
2. If the Item has checkbox "Is Scrap Item" -> Add Item in the Inventory
3. If the Item has checkbox "Is Scrap Item" & "Is Process Loss" -> Add Item in the Inventory

[goapaints]

Hi Rohit,

Ideally, no inventory should be both process loss and a scrap item. ie A scrap which cannot be recovered is a process loss.

Further,

Process loss should not insist on batch No and especially QI, even if mandated in Item Master.

Value difference should be zero. ie FG Valuation = {(Value of Input- Value of Scrap)/(Is finished qty)}

On Tue, 27 Dec, 2022, 15:40 rohitwaghchaure, @.***> wrote:

@JitendraSAW https://github.com/JitendraSAW

We are adding process loss in the inventory to match up the "For Quantity" and sum of FG item's quantity in the manufacture stock entry. If we keep the FG item quantity less than "For Quantity" then system throws the error that "The finished product quantity and For Quantity cannot be different".

I understand your point that the Process Loss should not be the part of Inventory. To fix this problem I am suggesting a solution that if the Item has Is Process Loss flag then that item won't be added in the Inventory. If the item has the flag Is Scrap Item then item will add the item in the Inventory. If the item as both Is Process Loss and Is Scrap Item then also system will add that item in the stock.

Process Loss condition

1. If the Item has checkbox "Is Process Loss" -> Don't add Item in the Inventory
2. If the Item has checkbox "Is Scrap Item" -> Add Item in the Inventory
3. If the Item has checkbox "Is Scrap Item" & "Is Process Loss" -> Add Item in the Inventory

— Reply to this email directly, view it on GitHub https://github.com/frappe/erpnext/issues/33094#issuecomment-1365779892, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4XBYJ7B5O65ZNEKRLTJJCLWPK6BRANCNFSM6AAAAAASJ3JFHU . You are receiving this because you commented.Message ID: @.***>

[goapaints]

Hi Rohit,

Please note that process loss percentage should be editable in manufacturing stock entry, as exact process loss will vary from batch to batch.

Thanks.

On Wed, 4 Jan, 2023, 18:37 rohitwaghchaure, @.***> wrote:

Closed #33094 https://github.com/frappe/erpnext/issues/33094 as completed via #33524 https://github.com/frappe/erpnext/pull/33524.

— Reply to this email directly, view it on GitHub https://github.com/frappe/erpnext/issues/33094#event-8151001834, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4XBYJ5NSNIFT7AROLZR3NTWQVY2DANCNFSM6AAAAAASJ3JFHU . You are receiving this because you commented.Message ID: @.***>