Angband.oook.cz
Angband.oook.cz
AboutVariantsLadderForumCompetitionComicScreenshotsFunniesLinks

Go Back   Angband Forums > Angband > Vanilla

Reply
 
Thread Tools Display Modes
Old January 6, 2010, 17:08   #1
Tatami
Apprentice
 
Join Date: Oct 2009
Posts: 59
Tatami is on a distinguished road
deep descent bug 3.1.2 dev r1886

The scrolls only take you down to level 2.

This is my punishment for using them to go down to and past 127 to fetch the One before killing M I assume.
Tatami is offline   Reply With Quote
Old January 6, 2010, 17:12   #2
d_m
Angband Devteam member
 
d_m's Avatar
 
Join Date: Aug 2008
Location: Philadelphia, PA, USA
Age: 39
Posts: 1,516
d_m is on a distinguished road
Did that actually work previously?
__________________
linux->xterm->screen->pmacs
d_m is offline   Reply With Quote
Old January 6, 2010, 17:18   #3
Marble Dice
Swordsman
 
Join Date: Jun 2008
Location: Columbia, MO. USA
Posts: 405
Marble Dice is on a distinguished road
This is an intentional change, deep descent always takes you 2 floors below your current position, without skipping quest levels.
Marble Dice is offline   Reply With Quote
Old January 6, 2010, 17:37   #4
Tatami
Apprentice
 
Join Date: Oct 2009
Posts: 59
Tatami is on a distinguished road
Quote:
Originally Posted by d_m View Post
Did that actually work previously?
Aye, I believe it was r1866. With recall sometimes putting you on down stairs you could in fact go deeper than level 127.
Tatami is offline   Reply With Quote
Old January 8, 2010, 20:01   #5
Nightmarjoo
Adept
 
Nightmarjoo's Avatar
 
Join Date: May 2007
Location: Ohio, US
Posts: 104
Nightmarjoo is on a distinguished road
Send a message via AIM to Nightmarjoo Send a message via MSN to Nightmarjoo
Couldn't it take you deeper before? Why the change?
__________________
My first winner! http://angband.oook.cz/ladder-show.php?id=9326 Link, the Kobold Warrior!

My second winner! http://angband.oook.cz/ladder-show.php?id=9369 Cailet, the Hobbit Mage!

Damned be those who use High Elves, for they are the race of the weak!
Nightmarjoo is offline   Reply With Quote
Old January 8, 2010, 21:15   #6
Marble Dice
Swordsman
 
Join Date: Jun 2008
Location: Columbia, MO. USA
Posts: 405
Marble Dice is on a distinguished road
Quote:
Originally Posted by Nightmarjoo View Post
Couldn't it take you deeper before? Why the change?
The description used to read "two or three floors" even though it only took you 2 floors below the maximum. I'm not sure if it ever really did 1+1d2 or if the description just said it did.
Marble Dice is offline   Reply With Quote
Old January 10, 2010, 00:51   #7
d_m
Angband Devteam member
 
d_m's Avatar
 
Join Date: Aug 2008
Location: Philadelphia, PA, USA
Age: 39
Posts: 1,516
d_m is on a distinguished road
So, I do think that !DD should work as it does now, but use max-depth instead of current-depth to figure out what depth you should go to.

At one point I think I said something like "should work like teleport level but always takes you down to depth + 2" and I may have been ambiguous, but I think it should be maxdepth + 2.

Does that seem crazy? It's a trivial patch to change it, but since it was just changed I don't want to change it back without a discussion.
__________________
linux->xterm->screen->pmacs
d_m is offline   Reply With Quote
Old January 10, 2010, 01:35   #8
Marble Dice
Swordsman
 
Join Date: Jun 2008
Location: Columbia, MO. USA
Posts: 405
Marble Dice is on a distinguished road
Quote:
Originally Posted by d_m View Post
So, I do think that !DD should work as it does now, but use max-depth instead of current-depth to figure out what depth you should go to.
Fine with me either way, but the original ticket as Takkaria phrased it was "deep descent has wrong behavior at non-recall depth," and the ticket description specifically calls out the max_depth+2 behavior.
Marble Dice is offline   Reply With Quote
Old January 10, 2010, 09:06   #9
Magnate
Angband Devteam member
 
Join Date: May 2007
Location: London, UK
Posts: 5,057
Magnate is on a distinguished road
Send a message via MSN to Magnate Send a message via Yahoo to Magnate
Quote:
Originally Posted by Marble Dice View Post
Fine with me either way, but the original ticket as Takkaria phrased it was "deep descent has wrong behavior at non-recall depth," and the ticket description specifically calls out the max_depth+2 behavior.
Looking back, I'm not really sure why Takk thought that was wrong. The issue we were originally concerned with was reading ?DD in town, IIRC.

But whatever, he decided quite firmly that ?DD is the cousin of ?tlev and not of ?WoR, so it makes sense that maxdepth doesn't come into it.
Magnate is offline   Reply With Quote
Old January 10, 2010, 15:43   #10
d_m
Angband Devteam member
 
d_m's Avatar
 
Join Date: Aug 2008
Location: Philadelphia, PA, USA
Age: 39
Posts: 1,516
d_m is on a distinguished road
Huh. I had thought the issue was whether to set (or cancel) the recall flag, and whether there should be a delay (like recall) when in town. I think that bug was opened after the discussion on IRC, but I don't really remember.

Anyway, on some level the current behavior is fine--I will just have to get used to reading WoR before DD.
__________________
linux->xterm->screen->pmacs
d_m is offline   Reply With Quote
Reply


Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)
 
Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
r1626 deep descent bug fizzix Development 2 October 31, 2009 21:14
Dev Bug rdermyer Vanilla 13 December 24, 2008 02:49
Fear Bug In Dev Version LordArcher Vanilla 2 November 30, 2008 20:04
[Dev] I've located the cause of this major bug with fear. Irashtar Vanilla 0 August 31, 2008 11:04
[Dev] major Fear bug. Irashtar Vanilla 1 August 31, 2008 03:03


All times are GMT +1. The time now is 18:07.


Powered by vBulletin® Version 3.8.11
Copyright ©2000 - 2020, vBulletin Solutions Inc.