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

Add keep/drop to coefplot and refactor code to improve maintainability #341

Merged
merged 8 commits into from
Mar 4, 2024
Merged

Add keep/drop to coefplot and refactor code to improve maintainability #341

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
17e86c0
dev: add keep/drop/exact_match to coefplot/iplot
Wenzhi-Ding Mar 4, 2024
df6c2e2
dev: remove backward compatibility.
Wenzhi-Ding Mar 4, 2024
7f058fb
dev: remove backward compatibility.
Wenzhi-Ding Mar 4, 2024
6240db3
fix test bug.
Wenzhi-Ding Mar 4, 2024
6a21645
fix class referenced functions
Wenzhi-Ding Mar 4, 2024
cb9263f
lint
Wenzhi-Ding Mar 4, 2024
a50cd8c
dev: make functions and docstrings easier maintenance.
Wenzhi-Ding Mar 4, 2024
823ac89
fix: ignore ruff D205 linting rule
Wenzhi-Ding Mar 4, 2024
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
1 change: 1 addition & 0 deletions pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,7 @@ ignore = [
"D104", # missing docstring in public package
"SIM110", # Use all instead of `for` loop
"TRY003", # Avoid specifying long messages outside the exception class
"D205", # 1 blank line required between summary line and description
Copy link
Member

Choose a reason for hiding this comment

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

Yes, I also did not love that one ... 😄

]

[tool.ruff.lint.per-file-ignores]
Expand Down
Loading