Package Details: python-cartopy 0.24.1-1

Git Clone URL: https://aur.archlinux.org/python-cartopy.git (read-only, click to copy)
Package Base: python-cartopy
Description: A cartographic Python library with Matplotlib support for visualisation
Upstream URL: https://scitools.org.uk/cartopy/
Licenses: BSD-3-Clause
Submitter: gauteh
Maintainer: gauteh (bcb)
Last Packager: bcb
Votes: 10
Popularity: 0.000033
First Submitted: 2015-11-16 16:00 (UTC)
Last Updated: 2024-11-22 12:04 (UTC)

Pinned Comments

bcb commented on 2023-12-08 10:48 (UTC)

The currently enabled tests pass (repeatedly) for me in a clean chroot. I am reluctant to disable too many tests in the PKGBUILD, especially when I cannot reproduce the failures and make sure I am disabling the correct test for a build failure reported by somebody else. For now, if you run into failures you may have to build with --nocheck. I am hoping to make the checks more robust -- suggestions welcome!

Latest Comments

« First ‹ Previous 1 2 3

bcb commented on 2022-07-28 06:25 (UTC) (edited on 2022-07-28 06:26 (UTC) by bcb)

Hi @XavierCLL,

As per the wiki:

check()

Place for calls to make check and similar testing routines. It is highly recommended to have check() as it helps to make sure software has been built correctly and works fine with its dependencies.

Users who do not need it (and occasionally maintainers who can not fix a package for this to pass) can disable it using BUILDENV+=('!check') in PKGBUILD/makepkg.conf or call makepkg with --nocheck flag.

I have disabled a few image comparison tests which always seem to fail. The rest pass for me, on two different computers and in a clean chroot. At some point I would like to find a more reliable arrangement to allow these image comparisons to pass but I am not sure how straightforward that would be as the failures are often very small shifts in position rather than real errors.

In the meantime, I think the remaining ~650 tests are a useful check of the package's behaviour. You can edit /etc/makepkg.conf or use makepkg --nocheck as stated above if you wish to skip the checks on your machine (if you use an AUR helper, see if they have an equivalent option; yay uses --mflags to pass options to makepkg so you could use yay --mflags --nocheck -S python-cartopy for example).

Xavier commented on 2022-07-28 00:46 (UTC)

Hi @gauteh and @bcb, I think running the tests is more for the CI, development cycle and package maintainers but not for users. Right now the tests for this package are generating more cons than pros. Please disable it, thanks

krisboorger commented on 2021-12-10 19:57 (UTC)

The package doesn't build. Two tests fail. Here's the output: https://pastebin.com/c60aWGt2

The images referenced in the error log have a few lines shifted, compared to the ones expected by the test, if that helps with anything.

Downloaded this package using yay, so all dependencies listed here should be installed on my machine.

richli commented on 2019-08-27 23:54 (UTC)

@bcb: Thank you! I've included your hack into the PKGBUILD and disabled the tests for now.

bcb commented on 2019-08-05 12:02 (UTC) (edited on 2019-08-05 12:03 (UTC) by bcb)

Since PROJ (wrapped by python-pyproj) v6 the API cartopy uses has been deprecated. To use it requires a macro defined at build time. There is a pull request open to do so at https://github.com/SciTools/cartopy/pull/1289

In the meantime, this package fails to build. I added a quick hack in the prepare() function to add this define so it builds:

prepare() {
    cd "$srcdir/cartopy-${pkgver}"
    sed -i '/# Main setup/iextra_extension_args["define_macros"] = [("ACCEPT_USE_OF_DEPRECATED_PROJ_API_H", None)]\n' setup.py
}

Some of the tests fail (based on comments in the pull request linked above, I believe this is due to updates in PROJ which need to be reflected in the tests). Build with makepkg --nocheck to skip running the tests.

richli commented on 2018-12-06 23:16 (UTC)

@Stunts: I tried things out in a fresh container. It looks like by default, matplotlib, pillow, and scipy are all additionally required for the test suite. Since those are also probably required for "normal" usage of this package, I'll go ahead and promote them from optional to required dependencies. Thanks for the heads-up!

Stunts commented on 2018-12-06 23:01 (UTC)

@richli: It's required as in "the package fails to build unless it is installed". =-) It is also mentioned in ows.txt in the link you provided.

richli commented on 2018-12-06 18:46 (UTC)

@Stunts: How "required" is it? From what I can tell it's only optional.

https://github.com/SciTools/cartopy/tree/v0.17.0/requirements

Stunts commented on 2018-12-06 15:14 (UTC)

python-pillow is also a required dependency for this package.

Nestor_013 commented on 2018-04-22 14:39 (UTC) (edited on 2018-04-22 14:46 (UTC) by Nestor_013)

A pseudo-diff for version 0.16.0

-pkgver=0.15.0 +pkgver=0.16.0

-md5sums=('98da94b1b1f844b31e679bdfdd6e0d96') +md5sums=('f9e2e528d7758da7c64f824548a53f32')

Full diff@ https://pastebin.com/jySKr0rX

« First ‹ Previous 1 2 3