vmbatlle / super-mario-kart Goto Github PK

Si se cierra el juego cuando hay un thread pendiente (p. ej cálculo del gradiente), sale un mensaje de error al no hacer join

Al igual que con EffectDrown, hacer partículas cuando te chocas contra una pared (ver misc/particles.png, las amarillas)

Efecto esperado:
La música debería permanecer reproduciéndose durante toda la carrera. Opcionalmente, modificar la pista reproducida durante la última vuelta por la correspondiente música acelerada.

Efecto actual:
La música termina a los 01' 40'' de juego (caso Mario Circuit 2).
La música termina a los 02' 54'' de juego (caso Donut Plain).
Etc.

Hay que tener en cuenta tres cosas:

• Necesitamos sprite de sombra
• Todas las funciones getXXXDrawables(...) de map.cpp pueden devolver un segundo drawable que sea la sombra, situado en la misma posición que el jugador y con altura 0 (asegurar que la sombra se pinta detrás del jugador)
• Hay que tener en cuenta el caso especial donde es el propio jugador el que salta, necesitando añadir otro caso especial a la función Driver::getDrawable() (ahora podría devolver dos drawables)

• Ciertos wallobjects (otros drivers, thwomps, etc.)
• Driver del player character
• Lakitu

Por ahora todavía te deja en sitios raros y lleva velocidad mientras te tiene en el aire, ¿debería ser estático? ¿cuál es el mejor sitio al que devolver al jugador?

En el GP, al presentar el primer circuito (pista de arriba) el sonido de motor suena más alto que durante el resto de presentaciones de circuitos (pista de abajo).

Al pushear los dos a la vez se llama al init antes de que StateGPStandings le de tiempo a actualizar la clasificación

A veces tu followedPath no crece y solo tiene 1 elemento.

Observaciones:

• Parece que se añaden cosas, el problema está al quitarlas (siempre se puede hacer print debugging)
• A veces el bug se soluciona solo con parar el debugger, así que igual tiene que ver con el tiempo transcurrido

Añadir al fichero objects.txt de cada mapa:

• Información de los tres tipos de terreno: terreno normal, terreno lento, terreno outer
  • (opcional) Emplear esta información para las partículas de derrape (hay un documento "Sobre los circuitos" en Drive)
  • (opcional) Emplear esta información para generar nuevas partículas al conducir sobre hierba
  • (opcional) Emplear esta información para introducir nuevos SFX (mapas como ghost_valley_1 o derrapar/conducir por cualquier terreno de cualquier mapa)
  • Emplear esta información para el efecto EffectDrown (tanto para objetos como para jugadores que caigan al agua). Ver qué hacemos con la lava
• Información sobre el farvision de la IA (para 50cc, 100cc, 150cc), para que tome las curvas correctamente

En algunos casos (que no he sabido diferenciar) se quedan atascados NPCs en las paredes de bowser castle (1/4 circuito y despues de los primeros saltos)

Al cargar el juego se leen todos los controles de un fichero. Si este no existe o hay algun problema al leerlo, se sustituye por los defaults

Efecto esperado:
Cuando a un jugador (cualquiera) le cae un rayo, se debería frenar su aceleración y poner a 0 su velocidad angular, similar a un oil slick.

Efecto actual:
Al caer un rayo encima de un player character, este puede seguir girando/mantiene su velocidad angular, dando un menor efecto de ralentización y pérdida de control.

• Sale el trofeo, se "parte" por la mitad y se deja ver el mensaje de felicitaciones

StateCongratulations: Bandas negras, que ponga el puesto en el que has quedado/el resto de la clasificación etc.

StateRaceEnd: que se vea la clasificación a un lado conforme van terminando los corredores

Relacionada con #15

En game.cpp:86 se debería llamar a los métodos setWindowSize() o setScale() necesarios para que todos los elementos de la GUI sean escalados. Se puede asumir que el tamaño "normal" del juego es 512x448 píxeles y por tanto, la escala que se debería aplicar a todos los sprites es:

BACKGROUND_WIDTH = 512.0f
scale = window.getSize().x / BACKGROUND_WIDTH

Actualmente los saltos tienen algunos problemas:

• En bowser castle 1, para 50cc o 100cc es imposible saltar la lava sin caerse
• El salto para derrapar no funciona: al hacer un salto pequeño no se entra en modo derrape
• #11

Propongo los siguientes cambios para los saltos con rampa, para que sean más cómodos para el jugador:

Al saltar:

• El jugador no puede acelerar, pero tampoco se frena al no estar en contacto con el suelo
• El jugador sigue pudiendo girar. Ahora mismo la cámara se ve muy quieta en los saltos, girar sin acelerar puede dar algo de control en el aire al jugador, aunque igual el jugador gira demasiado al caer al suelo.

Ya diréis qué opináis sobre estos cambios

Versión del bug 2b2538a:

Por ejemplo, en la senda arcoiris. Si te caes mientras está presente el Lakitu de dirección errónea el sonido SFX no se detiene.

Para hacer un medio-fix, he puesto que a partir de ahora NO se puedan reproducir en bucle (loop = true) dos SFX iguales. Aún con todo, deber arreglar lo del pitido porque sigue sonando cuando Lakitu te devuelve con caña de pescar.

Solución: actualizar currentTime de StateRace desde StateRaceEnd

Backtrace from GDB:

Thread 1 received signal SIGSEGV, Segmentation fault.
0x66b86358 in sf::Image::getPixel(unsigned int, unsigned int) const ()
   from C:\mingw810\i686-810-posix-dwarf-rt_v6-rev0\mingw32\mingw\bin\sfml-graphics-2.dll
(gdb) bt
#0  0x66b86358 in sf::Image::getPixel(unsigned int, unsigned int) const ()
   from C:\mingw810\i686-810-posix-dwarf-rt_v6-rev0\mingw32\mingw\bin\sfml-graphics-2.dll
#1  0x0044370c in Map::sampleAsset (asset=..., sample=...) at map/map.h:81
#2  0x004197c0 in Map::sampleMap (this=0x4b3e40 <Map::instance>, sample=...) at map/map.cpp:6
#3  0x00419cf1 in Map::mode7 (this=0x4b3e40 <Map::instance>, position=..., angle=-0.0132949613,
    fovHalf=0.5, clipNear=0.000500000024, clipFar=0.0450000018, size=..., perspective=true)
    at map/map.cpp:49
#4  0x0041bd39 in Map::circuitTexture (player=
    std::shared_ptr<Driver> (use count 5, weak count 0) = {...}, circuitTexture=...) at map/map.cpp:384   
---Type <return> to continue, or q <return> to quit---
#5  0x00428e4b in StateRace::draw (this=0xf37d328, window=...) at states/race.cpp:130
#6  0x004395ed in Game::run (this=0xf5fc80) at game.cpp:69
#7  0x0043a130 in main () at main.cpp:5

Para cada carrera, mejorar la animación de la cámara (puede hacer giros, etc.)

Si sobra tiempo, es posible hacer alguna animación en la pantalla de inicio si el jugador no pulsa ninguna tecla

Explicación más detallada:

1. En la última vuelta, el player character es golpeado por un rayo, lo que lo coloca en el estado uncontrolled.
2. El player character termina la vuelta (y por tanto la carrera) con la propia inercia de su vehículo.
3. En el estado RaceEnd no se lleva cuenta de los estados de los jugadores, por lo que nunca se popea el estado Uncontrolled y todos los jugadores afectados por el rayo (incluido el PlayerCharacter) se quedan quietos.

Comprobar si la estrella:

• Hace inmune al jugador frente a terrenos lentos.
• Hace al jugador invencible contra choques con objetos.
• Tiene un efecto positivo para el jugador en caso de choque con otro.

• La estrella debe hacerte invencible frente a rayos

Esto puede dar efectos indeseados, como usar un champiñón mientras estás dando vueltas y que el champiñón acelere tu vehículo.

Por ahora el jugador gana monedas, pero no pierde monedas en ningún momento:

Debería perder monedas:

• Al entrar en modo HIT (chocarse con un oil slick, caparazones verdes, etc.)
• [?] Al entrar en modo SMASH (thwomps)
• [?] Al chocar contra otros jugadores (puedo encargarme yo)

• (opcional) Animación de GUI para ganar monedas y perder monedas

Parece que el tiempo de animación SMALL no está bien ajustado en función de la posición de los jugadores en el momento de recibir el rayo. Si es posible acceder a la posición desde driver.cpp:234, valdría con multiplicar SPEED_DOWN_DURATION por un escalar en función de la posición.

Nota: ¿por qué estos tiempos no coinciden?

animator.small(SPEED_DOWN_DURATION + SPEED_DOWN_DURATION);
pushStateEnd(DriverState::SPEED_DOWN, StateRace::currentTime + SPEED_DOWN_DURATION);

En cualquier resolución que no sea la normal el efecto FALLING escala al revés (el tamaño aumenta en vez de disminuir)

La altura no se ajusta a la de lakitu

Cuando termina una carrera y se entra en el estado RaceEnd:

• La posición de los corredores que ya han terminado la carrera no debería cambiar
• La posición de los corredores que NO han terminado la carrera puede seguir cambiando (mirar laps < 6)

Al correr dos carreras o más seguidas en GP, el SFX de semáforo deja de sonar

Para asegurar que la IA no se queda bloqueada durante mucho tiempo, p. ej. contra una pared; comprobar si no ha salido de un área (p. ej. de 3x3 tiles) durante un tiempo. Utilizar la misma primitiva que cuando se salgan del circuito (TP con/sin Lakitu).

• Lakitu puede dejar a algún NPC en la misma posición exacta de donde estaba bloqueado (añadir un random a la posición, aumentar el numero de tiles del gradiente que camina?)
• La variable onLakitu del driver puede no resetearse después de que te recoja Lakitu (y por tanto, no puedes caer)
• Si terminas la carrera y te caes al vacío en el estado RaceEnd, te quedas quieto
  • Imagino que Lakitu está ocupado con la animación de la bandera final
  • El personaje pasa de DriverControlType::PLAYER a DriverControlType::AI_GRADIENT
  • El personaje aparece en la posición donde lo dejaría Lakitu, en el suelo, pero no se puede mover
• Puedes empezar la carrera en el aire (si terminas la carrera anterior cayéndote? no me he fijado en los pasos para reproducirlo)
• Relacionado con lo anterior, después de jugar varias carreras es posible que el recolocamiento por lakitu funcione mal. Me ocurrió al jugar varias carreras seguidas, llegar a la senda arco íris y caerme, Lakitu siempre me devolvía a la posición inicial (intentaré reproducirlo con más detalles)

• Arreglar el hecho de que la IA lanza objetos al pulsar 1-7
• Fijar la cámara en algún corredor (añadir algún icono de "cámara fija"?)
• Modo "primera persona" (en lugar de seguir a un corredor en la distancia, seguirlo de cerca)

No tiene por qué ser diferente para cada pantalla, podría ser una música de fondo en bucle idéntica siempre que no se esté corriendo. Lo importante es que no se quede en silencio el juego.

Si al terminar la carrera hay un jugador que le está afectando el cambio de color por la estrella, este cambio de color se mantiene en pantallas posteriores (por ejemplo, en el estado congratulations donde se muestra a los corredores en el podio)

El color de la estrella (y bueno, creo que todos los efectos de items en general) deberían ser quitados cuando el player character termine cualquier carrera, para evitarnos este tipo de problemas.

Las comprobaciones de colisión solo se deberían hacer si los dos objetos (jugadores, items, etc.) están a una altura parecida (si se chocan A y B, abs(h_a - h_b) < valor_pequeño)

Ahora los objetos los usa con un random, se puede añadir una puntuación de "cómo de bueno sería usar este objeto ahora mismo" y hacer que sea más probable usarlo en momentos buenos

@JaviBite

• Al usar un champiñón sale un efecto de velocidad. Si te caes al vacío con el champiñón en efecto, el efecto perdura mientras estás quieto.
• El efecto de velocidad no parece una textura del juego al estar poco "pixelado".
• Si terminas una carrera con un objeto y luego empiezas otra, no se resetea la GUI de objetos. Sale que tienes ese objeto en la GUI, pero no lo puedes usar porque no lo tienes realmente. (mirar el resto de la GUI también)
• Si coges una caja de objetos y usas el objeto al momento, la animación de "objeto aleatorio" de la GUI sigue reproduciéndose (podemos hacer dos cosas: o evitar que el usuario use objetos mientras está en "objeto aleatorio" o parar en ese momento la animación de la GUI)

@diegoroyo

• Los jugadores en el minimapa no tienen el origen bien puesto cuando estan en modo SMALL
• Las sombras de los jugadores pequeños no se ven pequeñas en el circuito Map::getDriverDrawables(...)

A veces fixedUpdate se ejecuta más de una vez por cada handleTryPop, haciendo que haya 2 trypops en espera

Por ahora la IA es incapaz de dar el salto, aún cuando toma la rampa de forma casi recta. Cualquiera de estas cosas vale:

• Hacer que el zipper de más velocidad
• Hacer que dar un salto con rampa te de mas velocidad

En la pantalla de START si se hace clic en ESCAPE se sale directamente. Quizás sea mejor mostrar un mensaje de YES / NO para "Exit game?"

También se aprecia que apareces fuera del mapa ya que ahora no se para al caer y se guarda como posición valida en el "log de posiciones" [@vmbatlle]

En la línea driver.cpp:413 se invoca a la animación de DriverAnimator::fall() pero no es visible a causa de la relocalización instantánea de Lakitu. Se deben también retesar los efectos adversos y power-ups, tanto del estado del jugador como del sprite.

Quizás se podría hacer que el animator haga al jugador pequeño, luego se pase a fundido a negro la pantalla del jugador (parte superior), en ese momento se llama a Driver::relocateToNearestGoodPosition() y finalmente se quita el fundido a negro.

Viendo que hay varios problemas al llamar directamente al sistema de audio desde cualquier sitio (issues #1 #2 #12 #13), propongo lo siguiente:

• Modificar Audio::play(music) para que ponga las canciones en bucle correctamente
• Añadir una capa por encima de Audio::play(music) para la música (situada donde sea), a la que le puedas indicar donde estas (en carrera, en el menu 1, menu 2, etc.) y suene la música correspondiente
• Esta capa ha de pedirle a Map cual es la música del circuito acutal (tanto la normal como la de ir rápido)
• Para el caso de la estrella, esta capa se encarga de pausar la música actual, hacer sonar la estrella, y cuando termine vuelve a sonar la música anterior. Si suena más de una estrella a la vez símplemente se resetea (por ello, la estrella debe ser música y no SFX)

No se si me dejo algún caso especial, ya diréis qué os parece y cómo nos encargamos de esto

Evitaría los choques que ocurren al principio

Al dar estos problemas la música de la estrella, sugiero añadir alguna lógica a map para que sepa qué música suena en cada momento (estrella/normal/last lap) y que se pueda cambiar entre ellas, pero solo pueda sonar una a la vez.