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

Jump to bottom

[cuegui] fix cue monitor gpu min/max #1144

Merged
merged 1 commit into from
May 10, 2022
Merged

[cuegui] fix cue monitor gpu min/max #1144

merged 1 commit into from
May 10, 2022

Conversation

aoblet
Copy link
Contributor

@aoblet aoblet commented May 3, 2022

Signed-off-by: Oblet Alexis [email protected]

Summarize your change.

  • Fix regression on cue monitor view
Traceback (most recent call last):
  File "/tmp/OpenCue/cuegui/cuegui/CueJobMonitorTree.py", line 562, in contextMenuEvent
    self.__menuActions.jobs().addAction(menu, "setMinGpu")
  File "/tmp/OpenCue/cuegui/cuegui/MenuActions.py", line 139, in addAction
    info = getattr(self, "%s_info" % actionName)
AttributeError: 'JobActions' object has no attribute 'setMinGpu_info'

@aoblet
Copy link
Contributor Author

aoblet commented May 5, 2022

Can we merge ?

@bcipriano bcipriano merged commit ab727e5 into AcademySoftwareFoundation:master May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DiegoTavares DiegoTavares DiegoTavares approved these changes

@bcipriano bcipriano bcipriano approved these changes

@gregdenton gregdenton Awaiting requested review from gregdenton

@jrray jrray Awaiting requested review from jrray

@smith1511 smith1511 Awaiting requested review from smith1511

@larsbijl larsbijl Awaiting requested review from larsbijl

@IdrisMiles IdrisMiles Awaiting requested review from IdrisMiles

@splhack splhack Awaiting requested review from splhack

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aoblet @DiegoTavares @bcipriano