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

Reload not working #23

Closed
ntoles opened this issue Aug 12, 2024 · 5 comments · Fixed by #20
Closed

Reload not working #23

ntoles opened this issue Aug 12, 2024 · 5 comments · Fixed by #20

Comments

@ntoles
Copy link
Collaborator

ntoles commented Aug 12, 2024

Pressing Reload button triggers this error and program is unusable. It should check for the largest file in the directory.

Exception in Tkinter callback
Traceback (most recent call last):
File "/usr/licensed/anaconda3/2023.9/lib/python3.11/tkinter/init.py", line 1948, in call
return self.func(*args)
^^^^^^^^^^^^^^^^
File "/home/anatoly/tig/Bob-iseult/src/main_app.py", line 434, in OnReload
self.parent.ReloadPath()
File "/home/anatoly/tig/Bob-iseult/src/main_app.py", line 2180, in ReloadPath
self.TimeStep.setMax(len(self.PathDict['Flds']))
File "/home/anatoly/tig/Bob-iseult/src/main_app.py", line 342, in setMax
feedbackKnob.setKnob(self.value)
File "/home/anatoly/tig/Bob-iseult/src/main_app.py", line 3540, in setKnob
self.RenewCanvas()
File "/home/anatoly/tig/Bob-iseult/src/main_app.py", line 3003, in RenewCanvas
self.RefreshCanvas(keep_view = keep_view)
File "/home/anatoly/tig/Bob-iseult/src/main_app.py", line 3233, in RefreshCanvas
self.LoadAllKeys()
File "/home/anatoly/tig/Bob-iseult/src/main_app.py", line 2476, in LoadAllKeys
filepath = os.path.join(self.dirname,self.PathDict['Param'][self.TimeStep.value-1])
~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
IndexError: list index out of range

image
@bcaddy
Copy link
Collaborator

bcaddy commented Aug 12, 2024

Thanks, is this on master or one of the open PRs?

@ntoles
Copy link
Collaborator Author

ntoles commented Aug 12, 2024 via email

@bcaddy
Copy link
Collaborator

bcaddy commented Aug 12, 2024

Thanks, I'm not sure what the issue is yet but I can reproduce it so I'll work on figuring it out.

@bcaddy
Copy link
Collaborator

bcaddy commented Aug 13, 2024

I found the issue. It will be a little time consuming to fix but shouldn't be too hard. Will update when I have a fix

@bcaddy bcaddy linked a pull request Aug 14, 2024 that will close this issue
@bcaddy
Copy link
Collaborator

bcaddy commented Aug 14, 2024

This should be fixed in the latest version of PR #20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants