search

UM-ARM-Lab / arm_robots

Packages for controlling our robots at a high-level, though simple to use ROS and Python interfaces.
6 stars 1 forks source link

val or hdt_michigan? #19

Open PeterMitrano opened 3 years ago

PeterMitrano commented 3 years ago

This is probably the single biggest naming problem in our entire code base. All the code HDT wrote calls it "hdt_mihigan" but we have used val in a lot of places. Accepting that it's called hdt_michigan in code is the far easier option, and so we could go forward with this convention. We can still cal it val casually in that case. Or we can use val in code going forward, and eventually drop or rename places it was called hdt_michigan. I'm personally in favor of keeping hdt_michigan but I could be convinced otherwise.

bsaund commented 3 years ago

Starting from scratch, I'd rather call it "val"

PeterMitrano commented 2 years ago

I've just made this problem worse by calling it "husky" in some places...