Skip to content

Commit

Permalink
Release/0.5.0 (#93)
Browse files Browse the repository at this point in the history
Prepare a release of fv3gfs-python. See HISTORY.md for more details about this release.

Changes:
- Update fv3gfs-fortran submodule to current master
- Update fv3util sub-module to v0.5.1
- Cleanup bug fixes from moving fv3util to a separate submodule
  - Fixed bumperversion configuration
  - Avoid triggering fv3util tests (which were failing). This means that the gt4py numpy backend is currently not being tested anywhere.
  • Loading branch information
nbren12 authored Jul 28, 2020
1 parent fe4fdbe commit 82c5a5a
Show file tree
Hide file tree
Showing 7 changed files with 13 additions and 20 deletions.
7 changes: 7 additions & 0 deletions HISTORY.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,14 @@ History
latest
------


v0.5.0 (2020-05-15)
------

Breaking changes:
- fixed a bug where atmosphere hybrid coordinates were incorrectly marked as cell center variables, and were missing one point
- `send_buffer` and `recv_buffer` are modified to take in a `callable`, which is more easily serialized than a `numpy`-like module (necessary because we serialize the arguments to re-use buffers), and allows custom specification of the initialization if zeros are needed instead of empty.
- use remote data for restart conditions

Major changes:
- Added getters and setters for additional dynamics quantities needed to call an alternative dynamical core
Expand All @@ -18,6 +23,8 @@ Minor changes:
- Internally, get_state uses the new QuantityFactory class to initialize quantities
- fixed bug in fv3util setup.py which prevented `python setup.py install` from copying submodules
- fixed bug in dockerfile where gs://vcm-fv3config data was downloaded to incorrect location
- New hpc dockerfiles
- aquaplanet configuration

Deprecations:
- `Quantity.values` is deprecated
Expand Down
2 changes: 1 addition & 1 deletion external/fv3util
2 changes: 1 addition & 1 deletion fv3gfs/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,4 +47,4 @@
GridSizer,
)

__version__ = "0.4.3"
__version__ = "0.5.0"
2 changes: 1 addition & 1 deletion lib/external
10 changes: 1 addition & 9 deletions setup.cfg
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
[bumpversion]
current_version = 0.4.3
current_version = 0.5.0
commit = True
tag = False

Expand All @@ -11,14 +11,6 @@ replace = PACKAGE_VERSION = "{new_version}"
search = __version__ = "{current_version}"
replace = __version__ = "{new_version}"

[bumpversion:file:external/fv3util/fv3util/__init__.py]
search = __version__ = "{current_version}"
replace = __version__ = "{new_version}"

[bumpversion:file:external/fv3util/setup.py]
search = version="{current_version}"
replace = version="{new_version}"

[bdist_wheel]
universal = 1

Expand Down
4 changes: 2 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
# This line only needed if building with NumPy in Cython file.
from numpy import get_include

PACKAGE_VERSION = "0.4.3"
PACKAGE_VERSION = "0.5.0"

fv3gfs_build_path_environ_name = "FV3GFS_BUILD_DIR"
make_command = os.environ.get("MAKE", "make")
Expand Down Expand Up @@ -69,7 +69,7 @@ class BuildDirectoryError(Exception):
"xarray>=0.13.0",
"netCDF4>=1.4.2",
"numpy",
f"fv3util=={PACKAGE_VERSION}",
f"fv3util>=0.5.1",
]

setup_requirements = ["cython", "numpy", "jinja2"]
Expand Down
6 changes: 0 additions & 6 deletions test_docker.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,16 +12,10 @@ pytest ./tests/image_tests/*.py

if [[ "GOOGLE_APPLICATION_CREDENTIALS" == "" ]]
then
docker run -it $DOCKER_IMAGE bash -c "mpirun --oversubscribe -n 6 pytest /fv3gfs-python/external/fv3util/tests/test_mpi_mock.py"
docker run -it $DOCKER_IMAGE bash -c "make -C /fv3gfs-python/external/fv3util test test_mpi"
docker run -it $DOCKER_IMAGE bash -c "cd /fv3gfs-python; make test"
docker run -it $DOCKER_IMAGE bash -c "pytest /fv3gfs-python/external/fv3config/tests"
else
docker run -it $DOCKER_IMAGE bash -c "mpirun --allow-run-as-root --oversubscribe -n 6 pytest /fv3gfs-python/external/fv3util/tests/mpi"
# needed for circleci tests on machine executor, even though we're accessing public data only
docker run -v $GOOGLE_APPLICATION_CREDENTIALS:$GOOGLE_APPLICATION_CREDENTIALS \
--env GOOGLE_APPLICATION_CREDENTIALS=$GOOGLE_APPLICATION_CREDENTIALS \
-it $DOCKER_IMAGE bash -c "make -C /fv3gfs-python/external/fv3util test test_mpi"
docker run -v $GOOGLE_APPLICATION_CREDENTIALS:$GOOGLE_APPLICATION_CREDENTIALS \
--env GOOGLE_APPLICATION_CREDENTIALS=$GOOGLE_APPLICATION_CREDENTIALS \
-it $DOCKER_IMAGE bash -c "cd /fv3gfs-python; make test"
Expand Down

0 comments on commit 82c5a5a

Please sign in to comment.