-
-
Notifications
You must be signed in to change notification settings - Fork 546
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
[perfect-numbers] backticks on math and removed parens #2319
Merged
Merged
Changes from 3 commits
Commits
Show all changes
12 commits
Select commit
Hold shift + click to select a range
521f195
backticks on math and removed parens
habere-et-dispertire 2fe0056
markup "number", use has and English
habere-et-dispertire dee88ae
reformat to: when a number ... its aliquot sum
habere-et-dispertire 6c86cdb
backtick aliquot sum
habere-et-dispertire 544df6a
Another aliquot sum backticked
habere-et-dispertire dd192a6
Split into three headings
habere-et-dispertire 9a4254a
lint
habere-et-dispertire 47c7954
Link text references to headers
habere-et-dispertire ca2cec0
lint
habere-et-dispertire 486785f
Two => For
habere-et-dispertire 8230e0b
More intra-doc links
habere-et-dispertire 44f0c7b
examples => example
habere-et-dispertire File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should
number
andaliquot sum
be quoted rather than code blocked?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will take that as a no, then. ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's perhaps easier to read wth the code blocks marking them as variables which also draws us to their earlier mention -- but I don't mind. It currently looks like
The
aliquot sum
is defined as the sum of the factors of anumber
not including thenumber
itself.For example, the
aliquot sum
of15
is1 + 3 + 5 = 9
.number
equals itsaliquot sum
number
is less than itsaliquot sum
number
is greater than itsaliquot sum
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought we were describing the idea, rather than actual code. It seems that
aloquot sum
may or may not be an invocation of a function or method in some languages.I definitely will not stand in the way of it though.
It could look like:
The "aliquot sum" is defined as the sum of factors of a
number
not including thenumber
itself.For example, the "aliquot sum" of
15
is1 + 3 + 5
or9
.number
equals its "aliquot sum"number
less than its "aliquot sum"number
greater than its "aliquot sum"There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The aliquot sum is defined as the sum of factors of a
number
not including thenumber
itself. For example, the aliquot sum of15
is1 + 3 + 5
or9
.number
equals its aliquot sumnumber
less than its aliquot sumnumber
greater than its aliquot sum—-
Italics to me emphasize the technical term aliquot sum but also distinct from other terms that are bolded, a different category of emphasis. Quote marks to me don’t convey that much emphasis compared to italics.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would also prefer bolding it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reason I kept the emphasis on each occurrence was because perfect, abundant, and deficient are also bolded in later occurrences. It’d make sense though to only emphasize the first occurrence so maybe that needs to fixed as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've turned them into links referencing the relevant heading.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everyone happy with this ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wasn't terrible to start out with, but still looking much better now!