I think it should be expanded all the way `base_attack_progression`. And no that's not too long. Displaying BASE ATK PROG to the user (if that's a standard shortening) I think is ok, but here in the code it should be fully named IMO. Allows for unambiguous searches of the code (for attack or progression), which can be helpful in refactoring code. It also makes it extra clear to future developers what everything is.