Jonathan
/
Babushka
2
0
Fork 0
Code Pull Requests 3 Releases Activity
Labels Milestones

feature/detection_cross #21

Merged
kziolkowski merged 33 commits from feature/detection_cross into develop 2 months ago
Conversation 0 Commits 33 Files Changed 82
kziolkowski commented 3 months ago
Owner

Reworked interaction scheme to

  • work with areas instead of bodies
  • take player look direction into account by moving a detector shape in accordance with the input
  • make interaction work smoother with smaller colliders

bonus:

  • Adjusted Collision shapes and positions in farming scenes and indoor scene
  • Added Signals to the detector script that trigger whenever an interactable object has entered / exited for future use (sounds, vfx etc.)
Reworked interaction scheme to - work with areas instead of bodies - take player look direction into account by moving a detector shape in accordance with the input - make interaction work smoother with smaller colliders bonus: - Adjusted Collision shapes and positions in farming scenes and indoor scene - Added Signals to the detector script that trigger whenever an interactable object has entered / exited for future use (sounds, vfx etc.)
Screenshot from 2025-10-24 16-03-12.png
826 KiB
Screenshot from 2025-10-24 16-03-23.png
832 KiB
kziolkowski added 53 commits 3 months ago
09b2baab0a Merge remote-tracking branch 'origin/Animations' into feature/animations_update
8f097de476 📝🔥🔨
Jonathan was assigned by kziolkowski 3 months ago
kziolkowski changed title from feature/detection_cross to WIP: feature/detection_cross 3 months ago
kziolkowski changed title from WIP: feature/detection_cross to feature/detection_cross 3 months ago
kziolkowski requested review from Jonathan 3 months ago
Jonathan was unassigned by kziolkowski 3 months ago
kziolkowski self-assigned this 3 months ago
Jonathan requested changes 3 months ago
Jonathan left a comment
Owner

Der Branch baut aktuell auf feature/bugfixing_kathi_partIII auf. Kannst du bitte einmal auf den aktuellen develop rebasen?

Der Branch baut aktuell auf `feature/bugfixing_kathi_partIII` auf. Kannst du bitte einmal auf den aktuellen `develop` rebasen?
kziolkowski added 1 commit 3 months ago
kziolkowski added 2 commits 3 months ago
Jonathan force-pushed feature/detection_cross from 90b8dda16a to 32ac007256 3 months ago
Jonathan requested changes 3 months ago
Jonathan left a comment
Owner

  • Vesnas Namensschild sieht nicht richtig aus
    image

  • Nachdem man zum radischen Gaten geht, funktioniert viel nicht mehr. Z.b. gießen

  • Das Tor ist nicht ausgewählt, wenn man direkt davorsteht
    image

- [x] Vesnas Namensschild sieht nicht richtig aus ![image](/attachments/05132ad6-9053-4ea8-b1a4-82d2b2fd0d0d) - [ ] Nachdem man zum radischen Gaten geht, funktioniert viel nicht mehr. Z.b. gießen - [x] Das Tor ist nicht ausgewählt, wenn man direkt davorsteht ![image](/attachments/fb34f1f1-0996-416d-bc85-cbd8038de731)
scenes/Babushka_scene_farm_outside_2d.tscn
[sub_resource type="FastNoiseLite" id="FastNoiseLite_ubx8m"]
Jonathan commented 3 months ago
Owner

Beim Öffnen dieser Szene kommt ein Error. Er findet die Datei res://art/test_tomatos.png nicht.

Beim Öffnen dieser Szene kommt ein Error. Er findet die Datei `res://art/test_tomatos.png` nicht.
kziolkowski commented 2 months ago
Poster
Owner

Zwei der Bugs konnte ich einfach auflösen, das mit der Beet-Scene wird im Zuge eines größeren Farming-Reworks auf dem neuen Branch gefixt.
Ich hab jetzt erstmal alle größeren Exceptions, die ich finden und reproduzieren konnte gefixt und würde diesen Branch ansonsten erstmal so mergen, damit ich dann weitermachen kann. Ich hoffe, das ist okay 😄

Zwei der Bugs konnte ich einfach auflösen, das mit der Beet-Scene wird im Zuge eines größeren Farming-Reworks auf dem neuen Branch gefixt. Ich hab jetzt erstmal alle größeren Exceptions, die ich finden und reproduzieren konnte gefixt und würde diesen Branch ansonsten erstmal so mergen, damit ich dann weitermachen kann. Ich hoffe, das ist okay 😄
kziolkowski added 18 commits 2 months ago
8f097de476 📝🔥🔨
kziolkowski added 1 commit 2 months ago
1f13f3ea51 Merge branch 'develop' into feature/detection_cross
kziolkowski added 3 commits 2 months ago
kziolkowski merged commit 759933c1cd into develop 2 months ago

Reviewers

Jonathan requested changes 3 months ago
The pull request has been merged as 759933c1cd.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
Jonathan
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
kziolkowski
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Jonathan/Babushka#21
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'feature/detection_cross'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes