Skip to content

minor improvements(?)#135

Merged
gares merged 1 commit into
math-comp:masterfrom
darijgr:sugg1
Jul 13, 2021
Merged

minor improvements(?)#135
gares merged 1 commit into
math-comp:masterfrom
darijgr:sugg1

Conversation

@darijgr
Copy link
Copy Markdown
Contributor

@darijgr darijgr commented Jul 12, 2021

  • The \C{Compute} command is not exactly the \c{compute} tactic, right? Better to say either \C{Compute} or \C{vm_compute}.

  • "Expected value" sounds too much like the probability-theoretic concept of the same name. I suggest "explicit result" instead.

  • Functions with several arguments are "not represented" -> "not usually represented" with a tuple of arguments, since you can (if you really want) use Cartesian product types.

  • I'm not sure what "it appends that" means in French, but it's not idiomatic in English.

  • Added \hypersetup to main.tex so that the title and author metadata in the resulting PDF are meaningful.

- The \C{Compute} command is not exactly the \c{compute} tactic, right? Better to say either \C{Compute} or \C{vm_compute}.

- "Expected value" sounds too much like the probability-theoretic concept of the same name. I suggest "explicit result" instead.

- Functions with several arguments are "not represented" -> "not usually represented" with a tuple of arguments, since you can (if you really want) use Cartesian product types.

- I'm not sure what "it appends that" means in French, but it's not idiomatic in English.

- Added \hypersetup to main.tex so that the title and author metadata in the resulting PDF are meaningful.
@gares
Copy link
Copy Markdown
Member

gares commented Jul 13, 2021

Wow, thanks!

@gares gares merged commit c5fc23f into math-comp:master Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants