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

FIX: Update xradar branch to be even with main #1455

Closed
wants to merge 113 commits into from
Closed

FIX: Update xradar branch to be even with main #1455

wants to merge 113 commits into from

Conversation

mgrover1
Copy link
Collaborator

Update the xradar branch with latest main branch

mgrover1 and others added 30 commits July 7, 2022 19:55
* updated join_radar to also append instrument parameters

* updated join_radar to also append instrument parameters

* add pypi stats to readme (#1206)

* updated join_radar to also append instrument parameters

* updated join_radar to also append instrument parameters

* added unittest for join_radar

Co-authored-by: Max Grover <[email protected]>
* TST: Moving pyart main module tests to test directory.

* TSTS: Move bridge and core submodules to respective test directories.

* TSTS: Moving correct and filters tests to respective tests directories.

* TST: Moving graph io and map test modules.

* TST: Moving retrieve, testing and util submodule tests.

* FIX: Remove pyargs

* FIX: Fix pytest call.

* FIX: Remove extra netcdf reference.

* MNT: Remove duplicate pytest as well.

* MNT: Changing custom config path.

* FIX: Fix for config loading and not resetting.

* TST: Adding back in test_simualted_vel.

* FIX: Fix for missing sys paths.

* FIX: Attemped PIL fix.

* MNT: Remove pillow.
* FIX: Fix for only python 3.10 being used.

* FIX: Adding python call.

* MNT: Another attempt.

* FIX: Xarray pin removed.

* MNT: Remove cftime pin.

* MNT: Attemped fix for pil.

* MNT: Accidently removed gdal

* MNT: Pinning for now.

* MNT: Remove lib set.

* CI: Remove python 3.6 from testing.
…up doc (#1213)

* DOC: Updating Py-ART's how to release doc and removing most of ci_setup.

* DOC: Changes reflecting review.
* ENH: Corrected Height Gates to be height above sea level

* FIX: Removed character within comment line

* ENH: Updated Height Test to include altitude of radar.
…r simple moments. (#1214)

* TST: Fix median filter depreciation warnings as well as more tests for
simple moments.

* TST: Update values.

* MNT: Adding variables for simple moment calculation functions.

* FIX: Removal of depreciation warning for interp namespace.
* add pyrad to the readme

* move pyrad to extension section

* add space between _ and :
#1226)

* FIX: Return zero phase for sweeps completely filtered out with gatefilter.

* FIX: Remove debug statement
* TST: To see if trmm_rsl broke.

* MNT: Forgot final else.

* MNT: Remove full path.

* add hardcoded trmm_rsl path to docs

* add missing RSL_PATH=

* upin cftime

* build docs on macos

* update to macos hash

* add macos latest runner for docs

* add macOS-latest

* move toward micromamba

* change path to micromamba

* add ls and move to mamba exe

* try pkgs dir

Co-authored-by: mgrover1 <[email protected]>
Co-authored-by: Max Grover <[email protected]>
…nd more. (#1231)

* DOC: Updated pyart doc to include column code, incorrect syntax and
more.

* DOC: Forgot https

* MNT: Revert ssh to https

* MNT: Suggested changes.
* added cut_radar function in util

* added cut_radar function in util

* renamed cut_radar to subset_radar

* MNT: Update ci.yml.

* MNT: Updating environments.

* MNT: Duplicate 'latest'

* MNT: Revert back os runner check.

* MNT: Add bash to each run

Co-authored-by: zssherman <[email protected]>
* CI: Remove trmm_rsl from testing.

* MNT: Remove trmm_rsl from debug and install warning build message.

* CI: Remove trmm_rsl pathing from CI.

* MNT: Remove warning from setup.py

* remove trmm_rsl from doc env

* MNT: Don't code cover deprecated code.

* replace dealias with region based

* use cfradial with rhi two panel

* add new plot ppi cfradial remove old one using sigmet

* use cfradial with xsect example

Co-authored-by: mgrover1 <[email protected]>
* add column example to docs

* condense some lines
* inital comp_z commit

* Update tests/retrieve/test_comp_z.py

yup, this is what happens when I do code development in a Jupyter notebook ....

Co-authored-by: Max Grover <[email protected]>

* review changes suggested by Max

* ENH: First cleanup, add gatefilter, pep8, add field parameter, remove
unused parameters.

* MNT: Cleanup of radar util docs.

* TST: Update unittesting to represent new changes.

* DOC: Update doc for field parameter.

* MNT: Add radar class in.

* FIX: Forgot core Radar import.

* make interpolation more generic

* ensure tests are updated to use radar object

* ensure dtype is consistent for compz

* fix bug with the dtype

* ensure arrays are numpy32 bit

* FIX: Fix for newer matplotlib version.

* add an example for composite reflectivity

* fix the title for the composite reflectivity example

Co-authored-by: Max Grover <[email protected]>
Co-authored-by: mgrover1 <[email protected]>
Co-authored-by: zssherman <[email protected]>
* ADD: Add composite reflectivity citation

* fix header of the composite reflectivity example
)

* MNT: Update to find namespace packages to fix warning.

* MNT: Update syntax for registering colormaps to remove dep warnings.
* ADD: Add changelog from previous releases

* add changelog for v1.13.0
* update environment to include cibuildwheel

* add cibuildwheel workflow

* add check for history

* fix spacing in wheels workflow

* add setuptools_scm

* add pytest command

* remove requires manifest

* change odd xarray requirements

* add path to tests

* use cibuildwheel example verbatim

* add pip install netcdf4

* Update .github/workflows/wheels.yml

Co-authored-by: Clément Robert <[email protected]>

* Update .github/workflows/wheels.yml

Co-authored-by: Clément Robert <[email protected]>

* Update .github/workflows/wheels.yml

Co-authored-by: Clément Robert <[email protected]>

* update build systems

* Add additional flags

* Only use 3.8 to 3.10 wheels

* Add pip only binary flag

* Remove duplicated statement

* MNT: Test 64 for windows build, not definite.

Co-authored-by: Clément Robert <[email protected]>
Co-authored-by: zssherman <[email protected]>
joshua-hampton and others added 27 commits March 16, 2023 14:29
* Add colorbar_orient option

* Added colorbar_orient option to plot_ppi_map
* ADD: Add cfradial 1.4 version to io

* specify version 1.3
* ADD: Add cfradial 1.4 version to io

* specify version 1.3

* Add first part of xradar-pyart integration

* pin older version of pandas
* Add xradar as a dependency

* Add xradar in environment.yml file
…f field not in destination radar (#1418)

* FIX: GateMapper will now map source field to destination radar even if field does not exist in destination.

* FIX: Whitespace

---------

Co-authored-by: Robert Jackson <[email protected]>
* Update pre-commit hook versions

* MNT: Update changelog to reflect new versions

* fix new linting updates
* Ensure units are in feet for nexrad references

* Fix units for altitude tests
* MNT: Remove pandas pinning, testing.

* FIX: Fix for format error for pandas.

* STY: PEP8 fix.

* MNT: Update to handle different formats.

* add ablog to path

* unpin pandas in doc build

* move to new github action

* move ablog to pypi install

---------

Co-authored-by: mgrover1 <[email protected]>
* Create codeql.yml

* Update codeql.yml

Removing cpp scanning

* migrate to new github action

---------

Co-authored-by: mgrover1 <[email protected]>
…nd example (#1434)

* Fix spacing in documentation

* Fixing overlapping plots in example

* Fix minor issue causing arrays to be incorrectly masked

This fix prevents the final convective stratiform array from being masked by the background field. Due to the nature of the `calc_thres` (minimum number of points needed to calculate the background average), the background field does not compute on the edges of the echo. Instead, we want the convective stratiform field to cover the same region as the original reflectivity field.

* Fix lint error

* Fix linting error

* Fixing plot overlap in example
* Fix linting errors

* MNT: Pin to older cython

* try suggested fix for cython file

* fix location of parentheses

* Set the cpower argument

* remove equals float

* pin newer cython

* remove cython from ci env

* try previous version of libnetcdf
* FIX: Add isinstance checks to improve linting

* fix other linting errors with f-strings
…nd bug fixes to the load_kazr method (#1446)

* FIX: change deprecated np.str in arm_vpt to str

* Update default wind_size in calculate_velocity_texture from 4 to 3
Using an odd number prevents single index offsets in placement in
case of an even size

* ADD: optional rectangular window dims for velocity texture analysis.
Also, modify the default window size to 3 instead of the previous
value of 4 (even number resulting in a potential offset).

* FIX: nyquist velocity dtype in radar obj loaded with read_kazr

* pep8

* line reformatting per black

* TST: unit tests for rectangular texture window

* FIX: Add in linting fixes

---------

Co-authored-by: mgrover1 <[email protected]>
* FIX: Add isinstance checks to improve linting

* FIX: Fix the RSL interface for cython 3
* FIX: Add isinstance checks to improve linting

* FIX: Fix the isinstance test
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@mgrover1 mgrover1 deleted the branch xradar September 15, 2023 13:50
@mgrover1 mgrover1 closed this Sep 15, 2023
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

Successfully merging this pull request may close these issues.