search

uqbar-project / wollok-ts

TypeScript based Wollok language implementation
GNU General Public License v3.0
20 stars 14 forks source link

Mover código Wollok a Language #60

Closed PalumboN closed 2 years ago

PalumboN commented 4 years ago

Para la implementación de Game se tuvieron que agregar algunas definiciones (y tests) escritos en Wollok que estaría bueno tener en Language.

Ver los directorios src/wre y test/game.

nscarcella commented 2 years ago

@PalumboN @asanzo si llegara a poderse hacer esto con los pasantes sería un golazo... Es un atarea más bien atómica y nos permitiría simplificar el buildeo.

asanzo commented 2 years ago

Esto se resolvería junto con https://github.com/uqbar-project/wollok-language/issues/48

PalumboN commented 2 years ago

Holisss, venía acá a decir que si querían me podía ocupar de esto pero veo que justo fue asignado.

@FerRomMu @julimr avisen si necesitan ayuda con algo! :)

asanzo commented 2 years ago
FerRomMu commented 2 years ago

Algunas cosas que estuvimos viendo:

  1. Juli detectó que la razón por la que no fallaban los metodos que tenian 'game.mirror.mirror' era porque al hacer el cambio en language ya no los llamaban desde ts (ya no se usaba native). Así que basto con quitarlos y no afectó nada. Por otro lado, buscamos mas referencias respecto a io.io y gameMirror pero no parece haber mas. 7404d8016f6826f4aaca4e7e3383a897bd0588ab
  2. Estuvimos probando levantar el wollok-run-client local pero no pudimos ninguno de los dos. A juli le falla el npm install (incluso con la version de node que el repo pide) y no termina de instalar. A mi me logró instalar pero al hacer el npm start queda cargando en el explorador de manera eterna.
  3. Al final no tengo claro que falten tantos como pensamos. Resulto ser que esta dispersos entre varios archivos de .wtest, en el caso de los métodos que siguen siendo native y tienen retorno tienen test en sanity/game y en el caso de aquellos natives que retornan void tienen test de ts. Quedan algunos métodos sin testear que no tenemos claro como se podrían testear:
    • ground(image)
    • doCellSize
    • boardGround(image)
    • hideAttributes
    • showAttributes
    • errorReporter
  4. Ya existe en ts. https://github.com/uqbar-project/wollok-ts/blob/7404d8016f6826f4aaca4e7e3383a897bd0588ab/test/game.test.ts#L36-L41
  5. Lo hicimos todo pareciera seguir igual. Probamos levantar un game y nos lanzo el mismo error que ya lanzaba antes: image
nscarcella commented 2 years ago

@FerRomMu @julimr levantaron el proxy como dice el README?

FerRomMu commented 2 years ago

Como vimos en la meet utilizando REACT_APP_PROXY_URL=http://localhost:8787 npm start funciona. No pudimos hacer andar repositorios remotos, debemos estar usando mal el comando npm run cors (que de hecho actualmente no hace diferencia que este o no andando en otra terminal). Sí pudimos hacer andar locales. Todo esto con el ts de siempre. @PalumboN ¿Alguna idea de como hacer para que nos cargue los repositorios remotos? Ademas de: REACT_APP_PROXY_URL=http://localhost:8787 npm start y npm run cors en otra consola, ¿Habría que correr algo mas?

Por otro lado usando los comandos de: https://stackoverflow.com/a/38417065 o bien copiando y pegando la carpeta de manera manual de ts, tras andar un rato salta el siguiente error: Captura de pantalla de 2022-07-22 15-33-32

@nscarcella tenes idea de que podríamos estar haciendo mal a la hora de traernos el ts al node-modules del run-client? Checkoutie a master el ts para mover el master al run-client y ver si era la versión de la branch la que generaba el error, pero me siguió apareciendo lo mismo.

nscarcella commented 2 years ago

Es posible que el run-client esté usando una versión de wollok-ts vieja y no haya sido adaptado a los últimos cambios.

Les diría que antes de arrancar a probar contra el ts local le pongan la versión más reciente publicada y se fijen si todo compila. Si no, habría que arreglar lo que se esté rompiendo antes de empezar a testear los juegos.

El vie, 22 jul 2022 a la(s) 16:36, Fernando Romero @.***) escribió:

Como vimos en la meet utilizando REACT_APP_PROXY_URL=http://localhost:8787 npm start funciona. No pudimos hacer andar repositorios remotos, debemos estar usando mal el comando npm run cors (que de hecho actualmente no hace diferencia que este o no andando en otra terminal). Sí pudimos hacer andar locales. Todo esto con el ts de siempre. @PalumboN https://github.com/PalumboN ¿Alguna idea de como hacer para que nos cargue los repositorios remotos? Ademas de: REACT_APP_PROXY_URL=http://localhost:8787 npm start y npm run cors en otra consola, ¿Habría que correr algo mas?

Por otro lado usando los comandos de: https://stackoverflow.com/a/38417065 o bien copiando y pegando la carpeta de manera manual de ts, tras andar un rato salta el siguiente error: [image: Captura de pantalla de 2022-07-22 15-33-32] https://user-images.githubusercontent.com/70177008/180511735-2209be82-87c0-46dd-8286-ec9d39ede082.png

@nscarcella https://github.com/nscarcella tenes idea de que podríamos estar haciendo mal a la hora de traernos el ts al node-modules del run-client? Checkoutie a master el ts para mover el master al run-client y ver si era la versión de la branch la que generaba el error, pero me siguió apareciendo lo mismo.

— Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok-ts/issues/60#issuecomment-1192880015, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFPE2YXBDXE6TY7SKZA75LVVLZ5JANCNFSM4QIJR6JQ . You are receiving this because you were mentioned.Message ID: @.***>

PalumboN commented 2 years ago

@FerRomMu Para probar juegos desde git tienen que tener una consola con el CORS corriendo, y otra con el start como decís. Sino lo más fácil es cargar un proyecto localmente, desde sus compus. Cualquier de pepita debería funcionar (como el que viene por defecto en la url, lo pueden clonar y cargar localmente). Para eso no hace falta el CORS, solamente para clonar el repo desde github.

Si están teniendo problemas para probar los cambios que hicieron nos podemos conectar a una meet la semana que viene y lo vemos en vivo, antes del jueves (después me voy de vacas!).

FerRomMu commented 2 years ago

Es posible que el run-client esté usando una versión de wollok-ts vieja y no haya sido adaptado a los últimos cambios. Les diría que antes de arrancar a probar contra el ts local le pongan la versión más reciente publicada y se fijen si todo compila. Si no, habría que arreglar lo que se esté rompiendo antes de empezar a testear los juegos. El vie, 22 jul 2022 a la(s) 16:36, Fernando Romero @.) escribió: … Como vimos en la meet utilizando REACT_APP_PROXY_URL=http://localhost:8787 npm start funciona. No pudimos hacer andar repositorios remotos, debemos estar usando mal el comando npm run cors (que de hecho actualmente no hace diferencia que este o no andando en otra terminal). Sí pudimos hacer andar locales. Todo esto con el ts de siempre. @PalumboN https://github.com/PalumboN ¿Alguna idea de como hacer para que nos cargue los repositorios remotos? Ademas de: REACT_APP_PROXY_URL=http://localhost:8787 npm start y npm run cors en otra consola, ¿Habría que correr algo mas? Por otro lado usando los comandos de: https://stackoverflow.com/a/38417065 o bien copiando y pegando la carpeta de manera manual de ts, tras andar un rato salta el siguiente error: [image: Captura de pantalla de 2022-07-22 15-33-32] https://user-images.githubusercontent.com/70177008/180511735-2209be82-87c0-46dd-8286-ec9d39ede082.png @nscarcella https://github.com/nscarcella tenes idea de que podríamos estar haciendo mal a la hora de traernos el ts al node-modules del run-client? Checkoutie a master el ts para mover el master al run-client y ver si era la versión de la branch la que generaba el error, pero me siguió apareciendo lo mismo. — Reply to this email directly, view it on GitHub <#60 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFPE2YXBDXE6TY7SKZA75LVVLZ5JANCNFSM4QIJR6JQ . You are receiving this because you were mentioned.Message ID: @.>

Eso hicimos. Si estamos poniendo la versión actual (o sea main) de manera correcta, entonces esta fallando eso. Porque el error sigue apareciendo. Aún sigo sin estar seguro de estar instalando bien la dependencia de manera local. Lo que me hace pensar que lo estoy haciendo bien es que el error tarda en aparecer y en el mientras tengo el run-client corriendo. Tiré un npm view wollok-ts version y en ambos casos (tanto con el npm i por default como poniendole el main actual desde local) me dice que la version es 3.1.1.