TXN_DATA redesign
#173
TXN_DATA redesign
#173
Conversation
|
|
|
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
rlp_txn_v2/columns/transaction.tex
Outdated
There was a problem hiding this comment.
why is there modif here ? Some of them are shared columns already
There was a problem hiding this comment.
why is there modif here ? Some of them are shared columns already
There was a problem hiding this comment.
has to be deleted
txn_data_user/_all_txn_data_user.tex
Outdated
There was a problem hiding this comment.
has to be renaled txn_data
txn_data_user/_inputs.tex
Outdated
| \subsection{Introduction} \label{user txn data: generalities: intro} \input{intro} | ||
| \subsection{Columns} \label{user txn data: generalities: columns} \input{columns} | ||
| \section{Transaction data module} \label{user txn data: generalities} | ||
| \subsection{Introduction} \label{user txn data: intro} \input{intro} |
There was a problem hiding this comment.
In the intro,
has to be updated
There was a problem hiding this comment.
The picture doesn't load for me
| % $\txnDataRlpNumAccountDelegations$: | ||
| % number of account delegations | ||
| % for \cite{EIP-7702} transactions; | ||
| \item |
There was a problem hiding this comment.
we don't need it in txn_data, the lookup is rlp_txn into blk
rlp_txn_v2/columns/transaction.tex
Outdated
There was a problem hiding this comment.
why is there modif here ? Some of them are shared columns already
There was a problem hiding this comment.
only reordering right ?
There was a problem hiding this comment.
No, finishing the lookup which was incomplete.
The first one is already done here IIRC. I would rather do the other ones in separate PR's. |
* feat: adding WCP and EUC specific result columns * feat: update "RES" logging in shorthands * ras
No description provided.