usmap icon indicating copy to clipboard operation
usmap copied to clipboard

Revisit function naming

Open pdil opened this issue 6 years ago • 2 comments

There are now a lot more functions within usmap than when the project was started. There should be some thought given to the naming of these functions going forward (before we reach a stable 1.0 release).

If the functions are to be changed, they will be renamed by v0.7.0 and the old function names will remain but be deprecated until v1.0.0 (where they will be removed).

Current Name Proposed Name
fips
fips_info
map_with_data
plot_usmap
usmap_crs
usmap_transform
usmap

pdil avatar Sep 25 '19 19:09 pdil

I recommend changing usmap_transform to something different that does not include "usmap". This is confusing as it implies I am transforming your map, and not my data frame to conform to your maps.

trashbirdecology avatar Jan 14 '20 15:01 trashbirdecology

Changed the target release for this change from v0.6.0 to v0.7.0 since usmapdata changes are going into v0.6.0. This allows for more separation of large changes between multiple releases.

pdil avatar Feb 09 '22 14:02 pdil

Since it's been so long since I first came up with this idea I've decided to put it on hold since the functions have been named this way for many years at this point and there will be some friction to re-learning any new names, even with a deprecation schedule.

I may revisit this in the future, especially after 1.0 (if usmap ever even reaches that version number 🤪).

pdil avatar Jan 12 '24 19:01 pdil