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

In step-77, also talk about the other nonlinear solver packages we support. #15128

Merged
merged 1 commit into from May 8, 2023

Conversation

bangerth
Copy link
Member

In relation to #15065 and #14332.

identical interface. If you have installed a version of deal.II that
is configured to use either PETSc or Trilinos, but not SUNDIALS,
then it is not too difficult to switch this program to use either
of the former two packages instead: Basically everything that we
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
of the former two packages instead: Basically everything that we
of the former two packages instead: basically everything that we

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't think this is actually incorrect: The sentence that follows the colon is a complete sentence, and can consequently be capitalized. (See what I'm doing here? ;-)

I agree that the first word should not be capitalized if what follows is not a complete sentence: a list, or half sentence, for example.

Copy link
Contributor

Choose a reason for hiding this comment

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

american vs british english (and the rest of the other languages based on the alphabet ...)

@bangerth
Copy link
Member Author

bangerth commented May 3, 2023

Ping?

@peterrum peterrum merged commit 937f79b into dealii:master May 8, 2023
11 of 14 checks passed
@bangerth bangerth deleted the 77-2 branch May 8, 2023 19:08
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.

None yet

4 participants