Repository navigation
Conversation
….5.0 seventh encounter
….5.0 eight encounter
….5.0 nineth encounter
….5.0 tenth encounter
….5.0 twelveth encounter
…NotificationType::animatedObjectSetWalkPath
…rgin ( five percent) not exactly last point
…frag Channel0.png
…ght closed doors) fixed
… around the prison
New TileVisual types for all the rooms, and tileset.cfg modifed to allow proper config of neighbouring tiles, for example for TileVisual::hatcheryRoom the code shall compute the neighbouring index e.g. 1010 and give an appropriate tile
Minor bug not allowing the destruction of rooms also fixed
Missing room meshes added, treasury tiles should now display ok
TileVisuals can now impose setting the position ofa creature diffrent than z=0
…t from visual and index fixed
….e. the tile is unreachable the usual way
Now all ground tiles meshes should use Room.mesh
…k around celler or fight there when they are being dropped there
CMakeList.txt modfied due to paroj fixtures, all tests are now passed
tile info for debugging information -- seat, position, visualTileType, tileType, floodFillValue, floodfill bug introduced at 5409832
The loadLevel message rebuilt the map with createNewMap, which only recreates the tiles and trusts whoever started the connection to have cleared everything else. Any path that broke that promise - pressing Launch a second time from the same menu screen, or reconnecting after a failed start - piled a second copy of every creature class, seat and weapon onto the old ones. The editor then aborted while filling its Creatures menu, because CEGUI refuses two menu items with the same name: CEGUI::AlreadyExistsException : Failed to add Element named: Adventurer to element at: EDITORGUI/Menubar/Creatures/PopupMenu4 Clear the map in the handler itself, the same way the newMap message already does. Also make the editor load menu stop when the server could not be started instead of connecting to whatever server was already running; every other menu already returned there.
Let enemy workers claim room tiles (DK2-style), behind a config switch
"Exit OpenDungeons" did not fit the button well, and reads even longer once the game is called OpenDungeonsPlus. "Exit to System" pairs with "Quit to Menu" above it and says where the player ends up without assuming they know what an OS is. The tooltip still spells out that it leaves to the desktop. Claude-Session: https://claude.ai/code/session_01JiKh7f4nenYBTjhJfAGa3y
The editor's P shortcut refused to ask about any tile the client did not believe looked like a wave portal, and that belief proved wrong: the check compared against the tile visual, and the window then never opened at all, with the "Point at a wave portal" hint shown even over a portal. The client is only ever told what its tiles look like, never which room covers them, so it has no reliable way to tell. The server does, and it is also the only side that knows the waves. So the editor now always asks, and the server answers with the waves, or with an empty room name when the tile holds no wave portal, in which case the editor shows the hint. Claude-Session: https://claude.ai/code/session_01JiKh7f4nenYBTjhJfAGa3y
…editor-improvements
…ttons Say what the quit buttons do, and add one that actually exits
The editor offers two rooms of its own, a dungeon temple and a portal, so a wave portal could only ever be edited in a level that already had one. Add a third button for it, next to the portal, drawn with the portal's glyph in red so the two cannot be confused. Claude-Session: https://claude.ai/code/session_01JiKh7f4nenYBTjhJfAGa3y
…ments Editor improvements: creature levels, help window, wave portal editing and more
Clear the client game map before loading a level into it
Embed per-monitor DPI awareness and maintain the verified OGRE 13.6.5 renderer patch required for context-safe window replacement, runtime GL options and correct fullscreen client geometry. Validation: clean Release build, DPI and 3440x1440 fullscreen probes, three GL3Plus variants and the patch reverse check passed; version 0.7.1 remains unchanged because this is unreleased work and the project has no changelog.
Apply display, input, audio, gameplay, minimap, shadow and negotiated nickname changes to the running session while preserving rollback for failed window or input transitions and keeping the 3440x1440 navigation visible. Validation: clean Release and Debug builds, packet tests, DPI, layout and GL3Plus probes passed; the user confirmed the reported resolution, fullscreen, cursor, flicker and navigation failures are fixed. Version 0.7.1 remains unchanged because this is unreleased work and the project has no changelog.
The spells tab put its ten buttons in a single row of 60 pixel squares running out to x=700. The pane they live in is the window width minus the 200 pixels of minimap, so at the 800 wide minimum window it is 600 pixels across, and the last two spells, Weakness and the Eye of Evil, hung past its right edge where they could be neither seen nor clicked. Only a window at least 900 wide showed the whole row, and nothing said so. The rooms tab solved the same problem long ago: two rows of 40 pixel buttons. The spells tab now uses the same shape, five spells to a row, ending at x=300 with room to spare at any size the game accepts. The cooldown bar inside each button now covers exactly its button too, instead of overhanging 20 pixels on both sides, which with narrower buttons would have bled half way across the neighbours. The rooms tab had the tail of the same bug: the temple and portal buttons the editor uses sat at x=570 to 690, past the same edge. They join the end of the second row instead. (cherry picked from commit 10db1cb)
Replace the auto declarations introduced by this branch with explicit types, as requested in issue tomluchowski#42 and required by CLAUDE.md. No behavioural change.
Rokk001
force-pushed
the
pr/creature-population-complete
branch
from
September 27, 2026 17:16
1da6332 to
2c1ddf5
Compare
This was referenced Sep 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a live population panel with illustrated creature portraits, total/job/fighting/mood views, eligible worker counts, paging and pickup or camera-focus controls. Level shortcuts select the highest or lowest eligible creature; held creatures are excluded from pickup counts. The query control reuses the existing creature statistics window.
Includes the optional activity/mood protocol negotiation, owner-only population snapshots and the selection styling and panel height required for this interface. The portrait assets cover all 33 configured mesh identities, with model rendering retained for custom creatures without artwork.
Depends on #53 and #88. GitHub's comparison includes those prerequisites until they merge; review the focused contribution separately. Merge prerequisites first and recheck the remaining diff. This contributes creature selection controls relevant to #6; it does not resolve that broader issue in full. Supersedes the closed fragmented population/portrait submissions, which should not be reopened separately.
Validation: Windows x64 Release integration compilation and runtime preparation passed; protocol/mood/activity checks passed 1,117 assertions and population aggregation passed 77. The actual panel and CEGUI scaling/interaction probe passed after including its direct creature-definition header, selection property and required panel height. The user accepted portraits and the picker-count correction in the complete fork. Network endpoints in focused probes are test doubles; live multiplayer and other-platform runtime verification are not claimed.