GNU bug report logs -
#30894
Submission of patch for Add r-abc
Previous Next
Reported by: Sahithi Yarlagadda <sahi <at> swecha.net>
Date: Wed, 21 Mar 2018 11:17:02 UTC
Severity: normal
Tags: moreinfo
Done: Ricardo Wurmus <rekado <at> elephly.net>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 30894 in the body.
You can then email your comments to 30894 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#30894
; Package
guix-patches
.
(Wed, 21 Mar 2018 11:17:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Sahithi Yarlagadda <sahi <at> swecha.net>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Wed, 21 Mar 2018 11:17:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Hi,
I am submitting a patch for r-abc. Please review.
--
Regards
Sahithi
[0001-gnu-Add-r-abc.patch (text/x-patch, attachment)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#30894
; Package
guix-patches
.
(Wed, 21 Mar 2018 21:29:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 30894 <at> debbugs.gnu.org (full text, mbox):
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 <at> 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!
--
Ricardo
GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
https://elephly.net
Information forwarded
to
guix-patches <at> gnu.org
:
bug#30894
; Package
guix-patches
.
(Wed, 28 Mar 2018 19:46:01 GMT)
Full text and
rfc822 format available.
Message #11 received at 30894 <at> debbugs.gnu.org (full text, mbox):
Hi Sahiti,
have you been able to look at my review below?
Please let me know if anything is unclear.
~~ Ricardo
Ricardo Wurmus <rekado <at> 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 <at> 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!
Added tag(s) moreinfo.
Request was from
Ricardo Wurmus <rekado <at> elephly.net>
to
control <at> debbugs.gnu.org
.
(Wed, 28 Mar 2018 19:52:02 GMT)
Full text and
rfc822 format available.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#30894
; Package
guix-patches
.
(Thu, 29 Mar 2018 10:16:02 GMT)
Full text and
rfc822 format available.
Message #16 received at 30894 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
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 <at> 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 <at> 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
[Message part 2 (text/html, inline)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#30894
; Package
guix-patches
.
(Thu, 29 Mar 2018 10:34:01 GMT)
Full text and
rfc822 format available.
Message #19 received at 30894 <at> debbugs.gnu.org (full text, mbox):
Hi Sahithi,
> 1) Cloning the Source
>
> 2) Running guix environment guix
>
> 3) ./bootstrap
>
> 4) ./configure --localstatedir=/var
You need to perform these steps only once.
> 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
I suggest running make again after making changes to any file in the
Guix source tree. This way you can avoid the “.go file is older than
the .scm” messages.
> *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)
I’d do this instead:
git fetch origin # fetch any upstream changes
git stash # hide my uncommitted local changes
git rebase origin/master # rebase my local commits on top of upstream
git stash pop # apply my uncommitted local changes
make # compile .scm to .go
Note that you should be inside of a suitable environment when running
“make”, so better do all of this after “guix environment guix”.
--
Ricardo
GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
https://elephly.net
Information forwarded
to
guix-patches <at> gnu.org
:
bug#30894
; Package
guix-patches
.
(Thu, 29 Mar 2018 10:52:01 GMT)
Full text and
rfc822 format available.
Message #22 received at 30894 <at> debbugs.gnu.org (full text, mbox):
Thanks Ricardo
When i am building r-abc I found errors with the following.
(propagated-inputs
`(("r-abc-data" ,r-abc-data)
abc.data is the upstream name of abc-data. I am figuring out how to use
it. I have searched for various package description to compare the
usage, but in vain. Can u help me what attribute is to be given to make
it work.
On Thursday 29 March 2018 04:02 PM, Ricardo Wurmus wrote:
> Hi Sahithi,
>
>> 1) Cloning the Source
>>
>> 2) Running guix environment guix
>>
>> 3) ./bootstrap
>>
>> 4) ./configure --localstatedir=/var
> You need to perform these steps only once.
>
>> 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
> I suggest running make again after making changes to any file in the
> Guix source tree. This way you can avoid the “.go file is older than
> the .scm” messages.
>
>> *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)
> I’d do this instead:
>
> git fetch origin # fetch any upstream changes
> git stash # hide my uncommitted local changes
> git rebase origin/master # rebase my local commits on top of upstream
> git stash pop # apply my uncommitted local changes
> make # compile .scm to .go
>
> Note that you should be inside of a suitable environment when running
> “make”, so better do all of this after “guix environment guix”.
>
> --
> Ricardo
>
> GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
> https://elephly.net
>
>
>
--
Regards
Sahithi
Information forwarded
to
guix-patches <at> gnu.org
:
bug#30894
; Package
guix-patches
.
(Thu, 29 Mar 2018 11:01:01 GMT)
Full text and
rfc822 format available.
Message #25 received at 30894 <at> debbugs.gnu.org (full text, mbox):
Hi
I am planning to do this.
I will submit the Package Description for r-abc-data. Which builds and
create a derivative in gnu/store. Later i will send the abc package
description, where the build finds necessary data and does a successful
build.
On Thursday 29 March 2018 04:20 PM, Sahithi Yarlagadda wrote:
> Thanks Ricardo
>
> When i am building r-abc I found errors with the following.
>
> (propagated-inputs
> `(("r-abc-data" ,r-abc-data)
>
>
> abc.data is the upstream name of abc-data. I am figuring out how to use
> it. I have searched for various package description to compare the
> usage, but in vain. Can u help me what attribute is to be given to make
> it work.
>
>
>
> On Thursday 29 March 2018 04:02 PM, Ricardo Wurmus wrote:
>> Hi Sahithi,
>>
>>> 1) Cloning the Source
>>>
>>> 2) Running guix environment guix
>>>
>>> 3) ./bootstrap
>>>
>>> 4) ./configure --localstatedir=/var
>> You need to perform these steps only once.
>>
>>> 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
>> I suggest running make again after making changes to any file in the
>> Guix source tree. This way you can avoid the “.go file is older than
>> the .scm” messages.
>>
>>> *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)
>> I’d do this instead:
>>
>> git fetch origin # fetch any upstream changes
>> git stash # hide my uncommitted local changes
>> git rebase origin/master # rebase my local commits on top of upstream
>> git stash pop # apply my uncommitted local changes
>> make # compile .scm to .go
>>
>> Note that you should be inside of a suitable environment when running
>> “make”, so better do all of this after “guix environment guix”.
>>
>> --
>> Ricardo
>>
>> GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
>> https://elephly.net
>>
>>
>>
--
Regards
Sahithi
Information forwarded
to
guix-patches <at> gnu.org
:
bug#30894
; Package
guix-patches
.
(Thu, 29 Mar 2018 11:30:02 GMT)
Full text and
rfc822 format available.
Message #28 received at 30894 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Hi
On Thursday 29 March 2018 04:30 PM, Sahithi Yarlagadda wrote:
> Hi
>
> I am planning to do this.
>
> I will submit the Package Description for r-abc-data. Which builds and
> create a derivative in gnu/store. Later i will send the abc package
> description, where the build finds necessary data and does a successful
> build.
Submitted a Patch file for r-abc-data which is a dependency for r-abc
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=30983
Then,
Re-based the source
git fetch origin # fetch any upstream changes
git stash # hide my uncommitted local changes
git rebase origin/master # rebase my local commits on top of upstream
git stash pop # apply my uncommitted local changes
make # compile .scm to .go
I have done adding r-abc. And the build was success.
( I am still unsure whether i did it right. Fingers Crossed !!!! ...
Waiting for your comment)
Its a great experience while learning with each contribution. Each
Package Description have a different things to look into carefully and
contribute.
Ricardo have been of great help in this process.
Created a Patch (I sense something's not right here!!!).
Submitting for the review.
>
> On Thursday 29 March 2018 04:20 PM, Sahithi Yarlagadda wrote:
>> Thanks Ricardo
>>
>> When i am building r-abc I found errors with the following.
>>
>> (propagated-inputs
>> `(("r-abc-data" ,r-abc-data)
>>
>>
>> abc.data is the upstream name of abc-data. I am figuring out how to use
>> it. I have searched for various package description to compare the
>> usage, but in vain. Can u help me what attribute is to be given to make
>> it work.
>>
>>
>>
>> On Thursday 29 March 2018 04:02 PM, Ricardo Wurmus wrote:
>>> Hi Sahithi,
>>>
>>>> 1) Cloning the Source
>>>>
>>>> 2) Running guix environment guix
>>>>
>>>> 3) ./bootstrap
>>>>
>>>> 4) ./configure --localstatedir=/var
>>> You need to perform these steps only once.
>>>
>>>> 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
>>> I suggest running make again after making changes to any file in the
>>> Guix source tree. This way you can avoid the “.go file is older than
>>> the .scm” messages.
>>>
>>>> *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)
>>> I’d do this instead:
>>>
>>> git fetch origin # fetch any upstream changes
>>> git stash # hide my uncommitted local changes
>>> git rebase origin/master # rebase my local commits on top of upstream
>>> git stash pop # apply my uncommitted local changes
>>> make # compile .scm to .go
>>>
>>> Note that you should be inside of a suitable environment when running
>>> “make”, so better do all of this after “guix environment guix”.
>>>
>>> --
>>> Ricardo
>>>
>>> GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
>>> https://elephly.net
>>>
>>>
>>>
--
Regards
Sahithi
[0001-gnu-Add-r-abc.patch (text/x-patch, attachment)]
Reply sent
to
Ricardo Wurmus <rekado <at> elephly.net>
:
You have taken responsibility.
(Mon, 02 Apr 2018 21:28:01 GMT)
Full text and
rfc822 format available.
Notification sent
to
Sahithi Yarlagadda <sahi <at> swecha.net>
:
bug acknowledged by developer.
(Mon, 02 Apr 2018 21:28:02 GMT)
Full text and
rfc822 format available.
Message #33 received at 30894-done <at> debbugs.gnu.org (full text, mbox):
Hi Sahithi,
> I have done adding r-abc. And the build was success.
Excellent! I also built it without problems.
I couldn’t apply it cleanly after making modifications to r-abc-data,
but that’s not your problem. I performed these minor changes:
* changed license from gpl3 to gpl3+
* added a beginning to the description.
Pushed the patch to the master branch with commit 82c8e0ae0.
Thanks again!
~~ Ricardo
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Tue, 01 May 2018 11:24:05 GMT)
Full text and
rfc822 format available.
This bug report was last modified 5 years and 333 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.