Viewing Issue Simple Details Jump to Notes ] Wiki ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0008600 [DCSS] Bug Report minor always 2014-05-27 02:38 2014-05-31 23:13
Reporter crate View Status public  
Assigned To
Priority normal Resolution open  
Status new   Product Branch 0.15 ancient branch
Summary 0008600: "travel_avoid_terrain = shallow water" behaves strangely with octopode
Description http://crawl.akrasiac.org/saves/crate-crawl-git-5bce1962da-140526-2032.tar.bz2 [^]

Sometimes in this save, autotravel/autoexplore will gladly take me into shallow/deep water. Sometimes it will not, and I don't know why. See: top-right of the floor where my character is. Also, press G> and then check the top section of the floor below. There was also that one vampire island vault somewhere in Depths, and autoexplore gladly explored that.

Clearly autotravel goes through water sometimes.

I don't know what the desired behaviour is here (possibly just make "travel_avoid_terrain = shallow water" never affect op/mf, though mf has some corner-cases where you still want to avoid water (like Lear's Hauberk)) but it shouldn't be incomprehensible and confusing like this.
Additional Information
Tags No tags attached.
Attached Files

- Relationships

-  Notes
(0026274)
crate (reporter)
2014-05-27 02:44

edit: it seems that it takes me through deep water but not shallow water, mostly? I'm not actually entirely sure. Still the behaviour feels like a bug even if it isn't.
(0026348)
neil (administrator)
2014-05-31 19:29

The docs for the option do mention the merfolk/octopode/flying situation, but don't make it clear that just "shallow water" will have bizarre effects.

We could make "shallow water" imply "deep water" here; but I think at that point it might be better to have a new option that doesn't pretend to be a list option, and doesn't pretend to affect other terrain types.

   travel_avoid_water = yes | no | deep

Maybe it makes sense to understand the old option for compatibility purposes, though.
(0026355)
neil (administrator)
2014-05-31 23:13

As you pointed out, maybe it would also be good to have a way, other than lua, to specify "this doesn't apply to swimmers" and/or "this doesn't apply to permaflyers".

- Issue History
Date Modified Username Field Change
2014-05-27 02:38 crate New Issue
2014-05-27 02:44 crate Note Added: 0026274
2014-05-31 19:29 neil Note Added: 0026348
2014-05-31 23:13 neil Note Added: 0026355


Mantis 1.1.8[^]
Copyright © 2000 - 2009 Mantis Group
Powered by Mantis Bugtracker