Skip to content

feat: Level move #7

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.

Sign up for GitHub

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

Merged
Loule95450 merged 6 commits into from
Oct 19, 2024
Merged

feat: Level move #7

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
28d55cc
feature: First try to add level move in Unity
Thomlam Oct 17, 2024
24767bf
feat: fix Arrow, upgrade door system
Thomlam Oct 17, 2024
7a4935d
feat: Fix multiple problem
Thomlam Oct 17, 2024
74ac62c
fix: fix door teleport with disable player public selected
Thomlam Oct 18, 2024
ab113cf
feat: Addition of traps using sockets (#6)
AyakorK Oct 18, 2024
3817606
fix(level_move): Replace hard-coded values ​​with variables for bette…
Loule95450 Oct 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -290,4 +290,8 @@ $RECYCLE.BIN/
# Windows shortcuts
*.lnk


# Environment file
.env

# End of https://www.toptal.com/developers/gitignore/api/macos,windows,linux,unity,intellij+all,clion+all
2 changes: 1 addition & 1 deletion Assets/Prefabs/Player.prefab
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ Transform:
m_GameObject: {fileID: 1463738777561398421}
serializedVersion: 2
m_LocalRotation: {x: 0, y: 0, z: 0, w: 1}
m_LocalPosition: {x: 4.5, y: 0.5, z: 0}
m_LocalPosition: {x: 4.6, y: 0.2999997, z: 0}
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot Oct 19, 2024

Choose a reason for hiding this comment

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

💡 Codebase verification

Précision des coordonnées Y des prefabs

Plusieurs prefabs possèdent des coordonnées Y avec une précision supérieure à deux décimales :

  • Player.prefab : y = 0.2999997
  • CorridorX7.prefab : y = 2.8000002
  • Room.prefab : y = 2.8000002

Points de préoccupation :

  1. Impact sur le gameplay : Cette précision peut affecter le positionnement et les interactions en jeu de manière inattendue.
  2. Intentionnalité : Vérifiez si ces valeurs précises sont intentionnelles ou le résultat d'une erreur de sérialisation.

Il est recommandé de vérifier manuellement si ces modifications n'introduisent pas de problèmes de collision ou de positionnement et si elles sont cohérentes avec les autres éléments du jeu.

🔗 Analysis chain

Vérifiez l'impact du changement de position du joueur

La position locale du joueur a été modifiée :

  • Coordonnée x : de 4.5 à 4.6
  • Coordonnée y : de 0.5 à 0.2999997

Points à considérer :

  1. Ce changement de position pourrait-il affecter le gameplay ou les interactions avec d'autres éléments du jeu ?
  2. La valeur de y (0.2999997) semble avoir une précision inhabituelle. Est-ce intentionnel ou le résultat d'un arrondi imprévu ?

Veuillez vérifier si ces modifications sont cohérentes avec les autres éléments du jeu et si elles n'introduisent pas de problèmes de collision ou de positionnement.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Rechercher d'autres occurrences de coordonnées similaires dans les prefabs

echo "Recherche de coordonnées similaires dans d'autres prefabs :"
grep -r --include="*.prefab" "m_LocalPosition:" Assets/Prefabs

Length of output: 3395

Script:

#!/bin/bash
# Description: Rechercher des coordonnées y avec une précision supérieure à deux décimales dans les prefabs

echo "Recherche des coordonnées y avec une précision supérieure à deux décimales dans les prefabs :"
grep -r --include="*.prefab" -E 'm_LocalPosition:\s*\{x: [^,]+, y: [0-9]+\.[0-9]{3,}, z: [^}]+\}' Assets/Prefabs

Length of output: 596

m_LocalScale: {x: 5, y: 5, z: 5}
m_ConstrainProportionsScale: 0
m_Children: []
Expand Down
8 changes: 8 additions & 0 deletions Assets/Prefabs/RoomSystem/Corridor.meta
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading