njtierney / maxcovr

Tools in R to make it easier to solve the Maximal Coverage Location Problem
http://maxcovr.njtierney.com/
GNU General Public License v3.0
42 stars 11 forks source link

rename variables in `max_coverage` #27

Closed njtierney closed 6 years ago

njtierney commented 7 years ago

Currently there are some rather poorly named variables like c and d and Nx and Ny.

This makes it hard to work with in the future. I need to give these better more descriptive names.

njtierney commented 7 years ago

Additionally, these are copied for no real reason later on.

njtierney commented 6 years ago

The names c and d have been changed to c_vec and d_vec.

The variables not needed in max_coverage have been removed, but Nx and Ny have been kept