Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unique death normalization improvements #43

Open
bhaak opened this issue Aug 6, 2014 · 15 comments
Open

unique death normalization improvements #43

bhaak opened this issue Aug 6, 2014 · 15 comments
Labels

Comments

@bhaak
Copy link
Collaborator

bhaak commented Aug 6, 2014

Look at the proposed unique death normalization from these two posts:
http://mathematicalcoffee.blogspot.com.au/2014/07/junethack-2014-is-over.html http://74.135.83.0:8018/nethack-stuff/junethack-unique-deaths-comments-2014-jonadab.html

@bhaak bhaak added the design label Aug 6, 2014
@tsadok
Copy link

tsadok commented Oct 7, 2014

My IP address changed, so that second URL should now be http://jonadab.jumpingcrab.com:8018/nethack-stuff/junethack-unique-deaths-comments-2014-jonadab.html

@mathematicalcoffee
Copy link
Contributor

FYI #48 pull request implements some of the widely-agreed-upon death normalisations.

@FredrIQ
Copy link

FredrIQ commented Apr 9, 2016

Minions of deities can be polymorphed, this is not taken into account since the list is hardcoded. I suggest hardcoding deity names instead, which would also catch aligned/high priests.

@RojjaCebolla
Copy link
Contributor

Some death messages seem to lack a right paren, perhaps it should be made optional in the relevant regex?

slipped while mounting a monster (a saddled speedhorse
killed by a monster (Tasmanian devil

@actual-nh
Copy link
Contributor

Are things like "moderately wounded" ("killed by a moderately wounded goblin"), "seriously wounded", and "uninjured" supposed to be removed for unique deaths?

@bhaak
Copy link
Collaborator Author

bhaak commented Jun 2, 2021

Vanilla has removed the wounded state from the death message for the time being and I think every variant that had it before has upgraded already.
So there's is currently no pressing issue to look at that. But I guess it should be removed if it ever comes back. Otherwise you just have uncesseray grinding.

@actual-nh
Copy link
Contributor

Vanilla has removed the wounded state from the death message for the time being and I think every variant that had it before has upgraded already.
So there's is currently no pressing issue to look at that.

I was quoting from the current Junethack "unique deaths" listing, so there's at least one version still putting it in. (There aren't very many so far, but it's only June 2nd...)

But I guess it should be removed if it ever comes back. Otherwise you just have uncesseray grinding.

Agreed.

@actual-nh
Copy link
Contributor

actual-nh commented Jun 2, 2021

There are also a few cases in the current listing where the trimming seems to have gone a bit too far:

9 | imp
9 | killed by a fox
9 | orc

Additional ones include demon, hobgoblin, rock mole... here's part of the listing from OrderOfTheCockatrice (highest unique deaths by the current definition):

burning scroll
centaur
demon
dragon
drowned in a pool of water by an electric eel
escaped
falling object
giant ant
hobgoblin
homunculus
imp

@bhaak
Copy link
Collaborator Author

bhaak commented Jun 2, 2021

There are also a few cases in the current listing where the trimming seems to have gone a bit too far:

No, those are from 1.3d games: https://junethack.net/server/hdf_13d

This old version of NetHack didn't specify how the monster killed you.

@actual-nh
Copy link
Contributor

actual-nh commented Jun 2, 2021

There are also a few cases in the current listing where the trimming seems to have gone a bit too far:

No, those are from 1.3d games: https://junethack.net/server/hdf_13d

This old version of NetHack didn't specify how the monster killed you.

OK; changing such to "killed by [a|an] " would be nice in the long run. (Any idea what version still has wounded state?) EDIT: Changed my mind re reformatting 1.3d death methods; I suggest keeping them this way to act as an incentive to play that version...

@bhaak
Copy link
Collaborator Author

bhaak commented Jun 2, 2021

Without checking the code, I'm pretty sure that the wounded descriptions were a 3.7 thing.
xNetHack is the only variant that follows 3.7 closely, so it's probably from that.

But that also means I still have to fix that.

@actual-nh
Copy link
Contributor

actual-nh commented Jun 2, 2021

I can work on it. EDIT: See #72.

@RojjaCebolla
Copy link
Contributor

RojjaCebolla commented Jun 2, 2021

object materials are also showing up in deaths. harder but not impossible to grind

https://imgur.com/2qwRQZK.png

@actual-nh
Copy link
Contributor

actual-nh commented Jun 3, 2021

object materials are also showing up in deaths. harder but not impossible to grind

https://imgur.com/2qwRQZK.png

Should silver be removed? EDIT: As well as bone (except bone devil), wooden, steel (except steel golem), iron (except iron golem), and gold (except gold golem)?

@actual-nh
Copy link
Contributor

actual-nh commented Jun 3, 2021

"poisoned by" -> "killed by":

1 | poisoned by Pestilence (with the Amulet)
1 | poisoned by a bone orcish arrow
1 | poisoned by a giant mosquito
1 | poisoned by a killer bee
1 | poisoned by a little dart
1 | poisoned by a noxious patch
1 | poisoned by a quasit

EDIT: This will also be helpful for other fixes - note "poisoned by a bone orcish arrow".
EDIT2: I should be able to work on this also if I fixed the injury-related ones correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants