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

Fix GUI not updating bugs #1180

Merged
merged 3 commits into from
Jan 29, 2024
Merged

Fix GUI not updating bugs #1180

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e995951
Fix GUI not updating bugs
DiegoTavares Jul 28, 2022
de32c3f
Apply suggestions from code review
DiegoTavares Jan 29, 2024
1b3024d
Update MonitorJobsPlugin.py
DiegoTavares Jan 29, 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
2 changes: 1 addition & 1 deletion cuegui/cuegui/JobMonitorTree.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -490,7 +490,7 @@ def _getUpdate(self):
# an empty list for the id argument!
if not ids:
continue
tmp = opencue.api.getJobs(id=ids, all=True)
tmp = opencue.api.getJobs(id=ids, include_finished=True)
self.__dependentJobs[job] = tmp

if self.__loadMine:
Expand Down
27 changes: 14 additions & 13 deletions cuegui/cuegui/plugins/MonitorJobsPlugin.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -199,19 +199,20 @@ def _regexLoadJobsHandle(self):

self.jobMonitor.removeAllItems()

if cuegui.Utils.isStringId(substring):
# If a uuid is provided, load it
self.jobMonitor.addJob(substring)
elif load_finished_jobs or re.search(
r"^([a-z0-9_]+)\-([a-z0-9\.]+)\-", substring, re.IGNORECASE):
# If show and shot is provided, or if "load finished" checkbox is checked, load all jobs
for job in opencue.api.getJobs(regex=[substring], include_finished=True):
self.jobMonitor.addJob(job)
else:
# Otherwise, just load current matching jobs (except for the empty string)
if substring:
for job in opencue.api.getJobs(regex=[substring]):
self.jobMonitor.addJob(job)
if len(substring) > 0:
DiegoTavares marked this conversation as resolved.
Show resolved Hide resolved
if cuegui.Utils.isStringId(substring):
# If a uuid is provided, load it
self.jobMonitor.addJob(substring)
elif load_finished_jobs or re.search(
r"^([a-z0-9_]+)\-([a-z0-9\.]+)\-", substring, re.IGNORECASE):
# If show and shot is provided, or if "load finished" checkbox is checked, load all jobs
for job in opencue.api.getJobs(regex=[substring], include_finished=True):
self.jobMonitor.addJob(job)
else:
# Otherwise, just load current matching jobs (except for the empty string)
if substring:
DiegoTavares marked this conversation as resolved.
Show resolved Hide resolved
for job in opencue.api.getJobs(regex=[substring]):
self.jobMonitor.addJob(job)
DiegoTavares marked this conversation as resolved.
Show resolved Hide resolved

def getGrpDependent(self):
"""Is group dependent checked"""
Expand Down
1 change: 1 addition & 0 deletions pycue/opencue/api.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -325,6 +325,7 @@ def getJobs(**options):
- show: show names - list
- shot: shot names - list
- user: user names - list
- include_finished - bool

:rtype: list
:return: a list of Job objects
Expand Down