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.

Sign up for GitHub

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

Jump to bottom

add ruff and remove redundant pre-commits #278

Merged
merged 18 commits into from
Feb 16, 2023
Merged

add ruff and remove redundant pre-commits #278

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
183716a
fix D101 and B028 (stack level for warnings) introduced in #219
giovp Feb 14, 2023
7946c2a
add vscode to gitignore
giovp Feb 14, 2023
aa45bde
fix D102 on costs.py and try to simplify inheritance methods
giovp Feb 14, 2023
0880160
Merge branch 'main' into pre-commit/flake8
giovp Feb 14, 2023
987b31b
try to fix D102 everywhere
giovp Feb 14, 2023
7d2e5f8
fix D103
giovp Feb 14, 2023
f16a08d
remove comment from .flake8
giovp Feb 14, 2023
6dc02fb
add ruff to pyproject.toml
giovp Feb 14, 2023
503b284
add ruff
giovp Feb 14, 2023
cca9a87
add documentation on ruff select and add unfixable
giovp Feb 14, 2023
6aff3a6
add target version
giovp Feb 14, 2023
ca9baa0
add comment on tool.ruff
giovp Feb 14, 2023
001e0b3
pyproject.toml
giovp Feb 14, 2023
243bfbf
format
giovp Feb 14, 2023
3ddfb1c
merge
giovp Feb 15, 2023
eac98d1
remove artifcats from wrong merge
giovp Feb 16, 2023
9b1aa6e
address ruff comments
giovp Feb 16, 2023
004872c
fix ruff imports
giovp Feb 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into pre-commit/flake8
  • Loading branch information
@giovp
giovp authored Feb 14, 2023
commit 08801602257c3010d5479634b0e8275e070f6a4d
3 changes: 2 additions & 1 deletion src/ott/solvers/nn/neuraldual.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,8 @@ def setup(
else:
if self.parallel_updates:
warnings.warn(
'parallel_updates set to True but disabling it because num_inner_iters>1',
'parallel_updates set to True but disabling it '
'because num_inner_iters>1',
stacklevel=2
)
if self.back_and_forth:
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.