-
Notifications
You must be signed in to change notification settings - Fork 0
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
solve #1777 #3
solve #1777 #3
Conversation
The issue was that you could select quantMode != '-' even if you had no gtf which makes an error which is not meaningful. |
@bernt-matthias, could you review this? |
Can try :) Could need a bit, sice I give a course these days. Please ping me if I do not respond by next week. |
@bernt-matthias If I can take your time to review this PR... |
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.
Thanks for the reminder. Looks good to me. Just a few small things (I think).
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.
Good to go from my side.
clean xml files + add macros + tools name underscore
This is quite a big change as the
quantMode
is now inside theGTFconditional
.