lava
lava copied to clipboard
lava-nc
AbstractSubProcessModel should be renamed
Describe the bug AbstractSubProcessModel is inappropriately named. The thing which it describes is a container of other processes/process models, but the name suggests that it's one of the elements held by such a container.
Expected behavior AbstractSubProcessModel should be renamed to something that conveys its correct usage, such as:
- AbstractCompositeModel
- AbstractContainerModel
- AbstractSuperModel
Environment (please complete the following information):
- Lava version 0.7
Additional context This is a end-user facing class, so this would be an API-breaking change. Recommend deprecating the current class and references to it, then adding a new class with a more appropriate name.
