Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace all \$([ceg-hmnz])\$ with \$\mathrm{$1}\$ using Libre Open Office with match case and regular expressions ticked as options. #626

Open
wants to merge 684 commits into
base: master
Choose a base branch
from

Conversation

jamesray1
Copy link
Contributor

No description provided.

jamesray1 and others added 30 commits January 30, 2018 11:14
Using Libre Open Office with match case and regex.
Add \pagebreak[1]s which removes many empty lines
Add Davey2002_zbMATH01748069 ref + typo + List of mathematical symbols
hypertargets lines 871–882 exec params
\hypertarget{normal_halting_function_H}{normal_halting_function_H}
changed ordering of receipt items to correct client implementation
pirapira and others added 25 commits February 16, 2018 12:31
… match math order; ", \eg by \cite{ECDSAcerticom}."
\hyperlink{execenv}{execution environment} and linkdest
providing ``consumers'' with the ability to make micro-payments
`set -e` seems to be non-effective in Travis build
change ordering of ECDSAPUBKEY, ECDSASIGN and ECDSARECOVER in text to…
Make sure that Travis fails when latex fails
…fice with regular expressions ticked as an option.
@jamesray1 jamesray1 changed the title dummy Replace all \$([ceg-hmnz])\$ with \$\mathrm{$1}\$ using Libre Open Office with regular expressions ticked as an option. Feb 20, 2018
@jamesray1 jamesray1 changed the title Replace all \$([ceg-hmnz])\$ with \$\mathrm{$1}\$ using Libre Open Office with regular expressions ticked as an option. Replace all \$([ceg-hmnz])\$ with \$\mathrm{$1}\$ using Libre Open Office with match case and regular expressions ticked as options. Feb 20, 2018
@@ -181,7 +181,7 @@ \section{Conventions}\label{ch:conventions}

Tuples are typically denoted with an upper-case letter, \eg $T$, is used to denote an Ethereum transaction. This symbol may, if accordingly defined, be subscripted to refer to an individual component, \eg \hyperlink{transaction_nonce}{$T_{\mathrm{n}}$}, denotes the nonce of said transaction. The form of the subscript is used to denote its type; \eg uppercase subscripts refer to tuples with subscriptable components.

Scalars and fixed-size byte sequences (or, synonymously, arrays) are denoted with a normal lower-case letter, \eg $n$ is used in the document to denote a \hyperlink{transaction_nonce}{transaction nonce}. Those with a particularly special meaning may be Greek, \eg $\delta$, the number of items required on the stack for a given operation.
Scalars and fixed-size byte sequences (or, synonymously, arrays) are denoted with a normal lower-case letter, \eg $\mathrm{n}$ is used in the document to denote a \hyperlink{transaction_nonce}{transaction nonce}. Those with a particularly special meaning may be Greek, \eg $\delta$, the number of items required on the stack for a given operation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this edit is mistaken about the normal lower-case letter. Scalars and fixed-size byte sequences should be typeset with italic.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, OK, I guess I changed it like this (overlooking this part) because n, etc, are also used as a field, e.g. of transactions, accounts, blocks, where they are typeset in roman font. So it seems like a bit of a contradiction that they are typeset differently.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's keep it inconsistent for now, and tidy up the document first.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So what, just leave this PR open and make a decision whether to merge or close it some time later? Why not make a decision now? I guess we want to avoid merge conflicts.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you want a decision now, here

  • scalar or fixed-length without subscripts $x$
  • scalar or fixed-length component of a record $T_{\mathrm{n}}$
  • scalar of fixed-length component of an array specified by an index $R_i$
  • variable-length $\mathbb{x}$
  • variable-length component of a record $\mu_{\mathbb s}$

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting conventions, is there a style guide for that?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, just observing what's happening now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants