You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The limitation here is that component_factory needs to include logic for importing the right class needed to instantiate runtime objects for the component.
This could simplify the process of supporting new component types - i.e. component_factory will automatically attempt to load the class using the args provided.
It would perhaps be useful to expand the spec such that information about how to load the component is self contained in the spec.
Some components needs custom logic to map from declarative spec to runtime instances e.g.,
parsing the representation of combinations of termination conditions
adding default behaviours e.g., default system messages if none provided etc
In these cases, the benefit of an expanded spec is reduced (adds bloat to the spec) and perhaps can only be used for some validation (e.g., verifying that the intended class exists etc).
Discussions welcome.
The text was updated successfully, but these errors were encountered:
AGS component spec currently has a broad component_type attribute that is used to determine the class needed to load the spec.
The limitation here is that component_factory needs to include logic for importing the right class needed to instantiate runtime objects for the component.
This could simplify the process of supporting new component types - i.e. component_factory will automatically attempt to load the class using the args provided.
It would perhaps be useful to expand the spec such that information about how to load the component is self contained in the spec.
Design Issues
Some components needs custom logic to map from declarative spec to runtime instances e.g.,
In these cases, the benefit of an expanded spec is reduced (adds bloat to the spec) and perhaps can only be used for some validation (e.g., verifying that the intended class exists etc).
Discussions welcome.
The text was updated successfully, but these errors were encountered: