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

Casilla final - parte 1 #56

Merged
merged 9 commits into from
Oct 17, 2023
Merged

Casilla final - parte 1 #56

merged 9 commits into from
Oct 17, 2023

Conversation

dlopezalvas
Copy link
Contributor

@dlopezalvas dlopezalvas commented Oct 17, 2023

Related Program-AR/pilas-bloques-ember#1435

pilas-bloques: Program-AR/pilas-bloques-ember#1487

imagen
imagen
imagen
imagen

Con el cambio de las pelotas con más calidad quedó un cachito más grande acá pero me parece que queda bien:

imagen

Comment on lines +10 to +13
new Cuadros(32).repetirVeces(18)
.concat([33, 34, 34, 32, 32, 32, 32, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 44, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32])
.concat(new Cuadros(32).repetirVeces(18))
.concat([33, 34, 34, 32, 32, 32, 32, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 44, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32])
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto es un fix para la animación de Mañic

Comment on lines +43 to +53
static imagenesAdicionales(): string[] {
return [Casilla, Obstaculo, MetaFinal].map(clase => clase.imagenesPara(this.nombreAutomata())).reduce((acc, list) => list.concat(acc), []);
}

/**
* Devuelve el nombre del automata, usado para la precarga de metas finales, obstaculos y casillas
* Pensado para redefinirse por escena.
*/
static nombreAutomata() {
return ''
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Para no tener que hacer copypaste con cada escena nueva, solo se define el nombre del automata

Comment on lines +90 to +92
posicionValida(posFinal: [number, number]){
return posFinal[0] >= 0 && posFinal[1] >= 0
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En Chuy habían dos casos que tenían una posición no válida para que tenga una posición pero chequee por si llegó al final del camino en lugar de si está en la posición final. Esto puede que cambie cuando definamos la gramática; estaba como trucazo para el caso de tener mapas con diferentes posiciones finales.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Para que se vea mejor en casos como los ejercicios de dos casillas de Chuy:

imagen

Se veía pixeleado si no.

Comment on lines -71 to 67
todosLosActoresCumplen(actor, estado) {
return this.obtenerActoresConEtiqueta(actor).every(o => o.nombreAnimacionActual() == estado);
}

telescopiosArreglados(): boolean {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saque de Capy y Mañic métodos que ya existen en EscenaDesdeMapa

@dlopezalvas dlopezalvas marked this pull request as ready for review October 17, 2023 15:40
Copy link
Contributor

@danielferro69 danielferro69 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Que bueno ! 💯
Lo bajé y hasta donde lo probé todo funciona joya !

@dlopezalvas dlopezalvas merged commit ad41d95 into master Oct 17, 2023
3 checks passed
@dlopezalvas dlopezalvas deleted the casilla-final branch October 17, 2023 20:56
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

Successfully merging this pull request may close these issues.

2 participants