-
Notifications
You must be signed in to change notification settings - Fork 48
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
Change hypotheses boxes to QListWidget
to reduce space
#363
Conversation
Preview page for your plugin is ready here: |
Codecov ReportPatch coverage:
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## main #363 +/- ##
==========================================
- Coverage 86.10% 85.77% -0.34%
==========================================
Files 30 30
Lines 2109 2151 +42
Branches 392 394 +2
==========================================
+ Hits 1816 1845 +29
- Misses 213 225 +12
- Partials 80 81 +1
☔ View full report in Codecov by Sentry. |
@@ -353,7 +353,7 @@ class Tracklet: | |||
x values. | |||
""" | |||
|
|||
def __init__( | |||
def __init__( # noqa: PLR0913 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just wondering what these new ruff rules are?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one is complaining about too many arguments. I updated ruff
a bit but not all the way as there was more to fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
The current
QCheckBox
takes up a lot of space.Before
Looks like
Problem if you click
config name
fromcell
→particle
→cell
removes all ticksP_FP
should be enabled always. Not sure if this setting exists