Skip to content

fix typo Linael -> Linear - #1201

Merged
TobiKattmann merged 1 commit into
developfrom
fix_typoLinael
Feb 18, 2021
Merged

fix typo Linael -> Linear#1201
TobiKattmann merged 1 commit into
developfrom
fix_typoLinael

Conversation

@TobiKattmann

Copy link
Copy Markdown
Contributor

Proposed Changes

Description of LINEAR_SOLVER_ILU_FILL_IN wrote Linael instead of Linear .. and that was copied quite a few times
Done using find ./ -type -f -exec sed -i -e 's/Linael/Linear/g' {} \; in ./Testcases on Linux

As in none of the appearances the option is not using the default of zero, that option could be removed from all those testcases imo, but I put that up to discussion. As @pcarruscag mentioned in #1195 [...]simplify the config by removing options that are not needed, this is to make it easier to maintain the testcases if there are changes to some options.

Feel free to chip any other typos here.

Related Work

Seen in #1195 and annoyed me enough to open a PR, but of course not introduced there

  • I am submitting my contribution to the develop branch.
  • My contribution generates no new compiler warnings (try with the '-Wall -Wextra -Wno-unused-parameter -Wno-empty-body' compiler flags, or simply --warnlevel=2 when using meson).
  • My contribution is commented and consistent with SU2 style.
  • I have added a test case that demonstrates my contribution, if necessary.
  • I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp) , if necessary.

Comment thread config_template.cfg
Comment on lines 1022 to 1023
%
% Linael solver ILU preconditioner fill-in level (0 by default)
% Linear solver ILU preconditioner fill-in level (0 by default)
LINEAR_SOLVER_ILU_FILL_IN= 0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oh no... even the template xD

@TobiKattmann
TobiKattmann merged commit 88fbcd6 into develop Feb 18, 2021
@TobiKattmann
TobiKattmann deleted the fix_typoLinael branch February 18, 2021 22:05
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.

3 participants