View Issue Details

IDProjectCategoryView StatusLast Update
0011793Dwarf FortressDwarf Mode -- Tradepublic2024-03-07 05:28
ReporterQuietust Assigned To 
PrioritynormalSeverityminorReproducibilityhave not tried
Status newResolutionopen 
Product Version0.47.05 
Summary0011793: Caravans check pack_animal/wagon_puller biomes incorrectly?
DescriptionWhen caravans arrive (or when the player first embarks), there appears to be logic to determine which creatures are appropriate for use as wagon pullers and pack animals, drawing from entity resources and filtering according to the local temperature and nearby biomes.

However, the biome check appears to be examining the region tiles around (region_x,region_x) instead of the ones around (region_x,region_y), so if the player's fortress isn't along the diagonal, the animals in question may be filtered inappropriately.
Additional InformationThe function in question takes 8 parameters: a pair of int16 vectors to accept creature race/caste values, another pair of int16 vectors from which to draw race/caste values (typically from entity resources), min/max temperature values (taken from plotinfo), and the fortress's region X/Y coordinates.

This dates back to at least 0.28.181.40d, where it also affected sieger mounts and minions.
TagsProbable Quick Fix

Activities

Quietust

2023-11-20 18:07

reporter   ~0041881

To be more specific, the bug is in the "pick_most_suited_animal" function, with it looping through Y coordinates `rx-1` thru `rx+1` (instead of `ry-1` thru `ry+1`).

Quietust

2024-03-07 05:28

reporter   ~0042070

Judging from the patch notes ("Fixed issue in how environment-appropriate work animals are selected for visiting fortress"), this appears to have been fixed in verson 50.12.

Add Note

Note

Issue History

Date Modified Username Field Change
2021-08-07 08:23 Quietust New Issue
2023-11-20 18:07 Quietust Note Added: 0041881
2023-12-01 06:08 Quietust Tag Attached: Probable Quick Fix
2024-03-07 05:28 Quietust Note Added: 0042070