scverse / scverse/spatialdata-plot
Feature request: remove need for intermediate pl calls when chaining renderers
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 86
- Forks
- 21
- Merge medio
- 14 h 50 min
- PR fusionados (30 d)
- 3
Descripción
Description of feature
Thanks for the super nice work, @timtreis and @sagar87 ! I think it would be nice if we didn't have to add intermediate calls to pl when chaining renderers. That is, I think it would be nice if instead of this:
sdata.pl.render_images().pl.render_shapes().pl.show()
We could do this:
sdata.pl.render_images().render_shapes().show()
This perhaps could be possible if the render_*() methods returned the plot accessor instead of the SpatialData object. I suppose this means that one couldn't do a pp operation after a pl operation, but maybe that suggests the pl and pp operations should be on the same level of the hierarchy or one should do their preprocessing first, then their plotting. I'm not sure what's best here, but it might be good to experiment with this once all of the p0 functionality is finished.
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.
Línea de trabajo
Comienza inspeccionando el accesor pl y los métodos render_*, especialmente cómo devuelven actualmente SpatialData y admiten llamadas encadenadas. Compara el ejemplo existente sdata.pl.render_images().pl.render_shapes().pl.show() con el comportamiento de encadenamiento propuesto y resuelve cómo deben interactuar las operaciones pl y pp. Se considera terminado cuando haya un diseño de API decidido y el encadenamiento de renderers funcione sin llamadas pl intermedias.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- data-visualization
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Tranquilo
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100