Changes to taproot descriptors

[jnewbery]

A few observations on things that confused workshop particiapnts:

• OP_RIPEMD160 is an unusual opcode. I think OP_HASH160 is more commonly used (eg lightning HTLCs use OP_HASH160). We should replace OP_RIPEMD160 with OP_HASH160 in the notebooks.
• pkhash is confusing since it sounds very similar to the 'pubkey hash' in P2PKH. Consider renaming it, perhaps to pk_and_hash or pk_with_hash?
• hasholder and pkhasholder are slightly confusing since most people seem to parse them as has holder and pk has holder.

[jachiang]

Agree with above points. Some ideas:

• pk_delay (nsequence delay)
• pk_time (timelock)
• ...pk_hashlock_delay

[jnewbery]

I like pk_hashlock and pk_delay. (pk_time seems fine but isn't relevant to these notebooks).

[jachiang]

@jnewbery, just to clarify before I make the changes in the library and chapters. Our tapscript descriptor tags would be as follows (with hash-locks using OP_HASH160), right?

• pk
• pk_delay
• pk_hashlock
• pk_hashlock_delay
• csa
• csa_delay
• csa_hashlock
• csa_hashlock_delay

[jnewbery]

Our tapscript descriptor tags would be as follows

Yes, that looks good to me.

[jachiang]

I will work on this first thing tomorrow, unless you already are @jnewbery?

[jnewbery]

I will work on this first thing tomorrow, unless you already are @jnewbery?

Thanks! I'm not working on it.

[jnewbery]

Fixed in #138