You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As shown in #171, six get_*() functions are possible "values" of feature argument in map_dep(), while five other get_* functions are not. This is far from ideal. We would expect the same behavior for all get_* functions.
Proposal of @peterdesmet which I agree with: rename the get_*() functions used as feature values in map_dep() to summarize_*():
This issue has been indirectly tackled during the camtraptor July 2023 coding sprint. See all new issues made by @peterdesmet starting with "Rename" or "Combine".
As shown in #171, six
get_*()
functions are possible "values" of feature argument inmap_dep()
, while five otherget_*
functions are not. This is far from ideal. We would expect the same behavior for all get_* functions.Proposal of @peterdesmet which I agree with: rename the
get_*()
functions used asfeature
values inmap_dep()
tosummarize_*()
:Deprecation of the correspondent get_* functions is still possible of course.
To be discussed with @jimcasaer tomorrow
The text was updated successfully, but these errors were encountered: