update comments on the compute method
update comments on the compute method

Related to #4041

berkes/spree,jordan-brough/solidus,derekluo/spree,mindvolt/spree,firman/spree,dafontaine/spree,madetech/spree,ramkumar-kr/spree,Ropeney/spree,delphsoft/spree-store-ballchair,ujai/spree,jasonfb/spree,odk211/spree,reidblomquist/spree,tancnle/spree,jspizziri/spree,grzlus/spree,hoanghiep90/spree,volpejoaquin/spree,abhishekjain16/spree,watg/spree,Hates/spree,gautamsawhney/spree,azclick/spree,piousbox/spree,vmatekole/spree,joanblake/spree,pervino/solidus,jimblesm/spree,alepore/spree,quentinuys/spree,ahmetabdi/spree,tancnle/spree,rbngzlv/spree,trigrass2/spree,jeffboulet/spree,CJMrozek/spree,vcavallo/spree,brchristian/spree,richardnuno/solidus,forkata/solidus,alepore/spree,athal7/solidus,woboinc/spree,keatonrow/spree,CiscoCloud/spree,xuewenfei/solidus,progsri/spree,archSeer/spree,CJMrozek/spree,APohio/spree,bjornlinder/Spree,Hawaiideveloper/shoppingcart,richardnuno/solidus,mleglise/spree,calvinl/spree,Engeltj/spree,bonobos/solidus,project-eutopia/spree,radarseesradar/spree,JuandGirald/spree,jordan-brough/spree,JuandGirald/spree,joanblake/spree,alvinjean/spree,bonobos/solidus,Senjai/solidus,ahmetabdi/spree,fahidnasir/spree,jordan-brough/solidus,Kagetsuki/spree,abhishekjain16/spree,jaspreet21anand/spree,surfdome/spree,freerunningtech/spree,softr8/spree,DynamoMTL/spree,vinsol/spree,KMikhaylovCTG/spree,Hates/spree,karlitxo/spree,Mayvenn/spree,CiscoCloud/spree,tailic/spree,watg/spree,DarkoP/spree,gregoryrikson/spree-sample,JDutil/spree,vinsol/spree,maybii/spree,caiqinghua/spree,vinsol/spree,Mayvenn/spree,shekibobo/spree,abhishekjain16/spree,tomash/spree,moneyspyder/spree,grzlus/solidus,DarkoP/spree,yiqing95/spree,pervino/spree,piousbox/spree,jordan-brough/spree,ayb/spree,jspizziri/spree,shaywood2/spree,azclick/spree,useiichi/spree,groundctrl/spree,biagidp/spree,JDutil/spree,vcavallo/spree,ayb/spree,azclick/spree,rbngzlv/spree,reinaris/spree,beni55/spree,tailic/spree,azclick/spree,lyzxsc/spree,dafontaine/spree,project-eutopia/spree,RatioClothing/spree,odk211/spree,sideci-sample/sideci-sample-spree,mleglise/spree,moneyspyder/spree,edgward/spree,mindvolt/spree,CJMrozek/spree,progsri/spree,edgward/spree,rajeevriitm/spree,archSeer/spree,adaddeo/spree,fahidnasir/spree,patdec/spree,NerdsvilleCEO/spree,KMikhaylovCTG/spree,scottcrawford03/solidus,knuepwebdev/FloatTubeRodHolders,caiqinghua/spree,welitonfreitas/spree,imella/spree,shaywood2/spree,vulk/spree,adaddeo/spree,FadliKun/spree,priyank-gupta/spree,jasonfb/spree,dandanwei/spree,gautamsawhney/spree,rakibulislam/spree,SadTreeFriends/spree,net2b/spree,useiichi/spree,bjornlinder/Spree,TimurTarasenko/spree,joanblake/spree,delphsoft/spree-store-ballchair,StemboltHQ/spree,shioyama/spree,degica/spree,tesserakt/clean_spree,scottcrawford03/solidus,edgward/spree,wolfieorama/spree,alepore/spree,bjornlinder/Spree,njerrywerry/spree,ckk-scratch/solidus,lsirivong/spree,devilcoders/solidus,Arpsara/solidus,yushine/spree,xuewenfei/solidus,jspizziri/spree,yushine/spree,StemboltHQ/spree,azranel/spree,tomash/spree,PhoenixTeam/spree_phoenix,cutefrank/spree,Kagetsuki/spree,patdec/spree,alvinjean/spree,vcavallo/spree,jaspreet21anand/spree,alejandromangione/spree,miyazawatomoka/spree,reinaris/spree,beni55/spree,yushine/spree,KMikhaylovCTG/spree,CJMrozek/spree,alejandromangione/spree,project-eutopia/spree,sfcgeorge/spree,jeffboulet/spree,nooysters/spree,karlitxo/spree,welitonfreitas/spree,hoanghiep90/spree,Nevensoft/spree,camelmasa/spree,lsirivong/solidus,KMikhaylovCTG/spree,pervino/spree,adaddeo/spree,yomishra/pce,imella/spree,Migweld/spree,JuandGirald/spree,gregoryrikson/spree-sample,thogg4/spree,priyank-gupta/spree,biagidp/spree,rakibulislam/spree,priyank-gupta/spree,tancnle/spree,RatioClothing/spree,miyazawatomoka/spree,sideci-sample/sideci-sample-spree,Senjai/spree,Ropeney/spree,dandanwei/spree,beni55/spree,Lostmyname/spree,kewaunited/spree,alvinjean/spree,jimblesm/spree,volpejoaquin/spree,biagidp/spree,ramkumar-kr/spree,AgilTec/spree,softr8/spree,nooysters/spree,SadTreeFriends/spree,Hawaiideveloper/shoppingcart,quentinuys/spree,groundctrl/spree,DynamoMTL/spree,kewaunited/spree,devilcoders/solidus,grzlus/solidus,AgilTec/spree,PhoenixTeam/spree_phoenix,raow/spree,jaspreet21anand/spree,caiqinghua/spree,Kagetsuki/spree,bricesanchez/spree,firman/spree,camelmasa/spree,maybii/spree,tesserakt/clean_spree,tesserakt/clean_spree,FadliKun/spree,pervino/solidus,NerdsvilleCEO/spree,sliaquat/spree,TrialGuides/spree,azranel/spree,vinayvinsol/spree,berkes/spree,jaspreet21anand/spree,Kagetsuki/spree,athal7/solidus,omarsar/spree,builtbybuffalo/spree,orenf/spree,grzlus/solidus,PhoenixTeam/spree_phoenix,azranel/spree,Ropeney/spree,Machpowersystems/spree_mach,vulk/spree,ahmetabdi/spree,yiqing95/spree,shaywood2/spree,JDutil/spree,ahmetabdi/spree,lsirivong/solidus,njerrywerry/spree,athal7/solidus,Engeltj/spree,pervino/solidus,CiscoCloud/spree,kewaunited/spree,robodisco/spree,Senjai/solidus,Senjai/solidus,carlesjove/spree,Migweld/spree,jimblesm/spree,jparr/spree,pulkit21/spree,sliaquat/spree,edgward/spree,RatioClothing/spree,cutefrank/spree,Machpowersystems/spree_mach,welitonfreitas/spree,net2b/spree,Migweld/spree,ckk-scratch/solidus,pulkit21/spree,brchristian/spree,fahidnasir/spree,progsri/spree,cutefrank/spree,builtbybuffalo/spree,agient/agientstorefront,dafontaine/spree,builtbybuffalo/spree,raow/spree,lsirivong/solidus,archSeer/spree,lsirivong/spree,njerrywerry/spree,scottcrawford03/solidus,surfdome/spree,softr8/spree,net2b/spree,jeffboulet/spree,yushine/spree,dandanwei/spree,Boomkat/spree,omarsar/spree,richardnuno/solidus,zamiang/spree,Nevensoft/spree,piousbox/spree,gautamsawhney/spree,jordan-brough/solidus,assembledbrands/spree,thogg4/spree,bricesanchez/spree,locomotivapro/spree,FadliKun/spree,pulkit21/spree,kewaunited/spree,hifly/spree,orenf/spree,vcavallo/spree,trigrass2/spree,groundctrl/spree,keatonrow/spree,jspizziri/spree,rbngzlv/spree,odk211/spree,Hawaiideveloper/shoppingcart,Ropeney/spree,siddharth28/spree,scottcrawford03/solidus,JuandGirald/spree,DynamoMTL/spree,rajeevriitm/spree,reidblomquist/spree,quentinuys/spree,mleglise/spree,bonobos/solidus,radarseesradar/spree,delphsoft/spree-store-ballchair,orenf/spree,zamiang/spree,zaeznet/spree,tancnle/spree,jasonfb/spree,madetech/spree,devilcoders/solidus,DynamoMTL/spree,knuepwebdev/FloatTubeRodHolders,siddharth28/spree,dotandbo/spree,rajeevriitm/spree,Lostmyname/spree,dandanwei/spree,reidblomquist/spree,karlitxo/spree,Hates/spree,Senjai/spree,Mayvenn/spree,jasonfb/spree,pulkit21/spree,raow/spree,ayb/spree,vmatekole/spree,zaeznet/spree,vinayvinsol/spree,Boomkat/spree,watg/spree,hoanghiep90/spree,yiqing95/spree,radarseesradar/spree,moneyspyder/spree,carlesjove/spree,madetech/spree,ckk-scratch/solidus,Boomkat/spree,Engeltj/spree,quentinuys/spree,vulk/spree,tomash/spree,ujai/spree,ramkumar-kr/spree,freerunningtech/spree,shioyama/spree,trigrass2/spree,Lostmyname/spree,yomishra/pce,TrialGuides/spree,rakibulislam/spree,brchristian/spree,Boomkat/spree,mindvolt/spree,Nevensoft/spree,reinaris/spree,shaywood2/spree,Migweld/spree,LBRapid/spree,robodisco/spree,knuepwebdev/FloatTubeRodHolders,vulk/spree,degica/spree,Mayvenn/spree,ramkumar-kr/spree,forkata/solidus,xuewenfei/solidus,Arpsara/solidus,Engeltj/spree,moneyspyder/spree,lyzxsc/spree,siddharth28/spree,karlitxo/spree,devilcoders/solidus,patdec/spree,njerrywerry/spree,rbngzlv/spree,reidblomquist/spree,keatonrow/spree,surfdome/spree,vinsol/spree,rajeevriitm/spree,sfcgeorge/spree,nooysters/spree,DarkoP/spree,TimurTarasenko/spree,Senjai/solidus,welitonfreitas/spree,miyazawatomoka/spree,jparr/spree,jordan-brough/solidus,cutefrank/spree,AgilTec/spree,assembledbrands/spree,nooysters/spree,keatonrow/spree,zaeznet/spree,hifly/spree,surfdome/spree,CiscoCloud/spree,berkes/spree,omarsar/spree,sfcgeorge/spree,useiichi/spree,firman/spree,zamiang/spree,volpejoaquin/spree,agient/agientstorefront,richardnuno/solidus,zamiang/spree,LBRapid/spree,jimblesm/spree,NerdsvilleCEO/spree,jsurdilla/solidus,robodisco/spree,beni55/spree,hifly/spree,gregoryrikson/spree-sample,miyazawatomoka/spree,Arpsara/solidus,JDutil/spree,delphsoft/spree-store-ballchair,Hawaiideveloper/shoppingcart,groundctrl/spree,grzlus/solidus,grzlus/spree,lsirivong/solidus,derekluo/spree,Arpsara/solidus,reinaris/spree,ujai/spree,lsirivong/spree,assembledbrands/spree,camelmasa/spree,calvinl/spree,vmatekole/spree,tesserakt/clean_spree,locomotivapro/spree,Hates/spree,piousbox/spree,lyzxsc/spree,sunny2601/spree,mindvolt/spree,joanblake/spree,maybii/spree,carlesjove/spree,dafontaine/spree,freerunningtech/spree,LBRapid/spree,ckk-scratch/solidus,NerdsvilleCEO/spree,thogg4/spree,adaddeo/spree,bonobos/solidus,sliaquat/spree,SadTreeFriends/spree,dotandbo/spree,pervino/solidus,degica/spree,pervino/spree,wolfieorama/spree,berkes/spree,forkata/solidus,athal7/solidus,PhoenixTeam/spree_phoenix,mleglise/spree,builtbybuffalo/spree,maybii/spree,TimurTarasenko/spree,SadTreeFriends/spree,xuewenfei/solidus,zaeznet/spree,fahidnasir/spree,bricesanchez/spree,TimurTarasenko/spree,thogg4/spree,net2b/spree,progsri/spree,APohio/spree,useiichi/spree,ayb/spree,alejandromangione/spree,hifly/spree,grzlus/spree,wolfieorama/spree,shekibobo/spree,gregoryrikson/spree-sample,vinayvinsol/spree,madetech/spree,patdec/spree,sfcgeorge/spree,priyank-gupta/spree,hoanghiep90/spree,orenf/spree,FadliKun/spree,robodisco/spree,Senjai/spree,carlesjove/spree,volpejoaquin/spree,tomash/spree,rakibulislam/spree,sliaquat/spree,yomishra/pce,forkata/solidus,jeffboulet/spree,abhishekjain16/spree,locomotivapro/spree,siddharth28/spree,alejandromangione/spree,project-eutopia/spree,softr8/spree,brchristian/spree,sunny2601/spree,DarkoP/spree,shioyama/spree,vinayvinsol/spree,agient/agientstorefront,derekluo/spree,AgilTec/spree,APohio/spree,grzlus/spree,caiqinghua/spree,derekluo/spree,camelmasa/spree,jparr/spree,Lostmyname/spree,jparr/spree,jsurdilla/solidus,lsirivong/spree,shekibobo/spree,sunny2601/spree,trigrass2/spree,radarseesradar/spree,jordan-brough/spree,archSeer/spree,azranel/spree,wolfieorama/spree,locomotivapro/spree,yiqing95/spree,woboinc/spree,firman/spree,lyzxsc/spree,Nevensoft/spree,omarsar/spree,jsurdilla/solidus,jsurdilla/solidus,alvinjean/spree,dotandbo/spree,shekibobo/spree,TrialGuides/spree,gautamsawhney/spree,vmatekole/spree,raow/spree,calvinl/spree,tailic/spree,Machpowersystems/spree_mach,calvinl/spree,TrialGuides/spree,imella/spree,odk211/spree,woboinc/spree,StemboltHQ/spree,dotandbo/spree,sunny2601/spree,agient/agientstorefront,sideci-sample/sideci-sample-spree,pervino/spree,APohio/spree