guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#62801] [PATCH 0/4]: gnu: python-astroalign: Update to 2.4.2.


From: Maxim Cournoyer
Subject: [bug#62801] [PATCH 0/4]: gnu: python-astroalign: Update to 2.4.2.
Date: Wed, 19 Apr 2023 21:58:22 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)

Hello,

Sharlatan Hellseher <sharlatanus@gmail.com> writes:

> * gnu/packages/astronomy.scm (python-astroalign): Update to 2.4.2.
>   [source]: Use git checkout instead PyPI to enable tests.
>   [build-system]: Use pyproject-build-system.
>   [propagated-inputs]: Migrate from inputs. Remove labels. Add optional 
> python-bottleneck.
>   [native-inputs]: Add python-astropy, python-ccdproc, python-pillow.

There shouldn't be a hanging indent (leading whitespace) in GNU change
logs.  Please wrap lines at 80 chars.

> ---
>  gnu/packages/astronomy.scm | 30 +++++++++++++++++-------------
>  1 file changed, 17 insertions(+), 13 deletions(-)
>
> diff --git a/gnu/packages/astronomy.scm b/gnu/packages/astronomy.scm
> index 9961e19695..0d7212791a 100644
> --- a/gnu/packages/astronomy.scm
> +++ b/gnu/packages/astronomy.scm
> @@ -3070,22 +3070,26 @@ (define-public python-gwcs
>  (define-public python-astroalign
>    (package
>      (name "python-astroalign")
> -    (version "2.3.1")
> +    (version "2.4.2")
>      (source
>       (origin
> -       (method url-fetch)
> -       (uri (pypi-uri "astroalign" version))
> +       ;; There are no tests in the PyPI tarball.
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://github.com/quatrope/astroalign";)
> +             (commit (string-append "v" version))))
> +       (file-name (git-file-name name version))
>         (sha256
> -        (base32 "19qzv3552lgrd9qmj0rxs51wmx485hw04cbf76ds5pin85kfaiy1"))))
> -    (build-system python-build-system)
> -    (arguments
> -     ;; TODO: (Sharlatan-20210213T162940+0000): I could not make tests run
> -     `(#:tests? #f))
> -    (inputs
> -     `(("numpy" ,python-numpy)
> -       ("scikit-image" ,python-scikit-image)
> -       ("scipy" ,python-scipy)
> -       ("sep" ,python-sep)))
> +        (base32 "0hly20a65540hr3l1lsd1i4d90a0vdrbwnn6zx3z8s89ha9lq3pb"))))
> +    (build-system pyproject-build-system)
> +    (native-inputs
> +     (list python-astropy python-ccdproc python-pillow))

This line could fit on one line: (native-inputs (list python-astropy ...))

> +    (propagated-inputs
> +     (list python-bottleneck
> +           python-numpy
> +           python-scikit-image
> +           python-scipy
> +           python-sep))
>      (home-page "https://astroalign.readthedocs.io/";)
>      (synopsis "Astrometric Alignment of Images")
>      (description

Well done!  Could you please send a v2 with my smallish comments
addressed? :-)

-- 
Thanks,
Maxim





reply via email to

[Prev in Thread] Current Thread [Next in Thread]