Change:
(ICE), for Network Address Translator (NAT) traversal for UDP-based communication. <del> Typically, when using ICE, endpoints will exchange candidate addresses, form a list of candidate pairs, and then test each candidate pair to see if connectivity can be established.  If the test for a given pair fails, it is marked accordingly, and if all pairs have failed, the overall ICE process is considered to have failed. </del> <ins> When using ICE, endpoints will typically exchange ICE candidates, form a list of candidate pairs, and then test each candidate pair to see if connectivity can be established.  If the test for a given pair fails, it is marked accordingly, and if all pairs have failed, the overall ICE process typically is considered to have failed. </ins> During the process of connectivity checks, additional candidates may be created as a result of successful inbound checks from the remote

Discussion:
You are right, eventhoug 8445 in some places do say "whenever Ta fires". I'll update the PR. I think it is much easier to talk about "timer Tpac" than "the timer that does this and that". For example, during the ICEbis work, we often talked about "timer Ta". It may not sound like a big differnce at this point, when we are still working on the draft, but I think it will make a difference in the future when/if people start referring to the timer.
NAME NAME wdyt?
I don't think we need this, but if we want to name the timer, I would suggest: Call it "Tpac" and not "Timer Tpac" Not change so much of the existing text. Maybe put in the name (for future conversations) and just replace the once place where we say "the aforementioned timer".
I initially did that, but Justin pointed out that, when comparing to Ta, Ta is just the interval used by a timer. I think that, when we talk about about a timer, we should use the name of the timer.
The first mentions of timers in 8445 says "The gathering process is controlled using a timer, Ta. Every time Ta expires" but then later is says "The generation of ordinary and triggered connectivity checks is governed by timer Ta". So it looks like 8445 isn't self consistent. Meanwhile, we call the RTO timer "timer RTO", so we don't need the "Tfoo" model. How about we just call it "timer PAC", or better yet "the PAC timer"?
I left that part out on purpose :) The Terminology section uses "Timer Ta", though. I want to have a timer name. As I indicated to Justin when we discussed this, it will make future discussions much easier, when we refer to the timer. It will also make life easier for implementers. What about using "timer Tpac" in the first occurrence, and then "Tpac"? BTW, if you are concerned about the number of changes in the PR, note that we have already merged a set of PRs, so the new version of the draft will contain changes from the current version no matter what.
I don't see any reason to use the T prefix when always say "timer" along with it.

It's redundant. I'd prefer "PAC timer or "timer PAC". But I don't care enough to keep talking about it. I'd rather just pick something and move on.
I have no problem with "timer PAC" if you prefer that. All I want is a timer name that people can be refer to.
I think it's very useful to have a name for the timer and both Tpac and PAC timer work for me.
I will merge the pull request. There will still be time (during WGLC, IESG review) to fine tune the timer name, if there is a need to.
There’s still more to do on the PR, but sounds like we have a path forward.
Ok, so I will not merge yet.
Updated the PR to use 'PAC timer', and only where necessary. I'm happy with this if you are.
Looks good.
I think we should give the timer a name. Other ICE timers have names, and it also makes it easier to refer to it in future discussions, drafts, and implementations. I previously suggested Tpac. Tupac would also be cool :)
I created a pull request for this,
Note that 'Ta' in 8445 is the interval, not the timer. It uses 'timer Ta' to refer to the actual timer. I'm not sure this change is an improvement, but open to considering it.LGTM