Yvoty #32

dlopezalvas · 2023-05-15T22:51:21Z

Resolves Program-AR/pilas-bloques-ember#1305

Pilas-bloques: Program-AR/pilas-bloques-ember#1324

dlopezalvas · 2023-05-15T23:37:07Z

src/escenas/libroPrimaria/EscenaDesdeMapa.ts

@@ -244,6 +244,7 @@ class GeneradorDeMapasAleatorios implements GeneradorDeMapas {
        this.colecciones = opciones.colecciones || {};
        this.macros = opciones.macros || {};
        this._anotadosParaColeccion = [];
+        this._anotadosParaColecciones = {};


Esto lo agregué porque no estaba andando, no teníamos ningún ejercicio que usara múltiples colecciones asi que quizás es un bug que se pasó de antes

dlopezalvas · 2023-05-15T23:38:47Z

src/escenas/libroPrimaria/EscenaYvoty.ts

+			case 'A': return this.automata;
+			case 'O': return this.obtenerObstaculo(nroFila, nroColumna);
+			case 'C': return new Celular();
+			case 'P': return new Celular(true); //celular prendido
+			case 'K': return new Cargador();
+			case 'L': return new Luciernaga();
+			case 'M': return new Mariposa();
+			case 'T': return new CompuAnimada(0, 0);


Mi lógica para estos identificadores fue la siguiente:

C -> self explanatory

P -> celular Prendido (la C ya esta ocupada)

K -> porque k suena a CArgador (?) (la C ya esta ocupada)

L -> self explanatory

M -> self explanatory

T -> por compuTadora (?) (la C ya esta ocupada)

Posiblemente hayan mejores identificadores, si se les ocurren los cambiamos

Excelente 💯

dlopezalvas · 2023-05-15T23:40:44Z

src/actores/segundoCiclo/Yvoty/Yvoty.ts

+		this.definirAnimacion("error", [30, 30, 30, 30, 30, 30, 30, 31, 32, 33, 34, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 36, 36, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37], 12)
+		this.definirAnimacion("recoger", [0, 0, 0, 0, 1, 1, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 3, 3, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1], 20);
+		this.definirAnimacion("usarCelu", [38, 38, 38, 39, 39, 40, 40, 40, 41, 42, 43, 41, 43, 42, 42, 42, 44, 45, 46, 47, 46, 42, 42, 42, 45, 46, 46, 46, 45, 44, 48, 48, 48, 49, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 51, 51, 38, 38, 38, 38, 38, 38, 38], 20);
+		this.definirAnimacion("sacarFoto", [38, 38, 38, 39, 39, 40, 40, 40, 41, 42, 43, 41, 43, 42, 42, 42, 44, 45, 46, 47, 46, 42, 42, 42, 45, 46, 46, 46, 45, 44, 48, 48, 48, 49, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 51, 51, 38, 38, 38, 38, 38, 38, 38], 20);


Esta animación está exactamente igual a la de usarCelu a propósito, pero para cambiar solo acá al momento de que definamos cómo va a ser la de sacar foto.

LPM qué laburazo todo esto, bravo @rgonzalezt .
Sobre la animación que falta, qué les parece si no es simplemente 38, 39 y 40, o sea un sólo dedito apoyado. Si además podemos agregar a la grilla uno igual a la 40 pero con una luz tipo estrellita en el celular, genial. Es un hackazo rápido.

asanzo

Che, increíble laburo, @dlopezalvas . Es muchísimo, realmente zarpado que lo hayas completado en el tiempo que lo hiciste. Bravo. Marqué un par de cosas, todavía no me lo pude bajar para probarlo. Para mí amerita release, al menos para ir linkeándolo a PB.

asanzo · 2023-05-17T00:36:19Z

src/actores/CompuAnimada.ts

@@ -2,10 +2,12 @@

 class CompuAnimada extends ActorAnimado {

+  static _grilla = 'compu_animada.png'


UFFF bien al detectar esto. Esto es parte del sistema necesario para precargar sólo las imágenes de este desafío y no de otro, @danielferro69 tené en cuenta estos static _grilla. El algoritmo de precargado está acá:

pilas-bloques-exercises/src/escenas/EscenaActividad.ts

Lines 23 to 28 in 2b1173b

static imagenesPreCarga(): string[]{

var imgsPrecargar = this.pathFondo() ? [this.pathFondo()] : [];

this.clasesDeActoresInvolucrados().forEach(c => imgsPrecargar = imgsPrecargar.concat(c.imagenesPreCarga()));

var imgs = imgsPrecargar.concat(this.imagenesAdicionales());

return this.faltanImagenes ? [] : imgs

}

asanzo · 2023-05-17T00:42:57Z

src/assets/actor.yvoty.png

@rgonzalezt qué pasó con la silueta con flash?
Habíamos hablado de que parecía un glitch el cuadrado blanco, y mejor tener una silueta en blanco.

Tengo que aplaudir el laburo de optimización que hiciste, Ro.
Marco un par más que podrían optimizarse, aunque quizás ya lo intentaste y no son exactamente iguales. Igual quizás a esta altura no valga la pena, podemos patear. Marco con la misma letra los que me parecen la misma imagen:

asanzo · 2023-05-17T01:11:28Z

src/comportamientos/CargarCelular.ts

+    postAnimacion() {
+        super.postAnimacion()
+        if (pilas.escena_actual().celularesCargados()) {
+            pilas.escena_actual().estado = new Estado(() =>
+                true
+            )
+        }
+    }


Entender este método requiere haber leído un tocazo, excelente. Creo que casi todo exercises está acá.

asanzo · 2023-05-17T01:16:04Z

src/actores/segundoCiclo/Yvoty/Yvoty.ts

+		this.definirAnimacion("error", [30, 30, 30, 30, 30, 30, 30, 31, 32, 33, 34, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 36, 36, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37], 12)
+		this.definirAnimacion("recoger", [0, 0, 0, 0, 1, 1, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 3, 3, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1], 20);
+		this.definirAnimacion("usarCelu", [38, 38, 38, 39, 39, 40, 40, 40, 41, 42, 43, 41, 43, 42, 42, 42, 44, 45, 46, 47, 46, 42, 42, 42, 45, 46, 46, 46, 45, 44, 48, 48, 48, 49, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 51, 51, 38, 38, 38, 38, 38, 38, 38], 20);
+		this.definirAnimacion("sacarFoto", [38, 38, 38, 39, 39, 40, 40, 40, 41, 42, 43, 41, 43, 42, 42, 42, 44, 45, 46, 47, 46, 42, 42, 42, 45, 46, 46, 46, 45, 44, 48, 48, 48, 49, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 51, 51, 38, 38, 38, 38, 38, 38, 38], 20);


LPM qué laburazo todo esto, bravo @rgonzalezt .
Sobre la animación que falta, qué les parece si no es simplemente 38, 39 y 40, o sea un sólo dedito apoyado. Si además podemos agregar a la grilla uno igual a la 40 pero con una luz tipo estrellita en el celular, genial. Es un hackazo rápido.

asanzo · 2023-05-17T01:17:07Z

src/assets/actor.cargador.png

Me pregunto si vamos a tener obstáculos u objetos que alguna vez superen los 100px de alto. Podríamos definir ese máximo, encontrarlo mirando los ejercicios actuales. Porque si llegamos a entender que nunca van a superar cierto tamaño (o sólo pocas veces), achicamos bastante las imágenes y su peso.

asanzo · 2023-05-17T01:19:50Z

src/escenas/InstalandoJuegosYvoty.ts

+/// <reference path = "../comportamientos/ComportamientoAnimado.ts" />
+/// <reference path = "../comportamientos/MovimientosEnCuadricula.ts" />
+
+class InstalandoJuegosYvoty extends InstalandoJuegos {


Faltan los métodos de optimización de carga de imágenes: clasesDeActoresInvolucrados y pathFondo. Lo mismo en todas las escenas nuevas que se creen, ténganlo en cuenta.

pilas-bloques-exercises/src/escenas/libroPrimaria/EscenaLita.ts

Lines 12 to 18 in 2b1173b

static clasesDeActoresInvolucrados(): typeof ActorAnimado[] {

return [Lita, Ensaladera, Tomate, Lechuga];

};

static pathFondo(): string {

return 'fondo.lita.png';

}

asanzo · 2023-05-17T01:20:27Z

src/escenas/libroPrimaria/EscenaYvoty.ts

+			case 'A': return this.automata;
+			case 'O': return this.obtenerObstaculo(nroFila, nroColumna);
+			case 'C': return new Celular();
+			case 'P': return new Celular(true); //celular prendido
+			case 'K': return new Cargador();
+			case 'L': return new Luciernaga();
+			case 'M': return new Mariposa();
+			case 'T': return new CompuAnimada(0, 0);


Excelente 💯

asanzo · 2023-05-17T01:20:59Z

src/escenas/libroPrimaria/EscenaYvoty.ts

+	static clasesDeActoresInvolucrados(): typeof ActorAnimado[] {
+		return [Yvoty, Celular, Luciernaga, Cargador, Mariposa, CompuAnimada];
+	};
+
+	static pathFondo(): string {
+		return 'fondo.yvoty.png';
+	}


Ufff acá viste lo que menciono arriba. Zarpado, no tengo que explicar nada.

asanzo · 2023-05-17T01:22:26Z

src/comportamientos/SacarFoto.ts

+    public nombreProximaAnimacion(): string {
+        return "despierta"
+    }


Esta es la animación en el actor tocado, no? Qué mal nombre le puse a este método.

dlopezalvas added 9 commits May 8, 2023 13:57

luciernaga + yvoty

06c06e0

possible collection fix

acc871c

luciernaga + mariposa

2929e0b

celular + cargador

642f0ed

celulares prendidos

d8b1dfa

prendiendo compus

25e407b

instalando juegos yvoty

81b8341

some visual changes

bc093de

visual changes

a2f3184

dlopezalvas mentioned this pull request May 15, 2023

Yvoty Program-AR/pilas-bloques-ember#1324

Merged

6 tasks

sacar foto animacion

fb9f773

dlopezalvas commented May 15, 2023

View reviewed changes

dlopezalvas requested a review from a team May 15, 2023 23:41

Celular asset name

9011322

asanzo approved these changes May 17, 2023

View reviewed changes

dlopezalvas mentioned this pull request May 19, 2023

Iteración Yvoty Program-AR/pilas-bloques-ember#1334

Closed

5 tasks

1.2.18

4e2d058

dlopezalvas merged commit 1ff022c into master May 19, 2023

dlopezalvas deleted the yvoty branch May 19, 2023 15:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Yvoty #32

Yvoty #32

dlopezalvas commented May 15, 2023

dlopezalvas May 15, 2023

dlopezalvas May 15, 2023 •

edited

Loading

asanzo May 17, 2023

dlopezalvas May 15, 2023

asanzo May 17, 2023

asanzo left a comment

asanzo May 17, 2023

asanzo May 17, 2023

asanzo May 17, 2023

asanzo May 17, 2023

asanzo May 17, 2023

asanzo May 17, 2023

asanzo May 17, 2023

asanzo May 17, 2023

asanzo May 17, 2023

asanzo May 17, 2023

		@@ -2,10 +2,12 @@

		class CompuAnimada extends ActorAnimado {

		static _grilla = 'compu_animada.png'

	static imagenesPreCarga(): string[]{
	var imgsPrecargar = this.pathFondo() ? [this.pathFondo()] : [];
	this.clasesDeActoresInvolucrados().forEach(c => imgsPrecargar = imgsPrecargar.concat(c.imagenesPreCarga()));
	var imgs = imgsPrecargar.concat(this.imagenesAdicionales());
	return this.faltanImagenes ? [] : imgs
	}

	static clasesDeActoresInvolucrados(): typeof ActorAnimado[] {
	return [Lita, Ensaladera, Tomate, Lechuga];
	};

	static pathFondo(): string {
	return 'fondo.lita.png';
	}

Yvoty #32

Yvoty #32

Conversation

dlopezalvas commented May 15, 2023

Choose a reason for hiding this comment

dlopezalvas May 15, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

asanzo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dlopezalvas May 15, 2023 •

edited

Loading