Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Se interrumpe tlon-ai-fix-markdown-format en 7.1.1 #29

Closed
worldsaround opened this issue Mar 4, 2025 · 21 comments
Closed

Se interrumpe tlon-ai-fix-markdown-format en 7.1.1 #29

worldsaround opened this issue Mar 4, 2025 · 21 comments

Comments

@worldsaround
Copy link
Member

worldsaround commented Mar 4, 2025

Descripción

Cuando ejecuto tlon-ai-fix-markdown-format en 7.1.1, el proceso se interrumpe cuando alcanza el 99% y no se completa. El debugger no muestra ningún error. Esto es así en general, aunque excepcionalmente el proceso se sí se ha completado correctamente.

Pasos para reproducir

  1. H-r a f en cucalquier archivo markdown de los que contiene uqbar-ar. Por ejemplo, la contraparte árabe de what-is-evidence.md.

Backtrace

No "salta".

Información del sistema

  • Tlon version: 1.6.7
  • Latest commit: 1588078

Comentarios

No tiene ningún problema el perfil 6.9.1 con:

  • Package version: 1.6.7
  • Latest commit: f4e054b

Muy bueno el template!

@benthamite
Copy link
Member

Tal vez tenga que ver con el modelo? Si seteás manualmente claude-3-5-sonnet-20241022 (via H-s-m -m), que es el modelo que estábamos usando antes de la última actualización, ¿el problema se resuelve?

@benthamite benthamite changed the title Se interrumpe tlon-ai-fix-markdown-format en 7.7.1 Se interrumpe tlon-ai-fix-markdown-format en 7.1.1 Mar 4, 2025
@benthamite
Copy link
Member

(Cambié 7.7.1 a 7.1.1 dado que este último es la versión del perfil; la confusión se produjo porque inicialmente la tag que yo creé tenía el número incorrecto.)

@worldsaround
Copy link
Member Author

No es eso. Podrías probar si podés reproducir el error en 7.7.1?

@benthamite
Copy link
Member

Probé de correr H-r a f en what-is-evidence.md (en árabe) y puedo reproducir el problema, pero persiste aun cuando hago un checkout a f4e054b y reinicio Emacs. ¿Podés confirmar que todo sigue funcionando bien en tu máquina, con ese commit y perfil?

@benthamite
Copy link
Member

benthamite commented Mar 4, 2025

Me vendría bien también si podés identificar otro perfil anterior al 6.9.1 en el que funcione bien (corroborado reiniciando Emacs en ese perfil y corriendo el comando relevante en el archivo en cuestión). En el perfil 6.9.1 me tira un error cuando corro H-r a f no relacionado con este problema, probablemente porque en su momento actualicé algo y el perfil difiere del tuyo.

@benthamite
Copy link
Member

benthamite commented Mar 4, 2025

Recién, luego de evaluar el buffer de tlon-ai.md (en el perfil más reciente, con la más reciente versión de tlon.el), volví a correr H-r a f en el mismo archivo y funcionó. ¿Podés fijarte si a vos te funciona luego de re-evaluar tlon-ai (s-b) en 7.1.1? Tal vez sea un problem de en qué order carga la función, o algo por el estilo.

@worldsaround
Copy link
Member Author

No es un fix válido. porque funciona erráticamente. Me consta que no soluciona el problema. Quizá si lo volvieras a correr no te funcionaría (estoy casi seguro).

@benthamite
Copy link
Member

benthamite commented Mar 7, 2025

Necesito identificar una versión que determinísticamente produzca siempre el mismo resultado exitoso. Podrías pasarme lo siguiente (siempre del perfil 6.9.1, asumiendo que hayas confirmado a ciencia cierta que tlon-ai-fix-markdown-format funciona sin problemas)?

  1. Abrí cualquier archivo de tlon (H-r H-o), corré magit y pasame el commit más reciente.
  2. Abrí config.org, corré magit y pasame el commit más reciente.

@benthamite
Copy link
Member

Y también

  1. Abrí cualquier archivo de gptel (M-L gptel RET), corré magit y pasame el commit más reciente.

Es posible que el problema esté causado no por un cambio en tlon o mi config, sino en gptel.

@worldsaround
Copy link
Member Author

  1. f4e054b
  2. 65bd8bc9 (en master) 0ff6c60b en leo-stable (creo que ese perfil nació de esa rama: pero ya no estoy seguro)
  3. aa649b0

@benthamite
Copy link
Member

Gracias.

Primera conclusión: no es gptel dado que con los commits en (1) y (2), pero el commit más reciente en (3), sigue funcionando bien.

@benthamite
Copy link
Member

Cuando pruebes los cambios que empujé ayer, decime si el problema se resuelve. Sin apuro.

@worldsaround
Copy link
Member Author

Creo que lo probé y se resolvió. Cierro por ahora y reabro de ser necesario.

@benthamite
Copy link
Member

benthamite commented Mar 10, 2025

Creo que, en general, vale la pena prestar atención a cómo funciona este comando. Me parece (aunque no estoy seguro) que la versión anterior que parecía funcionar sin problemas fallaba silenciosamente con algunos requests: es decir que algunos de los párrafos quedaban sin cambios, no porque el modelo no los sugiriera sino porque la respuesta nunca volvía. Dado que la adaptación del Markdown es un proceso que lleva mucho tiempo, es importante considerar posibles optimizaciones que puedan hacerse a esta funcionalidad.

@worldsaround
Copy link
Member Author

Nunca noté eso que decís

@worldsaround
Copy link
Member Author

Reabro este issue, porque el nuevo perfil (o la nueva versión del paquete tlon, no lo sé porque actualicé llos dos juntos) tiene exactamente el mismo problema, que ya no tiene 7.1.1. Supongo que el fix será el mismo.

@worldsaround worldsaround reopened this Mar 17, 2025
@benthamite
Copy link
Member

Cuál es el último commit en los repos de config y de tlon en el perfil 7.1.1?

@worldsaround
Copy link
Member Author

tlon: ca4ebad
config: 65bd8bc9 @ 7.7.1 7.1.1 Update to config version 7.1.1

@benthamite
Copy link
Member

A cambios en tlon no puede deberse dado que hubo solo media docena de commit desde ca4ebad y ninguno es en tlon-ai.

Lo voy a investigar.

@benthamite
Copy link
Member

¿Alguna novedad de esto?

@worldsaround
Copy link
Member Author

Sí, llegó a funcionar en el último perfil 7.1.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants