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.

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

Update dev branch after patch release #11365

Merged
merged 19 commits into from
Apr 19, 2024
Merged

Update dev branch after patch release #11365

merged 19 commits into from
Apr 19, 2024

Conversation

magnesj
Copy link
Member

@magnesj magnesj commented Apr 15, 2024

Closes #11352
Closes #11353
Closes #11355
Closes #11337
Closes #11359
Closes #11370

magnesj and others added 19 commits April 15, 2024 10:33
Always allocate result data for active cells in destination case. Make sure that active cells is used for both population of expression variables and filtering of data. Improved naming to make it clear that we always work with active cell data.
addStaticScalarResult will clear already computed data, causing calculations to be triggered once more
If not checking if data is available, data is read from file and appended to result vector.
…tersection

Size of the overlay is could become negative, and would overflow for small views.

Fixes #11334.
Well pipe radius would become HUGE_VAL due to off-by-one error.
The assert could only happen on model with a single cell.

Fixes #11310.
Equal grid size is required if there is more than one grid case in the expression. If a cell filter view is active, the visibility is based on one view and reused for all other grid models, and requires equal grid size.
* Make sure we operate in the correct domain when picking points in the contour map
@magnesj magnesj marked this pull request as ready for review April 19, 2024 10:44
@magnesj magnesj merged commit e9ff354 into dev Apr 19, 2024
21 checks passed
@magnesj magnesj deleted the merge-from-main branch April 19, 2024 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment