-
Notifications
You must be signed in to change notification settings - Fork 264
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
Fix QoS deprecation warnings #879
Fix QoS deprecation warnings #879
Conversation
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 contribution!
Would it be possible to create a new variable with QoS setup before initializing all the services, and then reuse it? This should make the change a bit cleaner.
When you're setting up that variable can you copy all full setup (if different from default from the lines 39-48)
Thanks for the feedback @destogl. Attempted to address your comments. Let me know if I misunderstood. |
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.
With the rename I am happy.
* Fix floating point comparison in JTC * Fix format --------- Co-authored-by: Christoph Fröhlich <christophfroehlich@users.noreply.github.com>
First PR to fix #862
I'm not familiar with QoS so was a little unsure if
KeepAll()
is the correct fix here.