-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[MESH] fix streamlines and traces renderer when vector is null #36633
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.
it fixes which ticket? i think there is missing issue # in the description
@@ -550,6 +550,9 @@ void QgsMeshLayerRenderer::renderVectorDataset() | |||
if ( std::isnan( mVectorDatasetMagMinimum ) || std::isnan( mVectorDatasetMagMaximum ) ) | |||
return; // only NODATA values | |||
|
|||
if ( !( mVectorDatasetMagMaximum > 0 ) ) |
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.
mVectorDatasetMagMaximum <= 0
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.
Indeed, I hesitated, but I found more explicit >0
@@ -429,6 +430,9 @@ void QgsMeshStreamField::addTrace( QPoint startPixel ) | |||
if ( !mVectorValueInterpolator ) | |||
return; | |||
|
|||
if ( !( mMaximumMagnitude > 0 ) ) |
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.
mVectorDatasetMagMaximum <= 0
I found this issue without ticket during temporal tries. Maybe fix #35888, but very not sure. |
related #35888 |
Merged the ticket, I have |
In mesh rendering, the streamlines and the trace renderers didn't stop when vector magnitude was null.
This PR fix this issue