Viewing Issue Simple Details Jump to Notes ] Wiki ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0009818 [DCSS] Bug Report minor always 2015-06-26 04:56 2015-11-10 17:50
Reporter josh View Status public  
Assigned To wheals
Priority normal Resolution done  
Status resolved   Product Branch 0.17 ancient branch
Summary 0009818: Weapon attack delay description does not take speed brand into account
Description
A +3 dagger of speed.

A double-edged fighting knife with a sharp point.

Base accuracy: +6  Base damage: 4  Base attack delay: 1.0
This weapon's minimum attack delay of 0.5 is reached at skill level 10.
 (Your skill: 0.0)


Ideally, the display of the attack delay and minimum attack delay should take the speed brand into account.
Additional Information
Tags No tags attached.
Attached Files

- Relationships

-  Notes
(0029237)
chequers (reporter)
2015-06-26 05:45

The problem with taking speed brand into account is that mindelay for a weapon is not necessarily equivalent to 100% of attacks taking the same amount of time.

Two potential solutions:

a) take speed brand into account and show mindelay to two decimal places (problem: not obvious that attacks are randomly rounded to the nearest 0,1)

b) reword speed brand line to better make explicit that it reduces your attack delay by x%.
(0029238)
josh (reporter)
2015-06-26 06:34

Option a seems preferable, to show the actual value. Another reason to explicitly show the correct mindelay: making it clear whether it affects how much weapon skill you need to reach that mindelay. (For that matter, it would be nice to display the *current* effective delay based on your current weapon skill.)

One other option:
c) change speed to not randomly round to the nearest 0.1, and use deterministic and consistent values instead. That might be significantly harder, though.
(0029239)
PleasingFungus (administrator)
2015-06-26 07:08

(c) is pretty impossible; the game simulation only runs to a precision of 0.1.
(0029769)
wheals (administrator)
2015-11-10 17:50

Fixed in https://github.com/crawl/crawl/commit/2733cf4fda70. [^]

- Issue History
Date Modified Username Field Change
2015-06-26 04:56 josh New Issue
2015-06-26 05:45 chequers Note Added: 0029237
2015-06-26 06:34 josh Note Added: 0029238
2015-06-26 07:08 PleasingFungus Note Added: 0029239
2015-11-10 17:50 wheals Note Added: 0029769
2015-11-10 17:50 wheals Status new => resolved
2015-11-10 17:50 wheals Fixed in Branch => 0.18 development branch
2015-11-10 17:50 wheals Resolution open => done
2015-11-10 17:50 wheals Assigned To => wheals


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