bugBos Wars - Bugs: bug #30971, Units attack buildings too...

 
 

bug #30971: Units attack buildings too stubbornly

Submitted by:  Vytautas <prozacr>
Submitted on:  Sun 05 Sep 2010 08:18:43 PM UTC  
 
Severity: 3 - NormalItem Group: Bug
Status: FixedPrivacy: Public
Assigned to: Kalle Olavi Niemitalo <kon>Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sun 19 Sep 2010 12:15:51 PM UTC, comment #5:

Fixed in trunk r9960.

Kalle Olavi Niemitalo <kon>
Project MemberIn charge of this item.
Sat 18 Sep 2010 08:45:45 PM UTC, comment #4:

PATCHv2 fixes an assertion failure, I hope. This patch is for trunk r9957.

(file #21488)

Kalle Olavi Niemitalo <kon>
Project MemberIn charge of this item.
Fri 17 Sep 2010 06:06:43 AM UTC, comment #3:

They already try to hit engineers first. See the Priority settings in units/*/*.lua, FindRangeAttack in engine/unit/unit_find.cpp, and CheckForTargetInRange in engine/action/action_attack.cpp. And this even worked in my test.

Here's what I did to test it:

  1. Created a map with player 0 vault at (0,0); player 0 tank at (7,10); player 0 engineers at (7,9), (6,9), (6,10), (6,11), (7,11); player 1 camera at (11,10); and player 1 artil at (18,10).
  2. Configured player 0 as human with initial energy 10000 and initial magma 20000.
  3. Configured player 1 as computer with default AI, no energy, and no magma.
  4. Stated a game on this map, with Reveal Map disabled and Fog of War disabled.
  5. Let the tank blow up the camera.
  6. Ordered the tank to move to (9,10).
  7. The enemy artil then shot the engineers behind the tank before it began firing on the tank.

The Bos Wars in this test was r9950 + several patches:

  • Enable the scons build cache
  • Support implicit hot keys for check boxes
  • Add UI:HotKeyOfCaption for Lua
  • Add hot key for "Show terrain attributes" in editor menu
  • Allow neutral buildings on MapFieldNoBuilding
  • Disallow buildings in the pool of smallWetPlain
  • Disallow buildings in moon craters
  • Add RandomPosition property of unit types
  • Add NudgeUnitUnderCursor in the map editor
  • Randomize positions of trees and morels
  • Add drought.map
  • Fix tablet input in full-screen mode
  • SDL 1.3 SVN r5558 compatibility
  • Refuse to define variables after unit types
  • Alien mushroom tree set from Bagador
  • Update unit-tree08 for current engine and add to editor
  • Update unit-tree08 for current conventions
  • Speed up pathfinder by caching islands
  • Morel spores preview
  • More random sporulation
  • After a unit is fully harvested, look for similar ones
  • Tell the AI who can build the shipyard and ships
  • Help the AI find places to build shipyards
  • Added "GoshGeom" map patches by Michiel van der Wulp.
  • Added "Empty Buildings" map by Michiel van der Wulp.
  • Preserve WEAK_TARGET flag in more places
Kalle Olavi Niemitalo <kon>
Project MemberIn charge of this item.
Tue 14 Sep 2010 12:15:58 PM UTC, comment #2:

Sry I still not tested, because:
https://savannah.nongnu.org/bugs/?30976 #7

Also this concept of AI should be extended to engineers.
Now tank with engineering team behind it is kinda superunit.
AI of player units and pure AI should try to hit engineers first if they are in range.

Vytautas <prozacr>
Sat 11 Sep 2010 01:19:39 PM UTC, comment #1:

Please test the attached patch, for trunk r9950.

(file #21435)

Kalle Olavi Niemitalo <kon>
Project MemberIn charge of this item.
Sun 05 Sep 2010 08:18:43 PM UTC, original submission:

Reproduction:
Player unit squad is moved to assault with A+click.
It attacks enemy unit production building.
Produced units move away from building.
They ignore event and die unless micro-controlled to handle danger.

Expected behavior:
Units should try to survive in A+click assault and try kill dangerous things first unless ordered to kill building with direct click on it.
---
It should be very easy to abuse current behavior in multiplayer. Allow enemy to hit buildings with large amount of hitpoints then kill all his units in conterstrike. If player is will not handle situation manually his units will die even without fighting back.

Vytautas <prozacr>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #21488:  0001-Preserve-WEAK_TARGET-flag-in-more-places.patch added by kon (3KiB - text/x-patch - [PATCHv2] Preserve WEAK_TARGET flag in more places)
file #21435:  0001-Preserve-WEAK_TARGET-flag-in-more-places.patch added by kon (4KiB - text/x-patch - [PATCH] Preserve WEAK_TARGET flag in more places)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by kon (Updated the item)
  • -unavailable- added by prozacr (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 19 Sep 2010 12:15:51 PM UTCkonStatusPatch available=>Fixed
      Assigned toNone=>kon
      Open/ClosedOpen=>Closed
    Sat 18 Sep 2010 08:45:45 PM UTCkonAttached File-=>Added 0001-Preserve-WEAK_TARGET-flag-in-more-places.patch, #21488
    Sat 11 Sep 2010 01:19:39 PM UTCkonAttached File-=>Added 0001-Preserve-WEAK_TARGET-flag-in-more-places.patch, #21435
      StatusNone=>Patch available

    Back to the top


    Powered by Savane 3.1-cleanup1