beyond-all-reason/Beyond-All-Reason

Snap-to-Deposit buildings sometimes are terrain incompatible at snap location

Open

#3,299 opened on 2024年7月2日

GitHub で見る
 (5 comments) (0 reactions) (0 assignees)Lua (3,922 stars) (592 forks)batch import
BugGadgetGood First Issue

説明

Description

First time I saw this was with geothermal in the following clip:

https://youtu.be/JjfcURmweBQ?t=23m20s

Expected Behaviour

The snap location should be tested to make sure it can actually be built at the location and moved to the next nearest location, tested, moved if it can't place, repeat until a valid location

All this for both metal extractors of all types and geothermal of all types

Actual Behaviour

The player must find a valid build location for the deposit

Reproduction steps

Play on the map LostDeadMan was playing in the video (I forget the name) and explicitly select a geothermal (probably works with any type, but Cerberus was what he was trying to build in the video). Notice that there are lots of places it can't be set.

It might be required to deform the terrain a little bit first. In which case, make a Tzar or Fatboy to ground fire around the area.

Other

No response

コントリビューターガイド

技術スタック
lua
領域
desktop
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2 days
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Lua programming basicsUnderstanding of terrain heightmapFamiliarity with game's building placement system
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
75
調査方針
Investigate the building placement system for snap to deposit buildings, likely in files related to building placement or terrain validation. Examine the logic that checks terrain compatibility at snap locations. The expected behavior is to test each snap point in sequence until a valid location is found. Look for any existing loop or iteration over snap positions. The fix should modify the snap to deposit logic to test each candidate location and move to the next if invalid. Refer to the video linked for reproduction steps.