Fix kernel method missing slowing apps

[difernandez]

NO ELIMINAR RAMA, ESTE FORK NO ESTÁ PUBLICADO COMO GEMA Y SE USA DIRECTO DESDE LA GITHUB EN UNA RAMA

Contexto

Rulp es una gema para modelar y resolver problemas de optimización/programación lineal. Permite definir variables enteras, binarias y sin restricciones. Se usa en uno de los proyectos de platanus y se usa fork desde una rama debido al problema descrito en #2.
En el proyecto que se usa, desde que se usa esta gema hubo un problema que hacía que en ambiente de desarrollo recargar una vista fuera muy lento.

Qué se está haciendo

Se determinó que las causas del problema anterior eran los monkeypatch a Kernel#method_missing y Object#const_missing, que permitían definir variables del problema de programación lineal poniendo directamente algo como X_i en el código. El primero sobre todo generaba mucha lentitud, ese método se llama muchas veces al cargar una vista o hacer cualquier cosa en Rails. Probando, se descubrió que el problema no era qué se hacía en el monkeypatch, ya que algo tan simple como lo siguiente también provocaba la misma ralentización:

module Kernel
  alias_method :old_method_missing, :method_missing
  def method_missing(value, *args)
    old_method_missing(value, *args)
  end
end

Entonces, para solucionar esto, se hace que las variables no se puedan definir simplemente como X_i, si no que se requiere que tengan Rulp:: al principio, es decir sería Rulp::X_i. Así, el method_missing y const_missing se pueden implementar en el module Rulp en vez de Kernel, evitando que se llame en casi cada cosa que se hace en Rails.
Para que funcionara bien, se tuvo que también definir un method_missing en LV, clase que representa una variable, con distintas subclases para los distintos tipos. Se llamaba por ahí con el to_proc, no entendí demasiado bien por qué, no quise reestructurar nada ni indagar demasiado.
El cambio logro sacar la lentitud, pasando de uno o dos minutos a algunos pocos segundos

[ankaph]

lo veo bien!

[ldlsegovia]

Qué bravo lo que se estaba mandando la gema. No habrá una alternativa? El cambio me parece bien. No tengo mucho que decir si el tiempo bajó 👍🏻

[difernandez]

En ambiente de desarrollo, recargar una página es muy lento

[difernandez]

No habrá una alternativa?

mmm a la rápida vi y hay un par de cosas de Linear Programming, pero nada muy usado/mantenido. Además, es una de esas cosas que se usa hace mucho en el proyecto y no es tan simple de entender, al menos por ahora no veo que convenga cambiar a otra cosa