From debbugs-submit-bounces@debbugs.gnu.org Thu Mar 29 06:15:32 2018 Received: (at 30894) by debbugs.gnu.org; 29 Mar 2018 10:15:32 +0000 Received: from localhost ([127.0.0.1]:57978 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1f1Uah-0003a7-DU for submit@debbugs.gnu.org; Thu, 29 Mar 2018 06:15:31 -0400 Received: from swecha.net ([136.243.177.181]:51016) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1f1Uaf-0003Zy-4U for 30894@debbugs.gnu.org; Thu, 29 Mar 2018 06:15:30 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=swecha.net; s=default; h=Content-Type:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Sender:Reply-To:Content-Transfer-Encoding:Content-ID :Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To: Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe :List-Post:List-Owner:List-Archive; bh=hfGwxfC3OAX3K4c3EPRyJsewcPhbtK+KZG2xoO8BcMA=; b=G/KebSVRFVEOUQT+SNiLPeDJzx +xtmmZNmGtjwdWQejq3kd6uqSp7EzlEmT7CDGmesFsYMMtEz4VHi9lvpPbYpVKiay+9R+l7C7WZDA D6TSXisRICYfjvqf+VpLtFS7gwk9gXw0w+Eu0ZP3K+9+88VJqnezfAzdp0342HvltQsTEHJrhylna 5qprmLz7KiWZuDaarpOinPgcCZdf2wX8fxRyQohFiJap9YPlu/PU1Mu165vFfs69zi360ayzRkv5n HJc/pA11E8u/qikgykOZ7xBHTNixs5Echa/ULHo/x8joCN3T/JozYlPXmeVV9hmtGvsBS08PwGkPH nbhVFMRA==; Received: from [103.206.105.70] (port=43396 helo=[10.46.198.193]) by server.amistadraising.in with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.89_1) (envelope-from ) id 1f1Uad-0048WG-3r; Thu, 29 Mar 2018 15:45:27 +0530 Subject: Re: [bug#30894] Submission of patch for Add r-abc To: Ricardo Wurmus References: <071fa29f-107d-2097-586c-ce2f1eb91c23@swecha.net> <87tvt9azl6.fsf@elephly.net> <87vadgdlyd.fsf@elephly.net> From: Sahithi Yarlagadda Message-ID: <6c9c531d-75a5-be08-2784-ebae3c3a432b@swecha.net> Date: Thu, 29 Mar 2018 15:45:19 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <87vadgdlyd.fsf@elephly.net> Content-Type: multipart/alternative; boundary="------------22CF07FF15739B868F03AF52" Content-Language: en-US X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - server.amistadraising.in X-AntiAbuse: Original Domain - debbugs.gnu.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - swecha.net X-Get-Message-Sender-Via: server.amistadraising.in: authenticated_id: sahi@swecha.net X-Authenticated-Sender: server.amistadraising.in: sahi@swecha.net X-Source: X-Source-Args: X-Source-Dir: X-From-Rewrite: unmodified, already matched X-Spam-Score: -0.0 (/) X-Debbugs-Envelope-To: 30894 Cc: 30894@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -0.0 (/) This is a multi-part message in MIME format. --------------22CF07FF15739B868F03AF52 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Hi Ricardo I have a small doubt regarding the build. Let me explain the process i am doing the build. 1) Cloning the Source 2) Running guix environment guix 3) ./bootstrap 4) ./configure --localstatedir=/var 5) make 6) Make Changes to the cran.scm file 7) Build using  ./pre-inst-env guix build 8) If build is success, format the patch and submit the patch *When i am planning to write a different package description, i am following this* 9) rebasing the patch 10) Pull for any changes 11) running the process from (2) Am i doing the right way, I hope i have to compile everytime and then make changes each Package Description. Because when i tired to just make changes and build on old one, i got many errors and warning saying the .go file is older than the .scm. Please suggest me. On Thursday 29 March 2018 01:14 AM, Ricardo Wurmus wrote: > Hi Sahiti, > > have you been able to look at my review below? > Please let me know if anything is unclear. > > ~~ Ricardo > > Ricardo Wurmus writes: > >> Hi Sahithi, >> >>> I am submitting a patch for r-abc. Please review. >> Thank you for the patch! >> >> What follows is a short review that includes things I could change >> myself before applying the patch, but for the sake of learning I >> explicitly point out in this message. I hope you don’t mind that I’m >> focusing on what may seem like minor details. >> >>> From 4b3b3eee4995616059d8a27489affc43cee26ea4 Mon Sep 17 00:00:00 2001 >>> From: Sahithi Yarlagadda >>> Date: Wed, 21 Mar 2018 16:30:18 +0530 >>> Subject: [PATCH] gnu: Add r-abc. >>> >>> * gnu/packages/cran.scm (r-abc): New variable. >> Nitpick: please don’t indent this line. The asterisk should be the >> first character in this line. >> >>> + >>> +(define-public r-abc >>> + (package >>> + (name "r-abc") >>> + (version "2.1") >>> + (source >>> + (origin >>> + (method url-fetch) >>> + (uri (cran-uri "abc" version)) >>> + (sha256 >>> + (base32 >>> + "0ngzaaz2y2s03fhngvwipmy4kq38xrmyddaz6a6l858rxvadrlhb")))) >>> + (build-system r-build-system) >>> + (propagated-inputs >>> + `(("r-abc-data" ,r-abc-data) >>> + ("r-locfit" ,r-locfit) >>> + ("r-mass" ,r-mass) >>> + ("r-nnet" ,r-nnet) >>> + ("r-quantreg" ,r-quantreg))) >>> + (home-page >>> + "http://cran.r-project.org/web/packages/abc") >> Please pull this onto the previous line and use “https”. >> >>> + (synopsis >>> + "Tools for approximate bayesian computation (ABC)") >> Please combine this line with the previous line. >> >>> + (description >>> + "Implements several ABC algorithms for performing parameter estimation, >>> +model selection, and goodness-of-fit. Cross-validation tools are also >>> +available for measuring the accuracy of ABC estimates, and to calculate the >>> +misclassification probabilities of different models.") >> Please use full sentences. For CRAN packages the official description >> usually provides only a sentence fragment in place of the first >> sentence. Please also use “@dfn{Approximate Bayesian Computation} >> (ABC)” the first time “ABC” is mentioned. >> >> All thoughout the indentation is a bit off (that’s expected as the >> importer is very simple and doesn’t know about the expected >> indentation). If you’re using Emacs you can easily adjust it with M-C-q >> (also known as “indent-sexp”); otherwise please use >> “etc/indent-code.el”. >> >> Have you built this package with Guix? >> >> Finally, please rebase this patch so that it can be applied >> independently from the other patch you’ve sent, which cannot currently >> be applied due to problems in building the defined package. Rebasing >> this patch ensures that it can be merged even if the other patch is >> stuck. >> >> Thanks! > > -- Regards Sahithi --------------22CF07FF15739B868F03AF52 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: 8bit

Hi Ricardo


I have a small doubt regarding the build. Let me explain the process i am doing the build.

1) Cloning the Source

2) Running guix environment guix

3) ./bootstrap

4) ./configure --localstatedir=/var

5) make

6) Make Changes to the cran.scm file

7) Build using  ./pre-inst-env guix build <package>

8) If build is success, format the patch and submit the patch

When i am planning to write a different package description, i am following this

9) rebasing the patch

10) Pull for any changes

11) running the process from (2)


Am i doing the right way, I hope i have to compile everytime and then make changes each Package Description. Because when i tired to just make changes and build on old one, i got many errors and warning saying the .go file is older than the .scm.


Please suggest me.


On Thursday 29 March 2018 01:14 AM, Ricardo Wurmus wrote:
Hi Sahiti,

have you been able to look at my review below?

Please let me know if anything is unclear.

~~ Ricardo

Ricardo Wurmus <rekado@elephly.net> writes:

Hi Sahithi,

I am submitting a patch for r-abc. Please review.
Thank you for the patch!

What follows is a short review that includes things I could change
myself before applying the patch, but for the sake of learning I
explicitly point out in this message.  I hope you don’t mind that I’m
focusing on what may seem like minor details.

From 4b3b3eee4995616059d8a27489affc43cee26ea4 Mon Sep 17 00:00:00 2001
From: Sahithi Yarlagadda <sahi@swecha.net>
Date: Wed, 21 Mar 2018 16:30:18 +0530
Subject: [PATCH] gnu: Add r-abc.

    * gnu/packages/cran.scm (r-abc): New variable.
Nitpick: please don’t indent this line.  The asterisk should be the
first character in this line.

+
+(define-public r-abc
+  (package
+    (name "r-abc")
+    (version "2.1")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (cran-uri "abc" version))
+        (sha256
+          (base32
+            "0ngzaaz2y2s03fhngvwipmy4kq38xrmyddaz6a6l858rxvadrlhb"))))
+    (build-system r-build-system)
+    (propagated-inputs
+      `(("r-abc-data" ,r-abc-data)
+        ("r-locfit" ,r-locfit)
+        ("r-mass" ,r-mass)
+        ("r-nnet" ,r-nnet)
+        ("r-quantreg" ,r-quantreg)))
+    (home-page
+      "http://cran.r-project.org/web/packages/abc")
Please pull this onto the previous line and use “https”.

+    (synopsis
+      "Tools for approximate bayesian computation (ABC)")
Please combine this line with the previous line.

+    (description
+      "Implements several ABC algorithms for performing parameter estimation,
+model selection, and goodness-of-fit.  Cross-validation tools are also
+available for measuring the accuracy of ABC estimates, and to calculate the
+misclassification probabilities of different models.")
Please use full sentences.  For CRAN packages the official description
usually provides only a sentence fragment in place of the first
sentence.  Please also use “@dfn{Approximate Bayesian Computation}
(ABC)” the first time “ABC” is mentioned.

All thoughout the indentation is a bit off (that’s expected as the
importer is very simple and doesn’t know about the expected
indentation).  If you’re using Emacs you can easily adjust it with M-C-q
(also known as “indent-sexp”); otherwise please use
“etc/indent-code.el”.

Have you built this package with Guix?

Finally, please rebase this patch so that it can be applied
independently from the other patch you’ve sent, which cannot currently
be applied due to problems in building the defined package.  Rebasing
this patch ensures that it can be merged even if the other patch is
stuck.

Thanks!



-- 
Regards
Sahithi
--------------22CF07FF15739B868F03AF52--