Michael Kavulich
Michael Kavulich
I can see this being a useful feature, but I can not comment on it being needed for NCO/operational reasons. I have no plans to work on this, not sure...
I also experienced this issue with the following settings: ``` pylint 3.3.6 astroid 3.3.9 Python 3.12.10 ``` and ``` pylint 3.2.7 astroid 3.2.4 Python 3.12.10 ``` When I downgraded to...
@natalie-perlin please respond to [these suggested changes](https://github.com/ufs-community/ufs-srweather-app/pull/1201#discussion_r1974410537), after that I can approve.
@natalie-perlin I'm not seeing the restored `field_table.FV3_RRFS_v1beta` and `diag_table.FV3_RRFS_v1beta`, have you pushed all your changes?
> Instead of having seperate plotting functions at this stage, what if we instead expose the sampling that we do, which can be passed into whichever function the user desires?...
@MichaelLueken @natalie-perlin Since I'm not properly communicating my desired changes to this script, I felt it best to just make those changes and propose them as a PR to this...
I'm a bit conflicted on the merge destination here: While I would love to merge directly to `main` here to skip the requirements of needing to run all tests for...
@philipc2 Thanks for the quick response! Making this change appears to result in a different set of bad data: I see that the new transform automatically stitches the period elements...
>I'd suggest using to_raster() if you are looking for quicker plots. We only really suggest using the PolyCollection for small/exploratory visualizations. Okay thanks, I suspected that it was deprecated but...
@dustinswales Can you take a look at this to make sure it's fine by you?