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

Drag and drop same obs data on chart duplicates mapping #821

Closed
msevestre opened this issue Dec 6, 2022 · 4 comments · Fixed by #846
Closed

Drag and drop same obs data on chart duplicates mapping #821

msevestre opened this issue Dec 6, 2022 · 4 comments · Fixed by #846
Assignees

Comments

@msevestre
Copy link
Member

image
I believe this is NOT by design
Using project from #800

IMO, obs data should only be added once to the list

Thoughts? @rwmcintosh @Yuri05 @PavelBal @georgeDaskalakis ?

@Yuri05
Copy link
Member

Yuri05 commented Dec 6, 2022

obs data should only be added once to the list

Yes!

@PavelBal
Copy link
Member

PavelBal commented Dec 6, 2022

obs data should only be added once to the list

Hmm... In PI we are (finally!) allowed to map the same obs data to multiple simulations outputs. I would say it should also be possible in the simulations. However, multiple mapping of the same data to the same simulation output makes no sense. So:

  1. Mapping the same observed data to multiple outputs: YES
  2. Mapping multiple observed data to the same output: YES
  3. Duplicated mappings: NO.

@msevestre
Copy link
Member Author

Mapping the same observed data to multiple outputs: YES

Within ONE Simulation? Why does it make sense?

@PavelBal
Copy link
Member

PavelBal commented Dec 6, 2022

It makes sense in some cases but is not worth the effort... So we can agree that obs data should be added only once.

@Yuri05 Yuri05 moved this to In Progress in V11 - Update 1 Dec 27, 2022
@github-project-automation github-project-automation bot moved this from In Progress to Done in V11 - Update 1 Jan 6, 2023
@Yuri05 Yuri05 moved this from Done to Verified in V11 - Update 1 Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Verified
Development

Successfully merging a pull request may close this issue.

4 participants