Xin Zhang

Results 94 comments of Xin Zhang

Ha, it seems that's not used at all. The only appearance is [here](https://github.com/climate-processes/tobac/blob/77c8cc537ea588174981d97d500405a8f94a855b/tobac/themes/tobac_v1/feature_detection.py#L129-L130). But, that's commented. That should be the minimum number of features at one timestamp. BTW, `n_min_threshold` is...

I suppose if there're n_1 features at t_1 and n_2 features at t_2, and the n_min is between n_1 and n_2, then that may cause the missing features at t_2,...

@deeplycloudy Yes, you're right. Because we usually use `m` for grid spacing units and `s` for time step units, the unit of `v_max` is m/s. See the [`get_spacings`](https://github.com/climate-processes/tobac/blob/730fa9affc0b6e7d3facf525faf73cc3a6406e74/tobac/utils.py#L422-L450) in utils....

@freemansw1 It's fine to me except for the clarification of the unit. Leave the review on that PR now ;)

@w-k-jones Thanks for the idea. For polar-orbiting satellite data, the lon/lat, spacing, and spatial extent are all different. Here's an example. I guess I can use geopandas to convert segmentations...

@w-k-jones Thanks for the detailed explanation. I make a simple draft to accomplish your idea by pyresample: The orange grid contains the mask and I get the mask in another...

Note that pyresample can deal with both swath and grid data. It means that the model data, GEO, and polar-orbiting satellite observations can be all supported! The `radius_of_influence` can be...

As mentioned by @JuliaKukulies, it's related to the threshold. But, we don't understand why not all features are included in the masks. If the method is realistic, then it should...

@deeplycloudy Yes, Because np.NaN is float, it forces the entire array of integers to become floating-point numbers. https://github.com/pydata/xarray/issues/6091#issuecomment-998789248

Em ... How could I fork @mheikenfeld 's branch to my current old branch? I come up with creating a new branch and cloning data. But, it sounds like a...