Replace DeprecationWarning for FutureWarning #305
Merged
Conversation
This file contains 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
6 tasks
jessepisel
added a commit
to jessepisel/verde
that referenced
this pull request
Apr 13, 2021
* Fix typo on "coordinates" (fatiando#306) Fix a common typo on Verde: replace "coordiantes" for "coordinates". * Get INSTALL_REQUIRES from requirements.txt (fatiando#312) On setup.py, build the INSTALL_REQUIRES variable by reading the requirements.txt file. * Include requirements.txt in MANIFEST.in (fatiando#313) * Exclude Dask 2021.03.0 as a dependency (fatiando#311) Dask 2021.03.0 was causing the tests to fail under Python 3.8 on any OS. The problem seems to be originated by dask.distributed. By excluding this version, we avoid the problem in the future. * Replace Travis and Azure for GitHub Actions (fatiando#309) Remove Azure and Travis configuration files. Add Github Actions for continuous integration and deployment. Related to fatiando/maintenance#1 * Replace versioneer for setuptools-scm (fatiando#307) Replace `versioneer` with `setuptools_scm` for getting Semver version of Verde. `setuptools_scm` doesn't require to store additional files to work, it can be installed and used through `setup.py`. Remove all `versioneer` related files and mentions. Add `setuptools_scm` to `requirements.txt` and `environment.yml`. Replace `setup.cfg` for `.flake8`. Related to fatiando/maintenance#4 * Replace DeprecationWarning for FutureWarning (fatiando#305) `DeprecationWarning`s are intended for developers, while the warnings we want to raise are to let users know that a feature will be deprecated is the `FutureWarning`. Remove the override of `warning.simplefilter`. Following the comments in fatiando#293 Fixes fatiando#290 * Add license notice to every source file (fatiando#308) Add a license notice to every source file and a script for automatically check if the license is present on every file and add it if missing. * Update files from fatiando/contributing (fatiando#314) Changes have been made in https://github.com/fatiando/contributing to: MAINTENANCE.md Update the copies in this repository to match. See fatiando/community@b78e925 * Add .eggs to .gitignore (fatiando#315) Ignore the .eggs directory that is created after `make install` * Allow make_xarray_grid to take horizontal coordinates as 1d arrays (fatiando#300) Now make_xarray_grid could take 1d arrays for horizontal coordinates, such as how xr.DataArray stores the single dimension coordinates. Add a new non-public meshgrid_to_1d function to convert 2d meshgrids (like the ones generated by vd.grid_coordinates) to 1d arrays containing a single coordinate of the corresponding axis. This function checks if all elements inside the 2d array are equal along the right axis, i.e. a vaild meshgrid. * Add changelog entry for v1.6.0 (fatiando#316) * Fix wrong version numbers for PyPI releases (fatiando#317) For test releases to TestPyPI we have to edit the setuptools_scm configuration to produce valid version numbers. But it turns out that this breaks the number for actual releases. To fix it, only edit the configuration on non-release builds. * Allow make_xarray_grid to get data as None (fatiando#318) Used to generate a `xr.Dataset` without any `data_vars` array, containing only the coordinates. Will be useful in some cases to define an `xr.Dataset` with coordinates only and add the `data_vars` afterwards. * Changelog entry for v1.6.1 (fatiando#320) Minor release to include a small fix to make_xarray_grid * Update files from fatiando/contributing (fatiando#321) Changes have been made in https://github.com/fatiando/contributing to: MAINTENANCE.md Update the copies in this repository to match. See fatiando/community@26724ee * Extend support for Python 3.9 (fatiando#323) Add Python 3.9 to the CI and setup.py flags Co-authored-by: Santiago Soler <santiago.r.soler@gmail.com> Co-authored-by: Fatiando a Terra Bot <50936856+fatiando-bot@users.noreply.github.com> Co-authored-by: Leonardo Uieda <leouieda@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DeprecationWarnings are intended to developers, while the warnings wewant to raise to let users know that the feature they are using will be
deprecated is the
FutureWarning.Remove the lines that override the
warning.simplefilter.Fixes #290
Following the comments in #293
Reminders:
make formatandmake checkto make sure the code follows the style guide.doc/api/index.rstand the base__init__.pyfile for the package.AUTHORS.mdfile (if you haven't already) in case you'd like to be listed as an author on the Zenodo archive of the next release.