Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improving pyroms installation: csa #34

Open
hadfieldnz opened this issue Apr 8, 2021 · 0 comments
Open

Improving pyroms installation: csa #34

hadfieldnz opened this issue Apr 8, 2021 · 0 comments

Comments

@hadfieldnz
Copy link
Contributor

Hi Kate

I am continuing with my programme of rationalising and/or streamlining and/or documenting the components that make up pyroms and I have come to csa (pyroms/external/csa). From the contents of this directory, this is Pavel Sakov's Bivariate Cubic Spline approximation library + standalone utility, Version 1.21. The make file installs a library (libcsa.a), a binary (csabathy), a header file (csa.h) and maybe some other stuff, but not a Python module. csa is involved in the build process for gridutils (pyroms/external/gridutils/makefile, etc) and is also mentioned in a comment in pyroms/external/nn/delaunay.c, but not actually used.

There is now a Conda csa package on conda-forge (https://github.com/conda-forge/csa-feedstock) referring to the home page at https://github.com/sakov/csa-c. The Conda package is at version 1.22.0. When installed, it gives the library, the binary and the header file.

I therefore propose to delete the csa subdirectory from the pyroms repository, but mention csa in the README file as an optional Conda package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant