-
-
Notifications
You must be signed in to change notification settings - Fork 3
merge_ui #17
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
Comments
NEST/teal.modules.general/issues/902 it can be quite confusing exactly what the analysis data set is:
Provenance:
|
@pawelru yes, but I'm thinking weather it should be a in:
Provenance:
|
For me (left) data extract panel is only about input specification. It's already too much there so I wouldn't put a new item in there. But let's discuss it. Moreover, there are modules that has more than one analysis dataset (thus: how to name them? would it be clear for the end-user?). There are modules that has no analysis dataset per se (see missing data module). This should be also taken into account. Provenance:
|
I see both sides of the discussion here. On one hand, it's confusing, on the other hand, we already provide the code for the joins, so it should be clear what teal does. Provenance:
|
We use to call
data_merge_module
in the server but it doesn't return any information to ui. We might need to present ANL details in the encoding panel.The reason for this is that
left_join
,full_join
causes some confusion. Reportedly it's not clear what is the output of used datasets - even if we havedatasets: ADSL, ADLB, ...
in the encoding panel and datasets sizes in the flter panel - it's still vague what is the result of these joins.merge_ui
(or other name) can be put (optionaly) in the ui and can receive some information from the module.Provenance:
The text was updated successfully, but these errors were encountered: