This looks like a very useful enhancement. I put some suggestions on github.
A more significant question that came up when I asked pingraham about the driver for this code is that the M1 and M2 air pressure should perhaps be based on the current altitude instead of the target altitude. I don't think we know yet, so it is probably smarter to keep the code simple and use the target for everything (as you are doing). But it is worth noting this question somehow – perhaps has a ticket to evaluate that with on-telescope testing.