-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
35 improve doc strings #48
base: main
Are you sure you want to change the base?
Conversation
* concatenate doc strings with trailing name * insert blank lines to improve readability
Fit text within reasonable screen width.
-delete trailing spaces -reformat ouput of Base.print for {T<:Unit} -correct mass unit string in printunits() -reformat the code for initializing the different systems of units
I am temporarily exportting those elements so as to more easily debug their docstrings.
Following our discussion about moving to Unitful, I am suspending this effort for now.
This should allow the expression :(eV/c^2) to parse correctly.
I amended the Unit dictionary so that the tests now all pass. |
@@ -1,47 +1,57 @@ | |||
|
|||
# include("PhysicalConstants.jl") |
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.
Includes should be suppressed in the release version, as repeated includes cause errors and everything is included directly in AtomicAndPhysicalConstants.jl
In addition to clarifying many of the docstrings—especially the direction in
which the conversion factor operates—I have extended the dictionary of SI
prefixes, fixed a bug (typo) in the nano prefix, improved the output formatting
of printunits(), and reformatted some of the code.
I think we can close this issue for now. Further work on the docstrings can
happen as we transition to the use of
Unitful
.