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

Grid Cross Plot: Crash when changing result property #12020

Closed
magnesj opened this issue Jan 3, 2025 · 0 comments · Fixed by #12022
Closed

Grid Cross Plot: Crash when changing result property #12020

magnesj opened this issue Jan 3, 2025 · 0 comments · Fixed by #12022
Assignees
@magnesj
Labels
BugInRelease Bug in an official release PendingRelease Issues is fixed and will be available in next release
Milestone
Maintenance 2025 ...

Comments

@magnesj
Copy link
Member

magnesj commented Jan 3, 2025

Create a grid cross plot
Select a different property for the x-axis -> crash
@magnesj magnesj added the BugInRelease Bug in an official release label Jan 3, 2025
@magnesj magnesj added this to the Maintenance 2025 - Sprint 1 milestone Jan 3, 2025
@magnesj magnesj self-assigned this Jan 3, 2025
magnesj added a commit that referenced this issue Jan 3, 2025
@magnesj
#12020 Delay the call to updateConnectedEditors
eead663 
Make sure the call to updateConnectedEditors happens after other slots have been processed. Also make sure that the call is done only once.
@magnesj magnesj mentioned this issue Jan 3, 2025
Merged
magnesj added a commit that referenced this issue Jan 3, 2025
@magnesj
#12020-grid-cross-plot: #12021 Wrong calculation of INDEX_IJK
7db1319 
INDEX_I INDEX_J INDEX_K are result properties computed by ResInsight. When using opm-common, only active cells are defined. In this situation, the computation fails.

Change the implementation to compute these values only for active cells. This will also improve performance and use less memory.
magnesj added a commit that referenced this issue Jan 3, 2025
@magnesj
#12020 Delay the call to updateConnectedEditors
756c1cb 
Move scheduler to AppFwk

Make sure the call to updateConnectedEditors happens after other slots have been processed. Also make sure that the call is done only once.

Add and use scheduler for updateConnectedEditors
magnesj added a commit that referenced this issue Jan 4, 2025
@magnesj
#12020 Delay the call to updateConnectedEditors
07d06c2 
Make sure the call to updateConnectedEditors happens after other slots have been processed. Also make sure that the call is done only once.
magnesj added a commit that referenced this issue Jan 4, 2025
@magnesj
#12020 Delay the call to updateConnectedEditors
93059a5 
Make sure the call to updateConnectedEditors happens after other slots have been processed. Also make sure that the call is done only once.
magnesj added a commit that referenced this issue Jan 6, 2025
@magnesj
#12020 Delay the call to updateConnectedEditors
1f7af45 
Make sure the call to updateConnectedEditors happens after other slots have been processed. Also make sure that the call is done only once.
@github-actions github-actions bot added the PendingRelease Issues is fixed and will be available in next release label Jan 6, 2025
magnesj added a commit that referenced this issue Jan 7, 2025
@magnesj
#12020 Delay the call to updateConnectedEditors
b9e348d 
Make sure the call to updateConnectedEditors happens after other slots have been processed. Also make sure that the call is done only once.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@magnesj magnesj

Labels
BugInRelease Bug in an official release PendingRelease Issues is fixed and will be available in next release
Projects
None yet
Milestone
Maintenance 2025 - Sprint 1
Development

Successfully merging a pull request may close this issue.

Fix crash in grid cross plot OPM/ResInsight
1 participant
@magnesj