Documentation issue with wradlib.clutter.classify_echo_fuzzy

18 views
Skip to first unread message

Sylvain Watelet

unread,
Nov 15, 2021, 11:39:34 AM11/15/21
to wradlib-users
Dear wradlib users,

This is my first message on this forum, so I will briefly introduce myself: I work as a scientist at the Royal Meteorological Institute of Belgium (RMI), on the project PRECIP-TYPE aiming at mapping in real-time the precipitation type.

I recently tested this function https://docs.wradlib.org/en/stable/generated/wradlib.clutter.classify_echo_fuzzy.html#wradlib.clutter.classify_echo_fuzzy and according to its documentation, the 3 first decision variables are textures of radar variables, and the decision variables should be put in "dat". But when I look at the code on Github (https://github.com/wradlib/wradlib/blob/main/wradlib/clutter.py , l471), these textures are computed. My understanding is that there is a mistake in the documentation and that "dat" should not contain textures. Can someone confirm?

Thank you in advance!

Cheers,
Sylvain Watelet

Kai Mühlbauer

unread,
Nov 15, 2021, 12:07:38 PM11/15/21
to wradli...@googlegroups.com
Dear Sylvain,

First of all, welcome to the wradlib user community. It's great to see that RMI is continuing to use and contribute to wradlib.

From looking at the code history, the texture has always been computed inside the function for the respective moments. At some point in time the docstring has changed (about 7 years ago) to mention textures.

So I would conclude the same, the docstring should probably mention that the texture is calculated inside the function and the moments should be given as is.

Our colleagues from KNMI (Aart, Tiemo) are using this functionality a lot operationally, so they might chime in here.

To fix this, you might open an issue on github and/or a pull request.

Cheers,
Kai
--
Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.
Reply all
Reply to author
Forward
0 new messages