-
Notifications
You must be signed in to change notification settings - Fork 153
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
Document mathematical notation #811
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #811 +/- ##
=====================================
Coverage 95.6% 95.6%
=====================================
Files 46 46
Lines 4344 4344
=====================================
Hits 4154 4154
Misses 190 190 |
Thanks @Jihoon for the close read! These are all important points to consider.
These are both errors I think. For (2) I guess it should be y^A on the summation mark for consistency. I'm not sure about (1). Yes. This is supposed to reflect that some parameters have dimension n^D, others n^O, others n. If one is e.g. "R12_AFR", then the others are the same. This is for instance not the case for n^L: n^L can have one value while n^D, n^O, and n have a different one (the same value for all three). IMHO this is a tricky concept per se, and hard to convey it concisely and clearly through notation.
In LaTeX it is possible to write I would be fine for describing an order (e.g. () innermost, followed by [], then alternate…or something like that) but it would be a bit harder to apply consistently. Will consider this further.
Yes, these are already given on the page for Sets and mappings. I think this could be improved, but didn't want to get into that yet.
This would produce more line wraps and possibly longer lines that overflow.
My one worry with this is that it may reverse the way some equations are named and actually implemented in the code. For example, if a equation is called BLAH_GT or BLAH_LT but then we end up with respectively the symbol ≤ ("less than or equal") or ≥ ("greater than or equal"), the mismatch is potentially confusing. But I will try and see how it looks.
It should be defined at (6), maybe isn't. |
- Express/document different alignment of duration_time_rel. - Correct order for "Y^A in Y". - Reflow some text.
Update TODOs
f5f7ff6
to
d7d963f
Compare
Following on the 2024-03-28 MESSAGE meeting (#388 (comment)), this PR proposes a style of mathematical notation to be used in the docs.
This style is:
The previous/extant style was not documented fully or in any user-visible location.
This PR is not intended to include revision of all equations to match the style, only to establish it. That revision would be accomplished in 1 or more follow-up PRs by a team of contributors.
How to review
PR checklist
In particular, we could define LaTeX commands like
\output
to expand to\text{output}_{c h h^D l m n^D n^L t y^A y^V}
, since these are almost always written the same way. This would make the LaTeX source of the equations much easier to read and edit.Add or expand tests;coverage checks both ✅