Bundling Actions Together
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Comienza con AppBuilder y GraphBuilder y, después, sigue cómo se representan actions, transitions, subgraphs y MappedStates. Compara los puntos de entrada MultiAction y ActionChain propuestos con las API de builder existentes, incluida la alternativa with_subgraph. Se considera terminado cuando el proyecto tenga un diseño de action-graph reutilizable y acordado, incluido cualquier remapeo de entrada/salida y el comportamiento de validación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
@gamarin2 asks:
Hey! While working with class-based actions, I ended up creating state transitions that must logically be split in two separate actions, but where the second one must always be preceded by the first one.
In my case, it's a FormatSummarizer action that formats messages before transitioning to a Summarizer action. These two actions form what I would call an "action bundle": the second one can't be separated from the first one. For now, I have documented that requirement in my class docstring, but it would be nice to have it more strictly enforced.
I'm sure there are various ways we could achieve this, but perhaps the easiest would be to add a property to the action class where you could specify preceding/following action classes. The AppBuilder would then fail to build if you haven't added the proper transitions.
@mdrideout also suggests:
I would say that I have a very large number of use cases for this, and creating an entire sub-graph for it is painful.
To piggyback on this, I also have a very large number of use cases for running actions with high concurrency, and have been creating sub-graphs simply for the purpose of running a single action for ~10 items.
but I may be doing this as a workaround because I had a some issues trying to use MappedStates (?) with either pydantic or async.
Describe the solution you'd like
A way to have an action represent a graph.
class MultiAction:
@abc.abstractmethod
def get_graph() -> Graph:
...
class ActionChain(MultiAction):
@abc.abstractmethod
def get_actions() -> List[[Tuple[str, Union[Action, Callable]]]:
...
def get_graph() -> Graph:
named_actions = self.get_actions()
action_names = [name for name, _ in named_actions]
return GraphBuilder().with_actions(**dict(actions)).with_transitions(zip(action_name, action_names[1:]))
Describe alternatives you've considered
Could also have with_subgraph in builder, similar thing. This allows for packaging at the action level for reusability. Might also want the ability to remap inputs/outputs:
def state_maps() -> Tuple[dict, dict]:
....
Additional context
Add any other context or screenshots about the feature request here.
- Lenguaje dominante
- Python
- Estrellas
- 2.6k
- Forks
- 203
- Merge medio
- 3 d 10 h
- PR fusionados (30 d)
- 6
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de apache/burr
-
status/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
apache/burr#809 · 2 comentarios ·
Los mantenedores suelen responder en 2 días
-
Streaming Event type, type hint, should support union typePosiblemente ocupada @mvanhorn la tomó hace 184 días. Abiertoarea/streaming area/typing good first issue help wanted kind/bug priority/high
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
apache/burr#607 · 12 comentarios ·
Los mantenedores suelen responder en 2 días
-
Add more instructions for applying ORM migrations for S3 deploymentPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertoarea/storage area/ui help wanted kind/documentation priority/low
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
apache/burr#524 · 11 comentarios ·
Los mantenedores suelen responder en 2 días
-
Add documentation on how to run the Burr UI serverQuizá libre de nuevo @iamSerafinnn la tomó hace 123 días y no hay ningún pull request abierto. Abiertoarea/ui good first issue kind/documentation priority/low
Dificultad 2/5 Medio día Aptitud para principiantes 68/100
apache/burr#272 · 5 comentarios ·
Los mantenedores suelen responder en 2 días
-
status/needs-triage
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Los mantenedores suelen responder en 2 días
Todos los issues de apache/burr
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
MystenLabs/MemWal#1163 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
infertopics leaves new nodes without a topic when untopiced neighbours outnumber topiced onesPosiblemente ocupada @moneebullah25 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
FinanceFlash/unvibecode#218 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
NVIDIA/earth2studio#1241 ·
Los mantenedores suelen responder en 3 días