Bundling Actions Together
Los mantenedores suelen responder en 4 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
- 209
- Merge medio
- 8 d 9 h
- PR fusionados (30 d)
- 26
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
-
Notebook UI launcher should use the current Python interpreterPosiblemente ocupada @sivakumar-mahalingam la tomó hace 3 días. Abiertostatus/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 4 días
-
astep() passes unprocessed inputs to pre_run_step hooksPosiblemente ocupada @sivakumar-mahalingam la tomó hace 3 días. Abiertostatus/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 4 días
-
`burr` fails to start when the Python path has a space in itPosiblemente ocupada @enekoreto la tomó hace 6 días. Abiertostatus/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 4 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 4 días
-
Add documentation on how to run the Burr UI serverQuizá libre de nuevo @iamSerafinnn la tomó hace 135 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 4 días
Todos los issues de apache/burr
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
zach-blumenfeld/aip#13 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
nathanmarlor/foxess_modbus#1287 ·
Los mantenedores suelen responder en 4 días
-
bug good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
OpenLakeForge/openlakeforge#498 ·
Los mantenedores suelen responder en 1 día
-
good first issue hacktoberfest help wanted python
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
Los mantenedores suelen responder en 1 día
-
Exact onnxruntime==1.27.0 core pin makes Pipecat-based apps uninstallable from the dashboardAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
pollen-robotics/reachy_mini#1493 ·
Los mantenedores suelen responder en 1 día