Fix ruff #39
Fix ruff #39
Conversation
daklauss
force-pushed
the
fix-ruff
branch
3 times, most recently
from
47eb88e to
c43ad89
Compare
|
The great formatting has been done. Added preview rules to the linter/formater. Ignored Rules that caused warnings. added pre-commit-config.yaml. Also let ruff automaticaly by pre commit hook check and format linting stuff. |
| ], | ||
| [ | ||
| 0, 1, 0, 1, 4, 5, 6, 7, 8, 9, 10, |
There was a problem hiding this comment.
Not sure how to adress this one ....... :/
| 'parameters_start': { | ||
| 'inlet': { | ||
| 'c': np.array([0., 1.]), | ||
| "parameters_start": { |
There was a problem hiding this comment.
Do you prefer single or double quotes?
There was a problem hiding this comment.
I feel that double quotes are more standard these days...
| 0, 1, 0, 1, 4, 5, 6, 7, 8, 9, 10, | ||
| 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, | ||
| 22, 23, 24, 25, 26, 22, 23 | ||
| 0, |
There was a problem hiding this comment.
Like one line for every number ? ... mkay
There was a problem hiding this comment.
Add trailing comma to avoid line break. (See https://docs.astral.sh/ruff/settings/#format_skip-magic-trailing-comma)
There was a problem hiding this comment.
This wont work for this case, because the entire line is to long. Trailing comma does not prevent line break if line is to long i guess....
| state: str | ||
| ) -> np.ndarray: | ||
| self, origin_list: list[(dict, float)], state: str | ||
| ) -> np.ndarray: |
There was a problem hiding this comment.
Add trailing comma to avoid line break. (See https://docs.astral.sh/ruff/settings/#format_skip-magic-trailing-comma).
| def get_coupled_state(self, | ||
| origin_list: list[(dict, float)], | ||
| state: str | ||
| ) -> np.ndarray: |
There was a problem hiding this comment.
Add trailing comma to avoid line break. (See https://docs.astral.sh/ruff/settings/#format_skip-magic-trailing-comma)
Pull request to fix the linter problem as described in #36 . Added the preview rules that may be unstable but now, ruff linter checks also for things like linespace etc. Also trying to update Ruff pipeline.
Closes #36