unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / Atom feed
* [bug#43994] [PATCH] gnu: Add r-calculus.
@ 2020-10-14 19:06 Magali Lemes
  2020-10-15 11:09 ` zimoun
  2020-10-16 18:51 ` [bug#43994] [PATCH v2] " Magali Lemes
  0 siblings, 2 replies; 4+ messages in thread
From: Magali Lemes @ 2020-10-14 19:06 UTC (permalink / raw)
  To: 43994; +Cc: Magali Lemes

---
 gnu/packages/cran.scm | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/gnu/packages/cran.scm b/gnu/packages/cran.scm
index 59a409f8e9..8a4f0085f6 100644
--- a/gnu/packages/cran.scm
+++ b/gnu/packages/cran.scm
@@ -24596,3 +24596,30 @@ enrichment analysis (GSEA) calculation with or without the absolute filtering.
   Without filtering, users can perform (original) two-tailed or one-tailed
 absolute GSEA.")
     (license license:gpl2)))
+
+(define-public r-calculus
+  (package
+    (name "r-calculus")
+    (version "0.2.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (cran-uri "calculus" version))
+       (sha256
+       (base32
+        "0hs7hzjl6xjza20v9zx9a1piywxa6w3h2rskr52d1dcbc0vwhinp"))))
+    (properties `((upstream-name . "calculus")))
+    (build-system r-build-system)
+    (propagated-inputs `(("r-rcpp" ,r-rcpp)))
+    (home-page "https://github.com/emanuele-guidotti/calculus")
+    (synopsis "High Dimensional Numerical and Symbolic Calculus")
+    (description
+     "Efficient C++ optimized functions for numerical and symbolic calculus.
+It includes basic symbolic arithmetic, tensor calculus, Einstein summing
+convention, fast computation of the Levi-Civita symbol and generalized 
+Kronecker delta, Taylor series expansion, multivariate Hermite polynomials,
+accurate high-order derivatives, differential operators (Gradient, Jacobian,
+Hessian, Divergence, Curl, Laplacian) and numerical integration in arbitrary
+orthogonal coordinate systems: cartesian, polar, spherical, cylindrical, 
+parabolic or user defined by custom scale factors.")
+    (license license:gpl3)))
\ No newline at end of file
-- 
2.25.1





^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#43994] [PATCH] gnu: Add r-calculus.
  2020-10-14 19:06 [bug#43994] [PATCH] gnu: Add r-calculus Magali Lemes
@ 2020-10-15 11:09 ` zimoun
  2020-10-16 18:51 ` [bug#43994] [PATCH v2] " Magali Lemes
  1 sibling, 0 replies; 4+ messages in thread
From: zimoun @ 2020-10-15 11:09 UTC (permalink / raw)
  To: Magali Lemes; +Cc: 43994

Hi,

Thank you for your contribution.  It is fine.  For the sake of the
learning process, I suggest some minor tweaks and then you can send a v2
patch.  Does it make sense?



In the commit message, it is missing:

* gnu/packages/cran.scm (r-calculus): New variable.


> ---
>  gnu/packages/cran.scm | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
>
> diff --git a/gnu/packages/cran.scm b/gnu/packages/cran.scm
> index 59a409f8e9..8a4f0085f6 100644
> --- a/gnu/packages/cran.scm
> +++ b/gnu/packages/cran.scm
> @@ -24596,3 +24596,30 @@ enrichment analysis (GSEA) calculation with or without the absolute filtering.
>    Without filtering, users can perform (original) two-tailed or one-tailed
>  absolute GSEA.")
>      (license license:gpl2)))
> +
> +(define-public r-calculus
> +  (package
> +    (name "r-calculus")
> +    (version "0.2.1")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (cran-uri "calculus" version))
> +       (sha256
> +       (base32

Here, and issue of indentation.

       (uri (cran-uri "calculus" version))
       (sha256
        (base32

Usually, with Emacs (with the correct setup “.dir-locals.el“), the
simplest is to go to the first opening parenthesis and press ‘C-M-q’.


> +        "0hs7hzjl6xjza20v9zx9a1piywxa6w3h2rskr52d1dcbc0vwhinp"))))
> +    (properties `((upstream-name . "calculus")))
> +    (build-system r-build-system)
> +    (propagated-inputs `(("r-rcpp" ,r-rcpp)))

Usually, it is preferred to list below:

    (propagated-inputs
     `(("r-rcpp" ,r-rcpp)))


> +    (home-page "https://github.com/emanuele-guidotti/calculus")
> +    (synopsis "High Dimensional Numerical and Symbolic Calculus")
> +    (description
> +     "Efficient C++ optimized functions for numerical and symbolic calculus.
> +It includes basic symbolic arithmetic, tensor calculus, Einstein summing
> +convention, fast computation of the Levi-Civita symbol and generalized 

End of line: extra whitespace 

> +Kronecker delta, Taylor series expansion, multivariate Hermite polynomials,
> +accurate high-order derivatives, differential operators (Gradient, Jacobian,
> +Hessian, Divergence, Curl, Laplacian) and numerical integration in arbitrary
> +orthogonal coordinate systems: cartesian, polar, spherical, cylindrical, 

Idem


Again with Emacs, I personally have ’(whitespace-mode)’ with this
config and then it is really easy to spot out.

--8<---------------cut here---------------start------------->8---
(with-eval-after-load 'whitespace
  (setq whitespace-line-column nil
        whitespace-style '(face trailing lines-tail
                                space-before-tab newline
                                indentation empty space-after-tab)))

--8<---------------cut here---------------end--------------->8---

> +parabolic or user defined by custom scale factors.")
> +    (license license:gpl3)))
> \ No newline at end of file

Add RET. I guess.



Well, now you should do an “interactive rebase” and resend.  Using Emacs
(again!) and Magit , it looks like:

  r i
  edit C-c C-c
  # do your tweaks
  r r
  W c C-m b master C-m v 2 c
  ! git send-email --to=43994@debbugs.gnu.org v2-0001-blabla.patch

Does it make sense?


All the best,
simon

PS:
You can use any tools you want. :-) Sorry for the Emacs’s
evangelism. ;-)




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#43994] [PATCH v2] gnu: Add r-calculus.
  2020-10-14 19:06 [bug#43994] [PATCH] gnu: Add r-calculus Magali Lemes
  2020-10-15 11:09 ` zimoun
@ 2020-10-16 18:51 ` Magali Lemes
  2020-10-17 20:39   ` bug#43994: " Ludovic Courtès
  1 sibling, 1 reply; 4+ messages in thread
From: Magali Lemes @ 2020-10-16 18:51 UTC (permalink / raw)
  To: 43994; +Cc: Magali Lemes

* gnu/packages/cran.scm (r-calculus): New variable.
---
 gnu/packages/cran.scm | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/gnu/packages/cran.scm b/gnu/packages/cran.scm
index 252699c691..ce5ae7f64a 100644
--- a/gnu/packages/cran.scm
+++ b/gnu/packages/cran.scm
@@ -30,6 +30,7 @@
 ;;; Copyright © 2020 Vinicius Monego <monego@posteo.net>
 ;;; Copyright © 2020 Antoine Côté <antoine.cote@posteo.net>
 ;;; Copyright © 2020 Arun Isaac <arunisaac@systemreboot.net>
+;;; Copyright © 2020 Magali Lemes <magalilemes00@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -24598,3 +24599,32 @@ enrichment analysis (GSEA) calculation with or without the absolute filtering.
   Without filtering, users can perform (original) two-tailed or one-tailed
 absolute GSEA.")
     (license license:gpl2)))
+
+(define-public r-calculus
+  (package
+    (name "r-calculus")
+    (version "0.2.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (cran-uri "calculus" version))
+       (sha256
+        (base32
+         "0hs7hzjl6xjza20v9zx9a1piywxa6w3h2rskr52d1dcbc0vwhinp"))))
+    (properties `((upstream-name . "calculus")))
+    (build-system r-build-system)
+    (propagated-inputs
+     `(("r-rcpp" ,r-rcpp)))
+    (home-page "https://github.com/emanuele-guidotti/calculus")
+    (synopsis "High Dimensional Numerical and Symbolic Calculus")
+    (description
+     "Efficient C++ optimized functions for numerical and symbolic calculus.
+It includes basic symbolic arithmetic, tensor calculus, Einstein summing
+convention, fast computation of the Levi-Civita symbol and generalized
+Kronecker delta, Taylor series expansion, multivariate Hermite polynomials,
+accurate high-order derivatives, differential operators (Gradient, Jacobian,
+Hessian, Divergence, Curl, Laplacian) and numerical integration in arbitrary
+orthogonal coordinate systems: cartesian, polar, spherical, cylindrical,
+parabolic or user defined by custom scale factors.")
+    (license license:gpl3)))
+
-- 
2.25.1





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#43994: [PATCH v2] gnu: Add r-calculus.
  2020-10-16 18:51 ` [bug#43994] [PATCH v2] " Magali Lemes
@ 2020-10-17 20:39   ` Ludovic Courtès
  0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2020-10-17 20:39 UTC (permalink / raw)
  To: Magali Lemes; +Cc: 43994-done

[-- Attachment #1: Type: text/plain, Size: 282 bytes --]

Hi Magali,

Magali Lemes <magalilemes00@gmail.com> skribis:

> * gnu/packages/cran.scm (r-calculus): New variable.

Applied with the minor changes below (the URL change was suggested by
‘guix lint r-calculus’.)  Apart from that it’s all good.

Thanks!

Ludo’.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 705 bytes --]

diff --git a/gnu/packages/cran.scm b/gnu/packages/cran.scm
index ce5ae7f64a..fea560f7a1 100644
--- a/gnu/packages/cran.scm
+++ b/gnu/packages/cran.scm
@@ -24615,8 +24615,8 @@ absolute GSEA.")
     (build-system r-build-system)
     (propagated-inputs
      `(("r-rcpp" ,r-rcpp)))
-    (home-page "https://github.com/emanuele-guidotti/calculus")
-    (synopsis "High Dimensional Numerical and Symbolic Calculus")
+    (home-page "https://github.com/guidotti/calculus")
+    (synopsis "High dimensional numerical and symbolic calculus")
     (description
      "Efficient C++ optimized functions for numerical and symbolic calculus.
 It includes basic symbolic arithmetic, tensor calculus, Einstein summing

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-10-17 20:40 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-14 19:06 [bug#43994] [PATCH] gnu: Add r-calculus Magali Lemes
2020-10-15 11:09 ` zimoun
2020-10-16 18:51 ` [bug#43994] [PATCH v2] " Magali Lemes
2020-10-17 20:39   ` bug#43994: " Ludovic Courtès

unofficial mirror of guix-patches@gnu.org 

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://yhetil.org/guix-patches/1 guix-patches/git/1.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 guix-patches guix-patches/ https://yhetil.org/guix-patches \
		guix-patches@gnu.org
	public-inbox-index guix-patches

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://news.yhetil.org/yhetil.gnu.guix.patches


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git