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

Bug importing observation data #10536

Closed
alifbe opened this issue Aug 22, 2023 · 1 comment · Fixed by #10733
Closed

Bug importing observation data #10536

alifbe opened this issue Aug 22, 2023 · 1 comment · Fixed by #10733
Assignees
Labels
Bug Bug not yet in an official release

Comments

@alifbe
Copy link

alifbe commented Aug 22, 2023

ResInsight imported the wrong value when I tried to import observation data (attached drogon_obs.csv). The ERROR column is imported twice and is used as value. In this file, I have both WWCT and WGOR. But WWCT is imported correctly.

It seems like the bug started from version 2023.03.0. I tried 2023.01.0 and it works fine.

image

@magnesj magnesj added this to the Maintenance 2023 - Sprint 8 milestone Aug 23, 2023
@magnesj magnesj added the Bug Bug not yet in an official release label Sep 25, 2023
kriben added a commit that referenced this issue Oct 18, 2023
…ting

Observed summary data with error bar results have two addresses: one for the
data, and one for the error data. When creating the RimSummaryAddress objects
in the "Data Sources" tree the error data address would occasionally be used
instead of the data address (due to lack of sorting and filtering).

Plots created from the error RimSummaryAddress would display the error data twice
in the "Show Plot Data" view.

Fixed by filtering out the error addresses and improving the sorting criteria.

Fixes #10536.
kriben added a commit that referenced this issue Oct 18, 2023
…ting

Observed summary data with error bar results have two addresses: one for the
data, and one for the error data. When creating the RimSummaryAddress objects
in the "Data Sources" tree the error data address would occasionally be used
instead of the data address (due to lack of sorting and filtering).

Plots created from the error RimSummaryAddress would display the error data twice
in the "Show Plot Data" view.

Fixed by filtering out the error addresses and improving the sorting criteria.

Fixes #10536.
magnesj pushed a commit that referenced this issue Oct 19, 2023
…ting

Observed summary data with error bar results have two addresses: one for the
data, and one for the error data. When creating the RimSummaryAddress objects
in the "Data Sources" tree the error data address would occasionally be used
instead of the data address (due to lack of sorting and filtering).

Plots created from the error RimSummaryAddress would display the error data twice
in the "Show Plot Data" view.

Fixed by filtering out the error addresses and improving the sorting criteria.

Fixes #10536.
@github-actions github-actions bot added the PendingRelease Issues is fixed and will be available in next release label Oct 19, 2023
@kriben
Copy link
Collaborator

kriben commented Oct 20, 2023

@alifbe Thanks for the good bug report. Fix will be release in next release (2023.10) which should come next week.

@magnesj magnesj removed the PendingRelease Issues is fixed and will be available in next release label Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bug not yet in an official release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants