-
-
Notifications
You must be signed in to change notification settings - Fork 269
Fix QuakeMDL animation #2128
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 QuakeMDL animation #2128
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release #2128 +/- ##
===========================================
+ Coverage 95.67% 95.86% +0.18%
===========================================
Files 125 127 +2
Lines 9947 11057 +1110
===========================================
+ Hits 9517 10600 +1083
- Misses 430 457 +27 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
9092324
to
4386808
Compare
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.
Some suggestions could not be made:
- plugins/native/module/vtkF3DQuakeMDLImporter.cxx
- lines 118-118
- lines 213-216
- lines 232-234
- lines 417-417
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.
Some suggestions could not be made:
- plugins/native/module/vtkF3DQuakeMDLImporter.cxx
- lines 123-124
- lines 401-402
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.
Some suggestions could not be made:
- plugins/native/module/vtkF3DQuakeMDLImporter.cxx
- lines 123-124
- lines 399-400
63ce603
to
9f320ea
Compare
TODO (not in this PR nor this release):