#3746 closed defect (fixed)
Fish on land — at Version 6
Reported by: | Odelpasso | Owned by: | elexis |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 21 |
Component: | Maps | Keywords: | simple |
Cc: | Patch: | Phab:D1729, Phab:D4315 |
Description (last modified by )
Hi,
There is a problem with the fish location when i want to fish. In fact, the fish is not in the sea but on the ground. Like you can see on the screenshot.
Odelpasso
Related ticket:
- #5797 (5/Aug/20)
Change History (7)
comment:1 by , 8 years ago
Component: | Art & Animation → Maps |
---|
by , 8 years ago
Attachment: | commands.txt added |
---|
Here an a19 replay of the random map script "Archipelago" where fish is on land in multiple places. Notice that fish and trees are the last entities to be placed and fish avoids the class clLand
. By changing the perspective to gaia, one can press the Alt button and doubleclick on a single fish spot to select all fish on a map.
comment:2 by , 8 years ago
The usage of tile classes - even more their names - are entirely up to the map designer.
There are several ways to ensure fish does not wind up at land.
IMO maps should be carefully tested before adding them. The designer is likely most reachable at this point to fix bugs that may be found.
In the long run it would be really convinient if the engine would simply remove invalid placed entities but I don't see that happen (and in some cases it's actually hard to determine what "invalid" means).
This is somehow related to collision detection (see https://wildfiregames.com/forum/index.php?/topic/16242-random-maps-suggestions-bug-reports-and-ideas-about-random-maps-here/&do=findComment&comment=297569). A passability map and further placement checks e.g. for buildings would be nice to have. That would mean much work though to simulate the placement restrictions in RMGen since none of those are in shared libraries.
Still a "detect possible unwanted placement" function or so would be nice to have at least for debug purposes - or as a mean of cleanup, wich on the other hand would encourage sloppy scripting ... so maybe not.
comment:3 by , 8 years ago
Keywords: | simple added |
---|---|
Summary: | Wrong location for the fish → Fish on land |
The task for this ticket is to ensure that fish isn't placed on land.
Confirmed maps: archipelago (see replay above) Probably affected maps: Unknown, Unknown Nomad, Islands, Migration.
It might be sufficient to mess with the tileclasses and distances. Perhaps the clLand
is painted incorrectly, for a wrong elevation.
comment:5 by , 8 years ago
Milestone: | Backlog → Alpha 21 |
---|
This is the only random map script affected by this bug. Every other map either places the clWater
/ clLand
class correctly (i.e. for exactly the tiles that are water/land) or also avoid the clPlayer
class which on those maps stands for the initial player base terrain.
This was really easy to find out by just placing a thousand mines on the map wherever clLand
would be.
comment:6 by , 3 years ago
Description: | modified (diff) |
---|---|
Patch: | → Phab:D1729, Phab:D4315 |
Happens on many maps. Which map is that on the screenshot?