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.

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

TypeError: QtSidebar.update() missing 1 required positional argument: 'w' #15

Closed
mschutter05 opened this issue Mar 10, 2024 · 1 comment
Assignees
Labels
bug Something isn't working from app Opened via in-app "Report Bug" button good first issue Good for newcomers

Comments

@mschutter05
Copy link

In line 182 of interface.py there is the following call to update with only 3 arguments:
self.sidebar.update(0, False, -1)

This was resulting in the following every time I tried to open a .bin file:
TypeError: QtSidebar.update() missing 1 required positional argument: 'w'

After changing line 182 to the following by adding None argument I was able to open the file without getting TypeError:
self.sidebar.update(0, False, -1, None)

I don't know what I did or if it is a proper fix.

@mschutter05 mschutter05 added bug Something isn't working from app Opened via in-app "Report Bug" button labels Mar 10, 2024
sam210723 added a commit that referenced this issue Mar 10, 2024
@sam210723
Copy link
Owner

sam210723 commented Mar 10, 2024

Looks like I missed this because my testing uses the -i argument to open the waveform files rather than File -> Open. You were pretty much correct with your fix, but I've used 0 for that argument so the channel selector drop down in the sidebar is cleared correctly.

I've released v2.3.1 with this fix.

@sam210723 sam210723 added the good first issue Good for newcomers label Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working from app Opened via in-app "Report Bug" button good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants