Put the locals INSIDE the case you want to change them in. Putting them outside makes them get updated with default values everytime the loop runs and does not execute the "CtrUp" or "CtrDown" cases.
Or you need to explictly set values for the locals in the other cases. The "Use Default if Unwired" is whats killing.
Optimally, you would combine CtrUp and CtrDown in the same event case and use the value of either CtrUp or CtrDown to decide which values the locals got.