Common practice is to make the class constants all capital. I understand that alm_name is not following that properly, but it has a story behind it (used to be a function of same name and is used by the base class as well ... the rest are constants withing the scope of this class only, so makes things easier to read if we go all capital.