Changelog updates for 0.25 release

First attempt at finalizing release highlights and update the debian
changelog.

(cherry picked from commit 35acc811ca793aae16c45c385c440107c0939756)

Fix an interaction of fallback randarts and acquirement (12277)

I hadn't fully understood one of the ways in which acquired objects
could interfere with unrand generation. After this commit, unrands that
place outside of levelgen (acquirement) or in the abyss or a trove
should always cause later instances of that unrand to simply be replaced
with a fallback randart, and have no other effect on item generation
during levelgen. Before this commit, unrands that were forced by vaults
would be correctly replaced with a fallback if they had already been
acquired, but unrands that were placed randomly (e.g. by | in a vault;
Slime:5 was a good place to check) could be affected by previous
acquirements in a way that had unpredictable results for those items.
I'm pretty sure that this bug had no impact on item generation (or
levelgen more broadly) outside of those specific grids.

This commit may change some seeds as I'm adding a subgenerator the will
impact certain rare cases of unrand placement (mainly for uniques who
can come with a special unrand). These uniques may need further testing.

No save compat for this, so upgrading will lead to the same behavior as
games from before this commit would have.

(cherry picked from commit 850bfe61e39c27fcdc3dcf3f7b28b0b55a6c1371)

Fix pregeneration for upgrades where branch size changes

Pregeneration was using the current branch depth, rather than branch
depth from the save. (It's possible that saves this old should be
automatically treated as `classic` levelgen mode, too.)

(cherry picked from commit 18c3a0dd01f83f30f4c71fb6e24ffc45e25cc65c)

Don't warn about blocked LOF for Starburst

Starburst is essentially untargeted, but its tracer beams were
generating line of fire warnings whenever any of its 8 tracer beams had
line of fire blocked from player to a location at spell radius away.
Some beams might hit a monster even if others have LOF blocked. The
tracer beam target locations aren't important in any way as they just
exist to let the tracers hit the entire spell radius.

It might be nice to give the LOF prompt if all beams that might hit a
monster have blocked LOF. That would require some significant
refactoring, so for release, this commit removes the warnings
specifically for Starburst.

Fix some target prompts under Fedhas

Fedhas completely protects allied plants from the effects of Airstrike,
Shatter, and tin of tremorstones, but players currently receive
incorrect penance warnings when using these effects. Add the relevant
Fedhas ally checks so that the attack prompt won't be triggered for
these cases.

Fedhas currently doesn't protect allied plants from player Freeze, but
the prompt correctly warns them in this case. Freeze and intentional
melee attacks are the remaining exception where we do apply Fedhas
penance, but there's no reason for these exceptions to exist that I'm
aware of. It would easier to centralize the prompting logic in
bad_attack() if all attempts at harming a Fedhas ally by the player were
prevented. This can be addressed in a future commit, but the fixes in
this commit will be enough for release.

Change = to == in checking crawl_state.type

Every game is displaying the seed, whether or not it's a custom game since the statement evaluates as true.

(cherry picked from commit 6c07f0ee8d465e3e9ea0214c88ad614b59fecd43)

Fix viewmap catch2 even more

(cherry picked from commit 79b31b866221e7ae4d7b6055fa5c36dd7884a028)

Fix catch2 tests for map panning / clamping

(cherry picked from commit eb7e7872c213f5cef4e0adbd47b661165b4c4ea2)

Show seed for seeded play with welcome message

This is because of reports of players forgetting that they had a seed
set in their rc file. This also prints it on the "welcome back" message,
not sure if that's too repetitive.

Also, tweak this message a bit.

(cherry picked from commit 00fb324cd09f9518cf33ae1896d3cc5e93489a07)

Tweak how seed games are displayed in ?v and chardump

Consolidated code between these two. This doesn't matter so much for
chardump, where the first line also says "Seeded", but it's a good
reminder for players looking at their seed in ?v. I've heard of players
forgetting they were starting a game in seeded mode (e.g. when setting
it via the rc) so there's maybe more telegraphing of this mode that
could be done somewhere.

(cherry picked from commit d69c119aaa7d7331e97fc7827915a25bf11017d5)

semicolon

(cherry picked from commit 5c1c449dd35ea1ef4cac4c5314d922873a168062)

Restore map view clamping behavior (elliptic)

ca36f614e438 conditioned clamping on `map_bounds` instead of
`known_map_bounds`, the behavior before this commit. I think this was
unintentional, and it leads to weird behavior on unexplored levels in
console (e.g. lets you move the cursor outside of the screen, restores
to known_map_bounds when you move outside of regular map_bounds). This
commit restores the previous behavior for `X` where you can't scroll
beyond known map bounds. As a side note, this is inconsistent with `x`,
where you can, which is kind of messy.

TBD how this commit interacts with the test in bd17af9.

Use correct DES syntax for wand charges

The correct field to set is "charges", not "plus". Fix this in
gammafunk_dig_for_victory. Also remove a reference to max wand charges
in the DES documentation.

(cherry picked from commit 15ccc7ae601939c08280535bd7162a7da788072c)

Changelog tweaks

(cherry picked from commit 45f243fec86c0523a2c7a296ca31fef7a4fcc11c)

Don't pick up duplicate tins of tremorstones

By default, autopickup ignores tins as all other misc and dangerous
items. But in case the player has `autopickup_exceptions` for them,
duplicate tins should be ignored.

(cherry picked from commit 0e6732e0a6af59df83ec7a023fcf1cbf5057c33f)

Remove a stray bolt of fire

This really should come with some sanity checking for vault-placed
spellbooks, but I'm just doing the quick fix for now.

Resolves #1442

(cherry picked from commit 98f9fe671ce0708bee87b5ea4978a1763ab68aef)

Consolidate code for applying game_scale

This moves the state out of Options (except for initial setup) and does
the scale calculation in one place; I lumped this in with other
density-related code.

As a side effect, this commit tweaks the timing of when game_scale is
initially applied: it will take effect after the initial loading screen
is displayed. This heads off a bunch of opengl errors that (I think)
were caused by game_scale being applied before device_density is fully
initialized.

Also, improve some opengl error messaging for debug builds,
constexpr-ify HiDPIState.

(cherry picked from commit 55d99e38fcdaf201cfef3d3b912917da5ba33330)

Let tile_filter_scaling=false work on mipmapped textures

Did you ever get the sense that this setting didn't do much? That's
because it didn't. (All dungeon textures, UI textures, probably more are
mipmapped.)

This makes high scale values lead to pixellated graphics.

(cherry picked from commit edd8c26f918388319369c116790f55e9084ebf2b)

Fix font glitches at very high device density

The code that loads in the glyphs was relying on face->bbox to give the
max glyph sizes. Which it does, but insensitive to device pixel scaling.
Coincidentally, the values worked up through 4x scaling, but no further.
This uses the actual correct advances in pixels. One open question: does
this affect font rendering at lower scale values? I'm pretty sure the
answer is no -- that it just affects how efficiently glyphs are packed
at more typical scale values.

(cherry picked from commit a07a990f657920bdbb40fa3395d38db5798f4a84)

Add a global scaling option for local tiles

OS support for high-dpi rendering varies widely, but the sizes of UI
elements are very hard to adjust for higher-resolution displays where
high-dpi is not supported. This commit adds a coarse global scaling
parameter that mimics high-dpi behavior, scaling by a value from 1-8.
I'm not sure if there's any current devices out there that would need 8,
but 2 and 3 work reasonably well for my 2k ultrawide monitor, so I
expect that even 4 will be usable for a 4k monitor without high-dpi
support.

This is mostly implemented at a very low level by exploiting existing
high-dpi code; it would be nice if this could be consolidated to be done
entirely at the `WindowManager` level.

(cherry picked from commit c699729d39af1c70ebb246fe7f1192c56898b471)

Restore note about pasting to local tiles seed selection

This was lost in 04990682366a as part of improvements to the seed
selection menu (since no button corresponds to [p] and pasting is more
generally possible in local tiles text input). However, I think it's
still useful for players to have a reminder that pasting is possible at
this screen, and describe the way in which it may not behave as
expected.

(cherry picked from commit 4f279345d96b1c64467ac45b33d21b7bd5263c16)

Don't print "levelgen mode: deterministic"

This is the default now and present in basically all morgues on all
servers, no need to mark it explicitly.

(cherry picked from commit b98801ac91897bf1cf898b1e98e55e93e3965249)

Fix some no_tele_into shops (minmay)

It was still possible to blink into them and get soft-locked.

Instead, add a helpful escape hatch!

(cherry picked from commit 50178cf780ef9f5f3f59026321a48792b778c185)

Fix up some typos in the docs folder. (#1434)

* Fix some typos in god_creation.txt.

* Fix a typo in the quickstart guide.

* Fix typo in monster_creation.txt.

* Fix some typos in mutation_creation.txt.

* Fix some typos in patch_guide.txt.

* Fix a typo in tiles_creation.txt.

* Fix a typo in translation.txt.

Co-authored-by: Aaron Golden <aegolden@gmail.com>
(cherry picked from commit 11e0b32562862e76f4207709cc9a1acbd337767c)

Tweak dispreferred newgame colors on local tiles

At darkgray, these read to newer players as unselectable, I think due to
standard UI conventions (noticed this here:
https://www.reddit.com/r/dcss/comments/gspvkg/i_made_a_review_of_dcss_let_me_know_what_you_think/).

Adjust them to be only slightly darker than the preferred colors. This
may now be too subtle? What might be better is to provide a key with
these menus.

(cherry picked from commit 58eb27aebac64d331345a585a9c4407b37352257)

Fix an unused param warning

(cherry picked from commit 9e0bf0358992db7f44c769a99012996eaaf19171)

Fix a signed/unsigned warning

(cherry picked from commit c8e6c4a4156af2025b822699c73fe445b003f88e)

Remove ability to memorize spells from their description

With the introduction of the spell library, this was only ever possible
by disabling autopickup and then examining a book you were standing on.

(cherry picked from commit c29e872307671ae0c30153365dbb077af791f614)

Update the changelog

(cherry picked from commit 56b8cd3df03db7992e354d2de7054f731ae50274)

A tile for tin of tremorstones (CanOfWorms)

(cherry picked from commit b080af96c6623553fa1add822c0cf60b7dc343bc)

Don't have allies avoid player line of fire

In 340c9e81, monster AI was improved to avoid LOF between a monster's
allies and their targets. This change had the player considered as an
ally, hence summons in front of a player would try to move out of the
path between the player and a hostile. This is convenient when the
player wants to use a ranged attack, but doesn't work well in the more
common case were allies are used for cover. This commit has the player
not be considered a ranged ally by allied monsters, hence they won't
avoid player LOF, but continues to allow that consideration for fellow
allied monsters.

There's probably no good way to predict when the player wants to make a
ranged attack behind allies, so it's possible that a new 't' command is
the best solution. A way to toggle this LOF-avoidance behaviour through
other 't' commands might need some consideration, if something like this
is implemented.

(cherry picked from commit c929df558523a23e63c82ce3060a57a1adc2bc26)

Fix a randart name class (12270 / Flugkiller)

(cherry picked from commit f97e55dc39afe998812bc2345bd20b6602f9d198)

Remove Lightli...

...'s lack of credit in CREDITS.txt. Thanks for your submissions!

(cherry picked from commit 5f92b90fa0c7481cd3aeafe41d74871641299f0b)

Update CREDITS.txt

(cherry picked from commit 5d2c98bf94f40ca7f773692c66e5d733e898998b)

Stimulate Xom on worship from a faded altar

Closes #1413

Xom doesn't use normal piety, but thinks its hilarious when you end up
worshipping from a faded altar.

(cherry picked from commit e6218251f0ce7c4ecea0b0d2f3e18f30be1eb5ef)

Display Zz and ? status icons for allies

Having this information on the main view is useful because confused
allies can attack the player, and paralysed allies block movement.
Also, since the player gets the top-tier stabbing bonus when
attacking sleeping/paralysed/petrified allies, that should be
indicated by the Zz's, TILE_FLAG_STAB, too.

(cherry picked from commit 3fdfaa9094b56e429a7f6979b57ba7e220c30e3b)

Add adjectives to the randart name generator

[ Committer's note: split some cross-loaded history from this into other
  commits, tweaked and edited. Closes #1414 ]

(cherry picked from commit 366e9434eded899b7cc1f26d78f6850309a408f0)

Add verbs to some randart names

Randarts can be "of the noun", let's expand that to "of the verbing noun".

[ Committer's note: split out new nouns into a separate commit, removed
  a future goal from the commit message that isn't in line with the
  devteam's goals. ]

(cherry picked from commit 7b3c7a5917ff391aefae6c483abb9c123f65e7a9)

New randart apperance adjectives (cebolla)

(cherry picked from commit b168a4ef397b0eeca58ddec512ad7be149049772)

More randart nouns (cebolla)

(cherry picked from commit 4aaa7df55cc7a939dd862ac85c32bb3b596a60ec)

Typo fix (Goratrix)

(cherry picked from commit 9c5a1843486b887ce3789b739fe6634b633808a1)

Revise transmutation miscasts

The polymorph effect was very flavorful, but even at minimal power a 150
aut badform was the result. Compared to other miscasts this was too
intense. Reducing form duration would require a significant overhaul of
player badform durations and power scaling because of how the existing
codepath is set-up.

Instead, since this is close to 0.25, this commit changes transmutations
miscasts to increase player contamination: double existing contamination
and then add the resulting miscast damage to the contamination score.

Contam is still flavorful (since transmutations change the body and
contamination is residual magic in the body and high contamination can
cause mutation), though not as dramatic as polymorph.

Possible idea for 0.26: add polymorph as a possible consequence of
decontamination.

(cherry picked from commit 709782ff74ba80b6e8f9adcdb196ad4498488618)

Default `restart_after_save` to true

What this means in practice is that saving returns to the main menu, a
behavior that is much more typical for what people expect these days out
of games. The option is retained for people who like the classic
behavior.

(cherry picked from commit 750a858e70e7b47fd22455d2acef4e0d8946c0c8)

Fix tiles

I'm not sure why, but this assert triggers when starting up local tiles.

(cherry picked from commit 32a1ce3a9d21b5af53381a3f13857037f39ffb93)

Fix some unused function warnings for non-tiles builds

(cherry picked from commit 448add79b74c593ee9ab6a6b9b31c5ceb4778e23)

Fix the tiles build

(cherry picked from commit bc5e7ad058631a9fc9a9fb4c381ca6ff4c891fb5)

Don't crash on too small terminal resize (6498)

The new behavior is that a message is printed in the upper left telling
the player their terminal is too small, and nothing else is drawn. This
is pretty rough and ready, and there may still be glitches. In most of
my testing a resize event also comes with an ESC (I'm not at all sure
why) and this ends up exiting any exitable menus. You can still get
glitches by resizing in menus that aren't exitable, e.g. the skill menu
with no skills selected. But it's still probably better than crashing.

This largely doesn't work on webtiles builds in terminal, for unclear
reasons.  However, that is not the main use case for this, and webtiles
console is played wrapped in dgamelaunch anyways. In my testing this
isn't a regression relative to current dgl behavior. (...which is a
mess.)

(cherry picked from commit 9b8842f3ab7c3e483b0781cf2baa8cfb29d4676d)

Keep ui sizes in sync with terminal resizes (12260)

Hopefully this is the right way to do it, but it stops a certain class
of crashes in my testing: start the terminal with size n, shrink to some
size <n that is still a valid terminal size, and hit `S`. (or any other
command that can trigger clear_text_region.)

This fixes the specific crashes reported in:
https://crawl.develz.org/mantis/view.php?id=12260&nbn=3#bugnotes

(though not all crashes on resizing.)

(cherry picked from commit d8d43ee9da1d943d423757aab809dd7a40961caf)

Add 0.24.1 to master changelog

was only in a branch before.

(cherry picked from commit cb1fb4266058919fbcd7752ec1fab272d3f05a04)

Don't use a valid direction to indicate a cancelled choice

Fixes a bug where it was impossible to choose UP_LEFT in the
target-compass prompt.

(cherry picked from commit d791c6801d209806e15f1e02e9c015be3b274ab3)

Crash more quickly if (offline) console is resized too small

If `winner` is not valid here, then some of the following coordinate
assignments will also be invalid and lead to somewhat unpredictable
crashes later (because the `valid` bool is set basically if the
coordinate setup aborts in the middle). This standardizes the crashes.

What would be much, much better is if the display just blanked with a
message saying "your term is too small" until the player resized again,
without crashing.

(cherry picked from commit 95c47da36813473c5bd7b879532fa55730c78476)

Remove +11 GDA (vt)

Adjust troves fees

Now that agility potions have become stabbing potions, their situational
utility makes them a poor choice for trove fees.

Scrolls of fog and fear are added as trove fees as well, since they are
both fairly strong defensive consumables, although an argument could be
made that fear's utility decreases as the player progresses in the game.
Both scrolls are given the same fee as blinking, since they have the same
generation weight, but it's also reasonable to make them cost more.

Scrolls of summoning are also a good candidate for trove fees, but I'm
not adding them for now since it requires some uselessness checks if the
player has sacrificed Love under Ru.

checkwhite

Insert a cap in spell failure calculations

It's possible to cause an overflow in the spell failure calculations with
extreme encumbrance values, achieved by having 0 or 1 strength and wearing
CPA. This commit sets the cap to a value slightly lower than where the
overflow should theoretically occur to avoid this.

This should be considered a temporary solution until something more elegant
is done with the spell failure calculations.

Require upper-case Y to forget Vehumet gifts

Resolves #1418

Update misc acquirement handling

Don't generate tremorstones in stacks (this would cause crashes since
they're now non-stackable), and don't generate tremorstones again once
they've been seen. Since this makes it possible for all misc items to be
useless (with Sacrifice Love and having found one of each of the XP
evokers), add a fallback to just generate duplicates of the XP evokers.

(Currently the only way to actually acquire misc should be via vault
placement in a couple of sprint maps.)

Remove acquirement items from some abyss vaults

Only troves should use actual acquirement code to generate tailored items -
other vaults use good_item for high-quality loot.

Fix a broken alphashops entry (#12262)

alphashops places items by subtype only and doesn't generate items that
require "ego" lines, so changing harm from an amulet to a scarf means it
should be removed from the vault (since the "S" shop already places
scarves).

Since scarves are defined purely by their ego and are more like jewellery
than other armour slots, it'd probably also be fine to rework the vault to
remove scarves from "S" and place each ego in the relevant shop.

Even more save info fixes / tweaks

Obvious bugfix

Correctly handle pre_options for json calls

In order for the dgamelaunch-config wrapper scripts to work, these have
to be passed. This also corrects a bunch of other issues that showed up
in live testing: don't pass other options, pass a dummy username for the
sake of launcher scripts (they crash if this isn't there).

Fixes for live-debug mode

Move sockets to a separate directory, so the real server doesn't try to
connect to them. (A few other tweaks.)

Add a debug mode for live-testing server updates

This may or may not be entirely safe to use in reality...

Fix a typo

Fix some issues with lobby template timing

* Send an initial empty template when logging in, since it can take a
  noticeable lag to fill in the save info.
* rerender the banner even for logged-out users - previously, updates to
  the banner would require a server restart for logged-out users to see.

Let --port disable config.py SSL for now

Since this options is mainly intended for testing, hopefully for running
a concurrent testing server, we don't want a bind error if the config
does set up SSL. So for now, just have the CLO disable SSL altogether.
If there's demand something better could be done.

Add some server.py command line options, refactor

Also, remove dependence on `from config import *`. I think I've gotten
everything, but because a lot of options are used only in very specific
server configs it's hard to test thoroughly.

Remove unused member

Fix a couple typos in options_guide.txt (#1412)

Placeholder tiles for cigotuvi's embrace

The item tile is a butchering of the shadow dragon armour with a bit of
the shape hacked off and some unpleasant brown-to-green colors replacing
the red.  The equip tile is just an existing green armour tile that was
kicking around unused (at least as far as I saw), it's not really a
perfect match but it's ok-ish for now. I eagerly await replacements from
a real artist, but for a release anything seems better to me than a TODO
tile...

Hood of the assassin tiles (RojjaCebolla)

This replaces some TODO tiles, see #1387.

Fix a bug with shared save slots in the lobby

When starting a new game or ending a game, save slot info did not get
updated properly for other game types that shared a slot. This is a sort
of basic fix, in that it isn't caching very well (because the way crawl
handles save slots is a bit of a mess).

No servers are using this yet, so I don't think it has been an issue in
practice.

Visual tweaks to lobby save browser

The endless [start] links didn't look great on servers that have lots of
binaries, especially ones that don't support showing save info.

Also, adjust the sample game config to something a bit more like what
typically shows up on servers.

Revert "Prevent formicids from falling far (for flavour)"

The flavour justification for having shafts work differently on Formicids is
fairly weak, and doesn't really make it worth adding on as an additional
minor species property.

This reverts commit a0b20f49667755fb8d06100433ec6928ee9a1179.

Formicids cannot cast Swiftness

This has been a long-standing bone of contention in vanilla (see Mantis
issue 7786) because the original formicid restrictions were a somewhat
arbitary duplication of the effects of the now-gone amulet of stasis.

If formicids can't benefit from Haste, well Swiftness is Haste Lite.

This change is borrowed from Stoat Soup.

Prevent formicids from falling far (for flavour)
As shafters and tunnelers, they needn't have their self-shaft
escape ability hosed by shaft traps;  they can halt their fall
and dig to safety after falling only one floor.

checkwhite (again??)

Another linting fix

checkwhite

Don't disallow % formatting

Fix flake8 errors

at least locally. Mostly false positives to do with bad interactions
with comment typing.

Update the changelog

Remove isort as a CI check

I think this is something that it's useful for someone to run every few
months, but not a good fit for a lint check run on every push.

Reveal invisible monsters that fire visible beams at the player (minmay)

This prevents an information asymmetry for players that don't use or
cannot see animations.

Use the magic mapping glyph for unvisited stairs and teleporters

By default this is not a distinct glyph. Using the magic mapping glyph
for unvisited stairs and teleporters allows visually impaired players to
screen read this information easily which was previously only
communicated by colour.

This does mean the magic mapping glyph has a slight overload for stairs
(unvisited vs. completely unseen) but the two different meanings are
close enough and the information about stair connectivity isn't provided
until the player actually views the stair; this is a strict improvement
for visually impaired players.

Allow customizing the travel trail

Add a dummy feature DNGN_TRAVEL_TRAIL for console display customization,
like DNGN_EXPLORE_HORIZON. Visually impaired players have asked that it
be easier to distinguish the auto travel trail using a screen reader by
setting it to a different glyph. Sighted players might also like this
option, or the option to customize the colour.

Fix some pep 8 stuff

Add a comment about JSON decoding errors

Handle a bad dgl-config + webtiles interaction

The dgl-config game launchers, except under whitelisted conditions, will
send a clear screen + go upper left escape code, which will mess up any
json output from various CLOs. This commit handles the resulting
corrupted output more gracefully. To actually fix this, servers running
dgamelaunch-config will need to merge the following commit into their
launcher scripts:

https://github.com/crawl/dgamelaunch-config/commit/6ad788ceb5614b3c83d65b61bf26a122e592b98d

Fix flake8 error

Add experimental save info to webtiles lobby

This adds some features for showing save info to the webtiles lobby, a
longstanding ui gap. This is all relatively experimental and disabled by
default; I think it may need some tweaking once it hits a live server.
Because this feature is aiming for backwards compatibility there are a
number of idiosyncracies to the implementation. Number one is that it
accepts the (extremely confusing) way dcss treats save slots - for each
player and game version, one save slot for sprint, and one for all other
game types.

CLO to give game types in json

Aimed at alternative save browsers.

Add a CLO that prints save info as json

For use with file browsers outside the game proper.

Disable yaml games by default

DGL is copying the games.d folder automatically, which leads to trying
to load the sample one on top of existing real configs. Disable by
default for update compatibility.

Python 2.7 compatibility

Add a recommended options section for blind players (#1407)

Closes #1407

[skip ci]

Add back in all non-default templates for compatibility

A running server will go into a crash loop without these, especially the
missing `shutdown.html` (which actually prevents the server shutting
down). Most DGL setups *shouldn't* be affected by these files being
missing, but given what happens if shutdown.html is missing, it's safer
to do the master commit without the files missing.  Someday these can be
deleted from the main repository.

Always show milestone column

A future commit will allow specifying milestone path in each game
definition, which significantly reduces the complexity of configuration.

Allow overriding some HTML templates

Server operators can create custom versions of the banner, footer, and
shutdown templates without needing to overwrite source-controlled files.