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

Families, inheritance and GUI interaction #3335

Closed
TomekTrzeciak opened this issue Aug 29, 2019 · 3 comments
Closed

Families, inheritance and GUI interaction #3335

TomekTrzeciak opened this issue Aug 29, 2019 · 3 comments
Labels
duplicate This is a duplicate of something else

Comments

@TomekTrzeciak
Copy link
Contributor

Describe exactly what you would like to see in an upcoming release

Task families play multiple roles in Cylc. They are used for (multiple) inheritance of settings, but also for task grouping in GUIs, graphs, queue and clock-trigger assignments, cylc commands, etc. Using the leading family inheritance for task grouping in GUIs is problematic, though. Consider the following example:

[runtime]
  [[FOO]]
  [[BAR]]
    inherit = FOO
  [[BAZ]]
    inherit = None, FOO

FOO will show up in GUI as a level of task grouping with BAR underneath, but selecting this group and, say, succeeding it will also succeed BAZ task due to inheritance, even though it was not part of the selection.

If leading (first) and secondary families formed disjoint sets, however, the above situation would be impossible. My suggestion would be to check for this condition during suite validation and issue a warning in Cylc 7.8.x and treat it as an error in Cylc 8+.

Additional context

Mentioned in #2701 (comment)

@sadielbartholomew
Copy link
Collaborator

I think this is discussed at least to some extent in #2570.

@hjoliver
Copy link
Member

hjoliver commented Aug 29, 2019

Good spotting @sadielbartholomew. This is technically a duplicate of that issue but I'll close the other one as superseded, as this expresses the problem more clearly [actually the other one is pretty clear, so let's move the discussion there @TomekTrzeciak].

@hjoliver
Copy link
Member

Closing as duplicate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This is a duplicate of something else
Projects
None yet
Development

No branches or pull requests

4 participants