FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Make gstools-core the default and gstools-cython optional by LSchueler · Pull Request #420 · GeoStat-Framework/GSTools · GitHub

Make gstools-core the default and gstools-cython optional - #420

Open
LSchueler wants to merge 2 commits into
mainfrom
new-backend-default
Open

Make gstools-core the default and gstools-cython optional#420
LSchueler wants to merge 2 commits into
mainfrom
new-backend-default

Conversation

Copy link
Copy Markdown
Member

This includes updating the installation instructions.

LSchueler added this to the v1.8 milestone Aug 11, 2026
LSchueler requested a review from MuellerSeb August 11, 2026 19:37
LSchueler self-assigned this Aug 11, 2026
LSchueler added the dependencies Pull requests that update a dependency file label Aug 11, 2026

Copy link
Copy Markdown
Member Author

We now have a few Python wrapper functions, which don't actually do anything anymore. However, they at least show the default arguments in GSTools' code. Otherwise, you would have to check the gstools-core repo...
I'm not sure if that's worth it to keep the wrapper functions.

Copy link
Copy Markdown
Member

I think we can remove the wrapper functions. Defaults are documented in the high-level routines calling these.
Other this looks good to me.

Beside that we now need to get gstools_core onto conda-forge.

LSchueler force-pushed the new-backend-default branch from 977b6ae to 405e3d5 Compare August 24, 2026 11:24

Copy link
Copy Markdown
Member Author

Done.
One thing still to decide: Shall pip install gstools[rust] throw an error or should we keep it for a while in case someone has an elaborate and automatized installation script?

We need gstools_core on conda forge before publishing a new version. So, merging this PR should be fine, as long as we keep that in mind.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL