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

1.2.0 release #73

Merged
merged 192 commits into from
Mar 20, 2020
Merged

1.2.0 release #73

merged 192 commits into from
Mar 20, 2020

Conversation

MuellerSeb
Copy link
Member

The Release of v1.2.0

[1.2.0] - Volatile Violet - 2020-03-20

Enhancements

Changes

Bugfixes

  • a race condition in the structured variogram estimation has been fixed Update cython #51

MuellerSeb and others added 30 commits November 8, 2019 14:19
* Add hard Python version requirements to package
* Dropped support for Py27 in setup.py
* Remove __future__ 3.0 imports, the future is now
* Drop Py27 and Py34 support in travis
* Update changelog
* Drop Py27 super
* Drop Py27 'new style classes'
* Remove Py27 tempfile hack from setup.py
* remove six as dependency
* CovModel: better handling of single hankel kw updates

* CovModel: N=200 as default for hankel transform
In cython, const's can't be used for local variables,
   only for function interfaces.
The estimator function and the normalization of the variogram are now being
calculated in a separate function. Function pointers are used for the
flexibility of choosing different estimator functions.
The function pointers in cython seem to be a bit limited, therefore I
wrote 2 separate functions choosing the estimator func. and the norm.
func.
In the cython code, merged the 1d-, 2d-, and 3d-versions of the structured
variogram estimators into the 3d version, with a tiny little bit of
preparation code in the python wrappers. Switched off the OpenMP loop
for the moment, as it always results in errors for the 1d case and
sometimes causes problems for higher dimensions. Super strange!
I learnt a lot about OpenMP. It seems very difficult, to adequately
parallelise the structured variogram estimator in Cython. Its OpenMP
support is simply to good enough. I parallelised the inner loop. It's
better than nothing. The function could be implemented in pure C, C++
or maybe Rust.
* [temp] make sphinx gallery

[temp] tutorial 1 migrate

rename tutorial 1 files

[temp] migrate tutorial 2

rename

migrate all tutorials

Reorganize examples

Fix references

Remove old doc example

* Fix typos

* move intro to misc

* finish moving misc

* Ignore MPL agg warnings

* Swap order of tutorial 3

* More gitignores

* CONTRIBUTING: add a hint for adding examples

* Examples: typo fix

Co-authored-by: Sebastian Müller <mueller.seb@posteo.de>
@MuellerSeb MuellerSeb added the enhancement New feature or request label Mar 20, 2020
@MuellerSeb MuellerSeb added this to the 1.2 milestone Mar 20, 2020
@MuellerSeb MuellerSeb requested a review from LSchueler March 20, 2020 16:57
@MuellerSeb MuellerSeb self-assigned this Mar 20, 2020
@MuellerSeb
Copy link
Member Author

CI was skipped since the server is quite busy at the moment and the release candidates succeeded.

Copy link
Member

@LSchueler LSchueler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Go for it!


## Do you want to enhance GSTools or fix something?

- Fork the repo on [GitHub](https://github.com/GeoStat-Framework/GSTools) from the [develop branch](https://github.com/GeoStat-Framework/GSTools/tree/develop).
- Add yourself to AUTHORS.md (if you want to).
- We use the black code format, please use the script `black --line-length 79 gstools/` after you have written your code.
- Add some tests if possible.
- Add an example showing your new feature in one of the examples sub-folders if possible.
Follow this [Sphinx-Gallary guide](https://sphinx-gallery.github.io/stable/syntax.html#embed-rst-in-your-example-python-files)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, you've thought of it all!

@MuellerSeb MuellerSeb merged commit a01da8e into master Mar 20, 2020
@MuellerSeb MuellerSeb deleted the 1.2.0_release branch April 6, 2020 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants