games/raycaster
1
0
Fork 0
Code Issues 30 Pull requests Projects Releases 2 Packages Wiki Activity Actions

SpatialMap::get and neighbors need to account for collision and multiple entities #76

New issue
Closed
opened 2025-08-01 17:15:16 +00:00 by zedshaw · 1 comment
zedshaw commented 2025-08-01 17:15:16 +00:00 (Migrated from lcthw.dev)
Copy link

Right now these functions just return the first thing found, but I need to figure out if that's what is needed. It could be fine, I could only return things with collision, or make it an option to return everything or collision only.

Right now these functions just return the first thing found, but I need to figure out if that's what is needed. It could be fine, I could only return things with collision, or make it an option to return everything or collision only.
zedshaw commented 2025-08-03 03:37:01 +00:00 (Migrated from lcthw.dev)
Copy link

Looks like SpatialMap::get is only used in main_ui.cpp::camera_aim and that could go away with a refactor of where the camera aim is maintained.

Neighbors is only used to get collision neighbors, so it's fine too.

Looks like SpatialMap::get is only used in main_ui.cpp::camera_aim and that could go away with a refactor of where the camera aim is maintained. Neighbors is only used to get collision neighbors, so it's fine too.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
version-0.2
version-0.1
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: games/raycaster#76
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?