Consider adding the ability to pass `p` arguments to `profile.start`
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- lua
- Área
- performance
Línea de trabajo
Start by reading lua/plenary/profile.lua around lines 12-18 and the option handling in lua/plenary/profile/p.lua around lines 22-38. Trace how profile.start currently invokes jit.p, then update the API so callers can provide pargs or override the defaults. Done means profile output can include options such as p and F without requiring direct jit.p calls.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I was using the plenary.profile package and capturing profiles using it. It looks like it's super useful for my case. I was using the flamegraph output to get some nice profiler to visualize in the Firefox Profiler. But the profiles that I gathered weren't so useful.
For example here's one: https://share.firefox.dev/4ejyKHG
It wasn't very useful to me because it wasn't possible to see which file or function comes from which module exactly.
But after looking at the documentation of jit.p, I realized that it's possible to send different options that might make it more useful for me.
For example, here I see that I can also pass p and F that could make the output more useful for me:
https://github.com/nvim-lua/plenary.nvim/blob/857c5ac632080dba10aae49dba902ce3abf91b35/lua/plenary/profile/p.lua#L22-L38
But currently there is no way to pass additional popts or override the default ones:
https://github.com/nvim-lua/plenary.nvim/blob/857c5ac632080dba10aae49dba902ce3abf91b35/lua/plenary/profile.lua#L12-L18
It's not the end of the worlds since I can still manually add these arguments or call jit.p directly. But it would be nicer if plenary had an option to do it too.
Here's the output with these:
https://share.firefox.dev/460Xsdw
Now it's a lot more actionable for me, since I can see which file comes from which package exactly.
So my feature request would be to make this profiler.start API accept pargs as an argument so users can override them.
- Lenguaje dominante
- Lua
- Estrellas
- 3.5k
- Forks
- 341
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 nvim-lua/plenary.nvim
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
nvim-lua/plenary.nvim#682 ·
-
uniform usage of `os.getenv`Posiblemente ocupada @rakotomandimby la tomó hace 122 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
nvim-lua/plenary.nvim#680 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
nvim-lua/plenary.nvim#675 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 28/100
nvim-lua/plenary.nvim#672 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
nvim-lua/plenary.nvim#671 · 1 comentario ·
Todos los issues de nvim-lua/plenary.nvim
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 3 días
-
Campaign WC
Dificultad 2/5 1-3 horas Aptitud para principiantes 60/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
public-transport/transitous#2569 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
-
core: a monitored method's raise rewrites every ? of its message, a resumed runtime's text includedPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertoseverity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
luainkernel/lunatik#1861 ·
Los mantenedores suelen responder en 1 día