pyrex/fledgling
1
0
Fork 0
Code Issues 17 Pull Requests Actions Packages Projects Releases Wiki Activity

Ceremonial PR: fix map gen #39

Merged
pyrex merged 14 commits from fix-mapgen into main 2025-02-23 05:41:20 +00:00
Conversation 0 Commits 14 Files Changed 4
+194 -8
pyrex commented 2025-02-23 05:41:03 +00:00
Owner
Copy Link
Copy Source

Kistaro found bugs! Fix them.

Kistaro found bugs! Fix them.
pyrex added 13 commits 2025-02-23 05:41:03 +00:00
improve errors 8bf7f0f151
merge state debug dumper 234a42b1e3
use detailed debugging in map gen 898a93d8e5
more distinct wall chars ba151a76fd
not all sealed walls are walls. okay 4117608073
handle negative region IDs 764d1e4892 
also catches some missed semis
stop using the "dark shade" character for standard walls b302538ade 
now uses inverse bullet for sealed walls and full block otherwise
also show final result with region numbers e2aa4a3ee7
fix fencepost error when merging regions 025b1c9333
map connectedness checker (floodfill) 0b7d447c5b
check for connectedness in mapgen ec2e21c712
add commented-out cheat and test buttons b4aa9329ad 
looks like mapgen is now fixed. here are the buttons I used to test it
autoformat code 70c2fcc491
pyrex added 1 commit 2025-02-23 05:41:09 +00:00
Merge branch 'main' into fix-mapgen 3b1c0af916
pyrex merged commit 1ffc0518b2 into main 2025-02-23 05:41:20 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
art
This is primarily a change to the pixel assets
 core
This is needed for the game to function on a basic level.
 good first issue
Kistaro! Work on these.
Importance
Important
This should be done before end of February
Importance
Postponed
This issue doesn't need to be done right now.
polish
This makes the game feel nice to play, but it's not essential
 refactor
The code is bad: fix it!
 speculative
"I don't know if this is a good idea yet."
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
bhijn (Bhijn) kistaro (Kistaro Windrider) pyrex (Pyrex)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pyrex/fledgling#39