[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#48046] [PATCH]: Gnu add astropy
From: |
Vinicius Monego |
Subject: |
[bug#48046] [PATCH]: Gnu add astropy |
Date: |
Wed, 19 May 2021 18:16:41 +0000 |
Hi,
Em seg, 2021-04-26 às 20:01 +0000, Sharlatan Hellseher escreveu:
> Hi Guix team!
>
> Here is my attempt to pack bulky python package -
> https://www.astropy.org/
> I've disabled test and add minor modification removing `_compiler.c'
> file.
I've tried to package AstroPy before but got stuck in the dependencies,
more specifically pytest-astropy. There is a bug somewhere that pytest-
filter-subpackage is not found by setup.py, even though it's listed in
the inputs. I encountered that problem again here, after enabling the
tests.
> + ;; NOTE: (Sharlatan-20210426T204315+0100): Tests require build
> astropy
> + ;; module, it needs a good review on how to enable them.
> + `(#:tests? #f
AstroPy uses pytest, which is not listed in your native-inputs. The
other dependencies I had to add are python-hypothesis, python-pytest-
cov, python-pytest-astropy and python-pytest-xdist. According to the
documentation, it should be as simple as running pytest to run the
entire test suite:
https://docs.astropy.org/en/latest/development/testguide.html#running-tests
If you end up in 'ModuleNotFoundError: No module named "astropy"', you
may have to update PYTHONPATH after the install phase with (add-
installed-pythonpath inputs outputs). That usually happens when
replacing the check phase.
> + (inputs
> + `(("asdf" ,python-asdf)
> + ("beautifulsoup4" ,python-beautifulsoup4)
> + ("bleach" ,python-bleach)
> + ("bottleneck" ,python-bottleneck)
> + ("cfitsio" ,cfitsio)
> + ("dask" ,python-dask)
> + ("expat" ,expat)
> + ("graphviz" ,graphviz)
> + ("h5py" ,python-h5py)
> + ("html5lib" ,python-html5lib)
> + ("jplephem" ,python-jplephem)
> + ("matplotlib" ,python-matplotlib)
> + ("mpmath" ,python-mpmath)
> + ("numpy" ,python-numpy)
> + ("pandas" ,python-pandas)
> + ("pyerfa" ,python-pyerfa)
> + ("pytz" ,python-pytz)
> + ("pyyaml" ,python-pyyaml)
> + ("scipy" ,python-scipy)
> + ("sortedcontainers" ,python-sortedcontainers)
> + ("wcslib" ,wcslib)))
AstroPy is a library, its Python inputs should be propagated here.
See
https://guix.gnu.org/manual/en/html_node/package-Reference.html#package-Reference
for reference.
Also, the documentation states that the only strict requirements are
PyERFA and NumPy:
https://docs.astropy.org/en/stable/install.html
IMO we should only propagate these, and leave the rest in native-inputs
as needed by tests.
Now speaking of the astropy dependencies, pytest-astropy is supposed to
propagate its six dependencies. AFAIK its purpose is to be a
metapackage to install these other 6 pytest modules. PyERFA should also
propagate numpy (liberfa can remain as normal input).
I'll take a look at this package again next week. Could you test these
suggestions in the meantime?
Vinicius
- [bug#48046] [PATCH]: Gnu add astropy,
Vinicius Monego <=