planets troops increasing even with attacking ships in orbit

Describe your experience with the latest version of FreeOrion to help us improve it.
Forum rules
Always mention the exact version of FreeOrion you are testing.

When reporting an issue regarding the AI, if possible provide the relevant AI log file and a save game file that demonstrates the issue.
Message
Author
User avatar
Geoff the Medio
Programming, Design, Admin
Posts: 12268
Joined: Wed Oct 08, 2003 1:33 am
Location: Munich

Re: planets troops increasing even with attacking ships in o

#16 Post by Geoff the Medio » Sat Feb 17, 2018 10:46 am

Oberlus wrote:The compiled version, pulled from github two days ago, has the same problem.
Update to the latest.

User avatar
Oberlus
Small Juggernaut
Posts: 704
Joined: Mon Apr 10, 2017 4:25 pm

Re: planets troops increasing even with attacking ships in o

#17 Post by Oberlus » Sat Feb 17, 2018 10:58 am

OK, thank you, I'll do it ASAP (probably this evening) and report.

User avatar
Oberlus
Small Juggernaut
Posts: 704
Joined: Mon Apr 10, 2017 4:25 pm

Re: planets troops increasing even with attacking ships in o

#18 Post by Oberlus » Sat Feb 17, 2018 4:36 pm

The invasion button appears, and it works. But it seems something is still broken for the AI, it wont invade any planet, despite having visibility, defenses down, armed ships and more than enough troops for the invasion in orbit of the planet for several (5+) turns.

User avatar
Dilvish
AI Lead, Programmer
Posts: 4712
Joined: Sat Sep 22, 2012 6:25 pm

Re: planets troops increasing even with attacking ships in o

#19 Post by Dilvish » Sun Feb 18, 2018 12:13 am

Oberlus wrote:The invasion button appears, and it works. But it seems something is still broken for the AI, it wont invade any planet, despite having visibility, defenses down, armed ships and more than enough troops for the invasion in orbit of the planet for several (5+) turns.
Yes, the AI hasn't yet gotten these updates from CurrentMeter to InitialMeter that the human UI just got. I'll be trying to get a PR together for that later tonight, hopefully in time for a little review and testing and then hopefully getting merged before next week's build.
If I provided any code, scripts or other content here, it's released under GPL 2.0 and CC-BY-SA 3.0

User avatar
Dilvish
AI Lead, Programmer
Posts: 4712
Joined: Sat Sep 22, 2012 6:25 pm

Re: planets troops increasing even with attacking ships in o

#20 Post by Dilvish » Sun Feb 18, 2018 5:44 pm

The fix for AI invasions of generally-shielded planets was a simple fix, I did a quick test and pushed it to master rather than holding it up with a PR.

I'll also do a review of other AI uses of CurrentMeter and decide which ones would probably be better as InitialMeter, depending on how clear the decision seems I may push it directly or put up a PR.

Side note, I also put in a fix for SP_SLY-- they had been suicidally dejected over their recent affliction with BAD_SUPPLY and were essentially not colonizing at all, that's fixed now.
If I provided any code, scripts or other content here, it's released under GPL 2.0 and CC-BY-SA 3.0

User avatar
Dilvish
AI Lead, Programmer
Posts: 4712
Joined: Sat Sep 22, 2012 6:25 pm

Re: planets troops increasing even with attacking ships in o

#21 Post by Dilvish » Tue Feb 20, 2018 8:23 pm

dbenage-cx wrote:Given a system with only a single planet with Ancient Guardians, still seeing +1 troop gain where ships have attacked for 5+ turns.
This issue is still ongoing, I'm going to make a github Issue on it. One AI planet I was camped on was still getting troop regen of 5 per turn even though it was under constant attack by my ships, and a similar thing happened when one of my planets was under blockade from an AI.
If I provided any code, scripts or other content here, it's released under GPL 2.0 and CC-BY-SA 3.0

User avatar
Dilvish
AI Lead, Programmer
Posts: 4712
Joined: Sat Sep 22, 2012 6:25 pm

Re: planets troops increasing even with attacking ships in o

#22 Post by Dilvish » Thu Feb 22, 2018 9:10 am

This troop regen glitch is proving tough to corral. In the meantime, the AI has probably been under-performing and making testing less enjoyable, so I've put up a PR that includes both one 'normal' invasion planning improvement for the AI, plus a temporary adjustment basically informing the AI of this regen glitch. I suggest any active testers cherrypick both commits for now.
If I provided any code, scripts or other content here, it's released under GPL 2.0 and CC-BY-SA 3.0

Post Reply