Feature/tc forecast max radius

[ThomasRoosli]

This changes the functionality of generating a TCForecast fetched from ECMWF tropical cyclone forecasts stored in bufr files. It now fills the radius_max_wind attribute. It calculates that radius as the distance of the provided "location of maximum windspeed" from the storm center.
I am happy for a review.
Best,
Thomas

[peanutfun]

@manniepmkam Could you give this a look?

[manniepmkam]

This pull request looks all good to me apart from the unit conversion is still a bit unclear to me (see the above comments)

[peanutfun]

the unit conversion is still a bit unclear to me (see the above comments)

@manniepmkam The "above comments" seem to have been dropped, at least I cannot see them. What do you mean by "unit conversion"?

[peanutfun]

@manniepmkam Can you give this another look?

[manniepmkam]

Yes I have checked again the unit conversion (line 496 in tc_tracks_forecast.py), it's totally fine now.

[peanutfun]

Looks good, apart from some leftover commented code.

[ThomasRoosli]

Thanks for the reviews @peanutfun and @manniepmkam. The last open change requested was now implemented: it was the removal of some commented lines and it was already preparded in a commit by @peanutfun. As this was a very small change, I now assume every thing is fine and I will merge this PR.