View Issue Details

IDProjectCategoryLast Update
0019527AI War 2Audio WorkDec 7, 2021 12:12 am
ReporterChris_McElligottPark Assigned ToChris_McElligottPark  
Severityminor 
Status resolvedResolutionfixed 
Summary0019527: Metal warnings should only warn me when I'm newly poor.
DescriptionBasically if I'm running out of metal a lot, I shouldn't be harassed about it with the voiceovers. If I've had a lot of metal for a while, but then drop to very low amounts, she should say something. But as long as I remain poor, she should be quiet. ;) Then if I become richer again, the flag trips saying "warn me again next time." But even there, potentially saying "I need to be reasonably metal-wealthy for x amount of time" would probably be good, because I might have a spike of metal and then a big dropoff.
TagsNo tags attached.

Activities

BadgerBadger

Apr 9, 2018 11:56 am

manager   ~0047394

Last edited: Apr 9, 2018 12:34 pm

So it looks like the LowOnMetal warnings never actually got implemented in the first place. Do we have the audio for that? If so then I'll add the hooks for LowOnMetal with a "Chris Todo" for you to set up the audio.

BadgerBadger

Apr 9, 2018 12:01 pm

manager   ~0047395

Last edited: Apr 9, 2018 12:01 pm

This is a question for Metal, Fuel and power warnings. Should we allow the "Low on Resource" threshold to be a Setting so the user can configure it, rather than an ExternalConstant?

Chris_McElligottPark

Apr 9, 2018 1:16 pm

administrator   ~0047397

We actually don't have any metal low warnings recorded. If there are a variety of things that we want to put in a new batch to the voice actor, I can get her a list of those things to work on sooner than later.

If there are some existing areas where we need more variety in what she says due to frequency, I can also get those to her in that same batch. I figure we have at least two new batches for her.

Regarding the settings thresholds, I'm certainly fine with that.

BadgerBadger

Apr 9, 2018 1:40 pm

manager   ~0047398

Okay, I'll put in some settings for Minumum Threshold for Warning, and also add the code for Low On Metal warnings.

For a Major Attack, previous what we'd done is just say "If the AI forces sufficiently stronger than your defenses, it's a major attack" which meant that undefended planets were getting Major Attack warnings whenever any AI forces were there. This was particularly annoying for planets that I wasn't planning on holding. I put in a limit for "Don't consider it a major attack unless the force is stronger than X", where X is defined in the XML (and is 5K). I could also make this a setting, or just leave it as an External Constant defined in the XML. Opinion?

BadgerBadger

Apr 9, 2018 1:56 pm

manager   ~0047399

We might consider having different wanrings for "Metal is getting low" and "Out of metal/Metal Critical level". Basically, a player might want to be able to say "Give me a warning at 100K metal left", but we might also want a "You are out of metal" warning that's distinct.

BadgerBadger

Apr 9, 2018 9:50 pm

manager   ~0047401

Code is done (if you want "out of metal" instead of "low on metal" seperately then let me know, it's easy to do). Waiting on vocals

Chris_McElligottPark

Dec 7, 2021 12:12 am

administrator   ~0063367

Fixed long ago.

Issue History

Date Modified Username Field Change
Mar 23, 2018 9:09 am Chris_McElligottPark New Issue
Mar 23, 2018 9:09 am Chris_McElligottPark Status new => assigned
Mar 23, 2018 9:09 am Chris_McElligottPark Assigned To => keith.lamothe
Apr 9, 2018 11:56 am BadgerBadger Note Added: 0047394
Apr 9, 2018 12:01 pm BadgerBadger Note Added: 0047395
Apr 9, 2018 12:01 pm BadgerBadger Note Edited: 0047395
Apr 9, 2018 12:34 pm BadgerBadger Note Edited: 0047394
Apr 9, 2018 1:16 pm Chris_McElligottPark Note Added: 0047397
Apr 9, 2018 1:40 pm BadgerBadger Note Added: 0047398
Apr 9, 2018 1:56 pm BadgerBadger Note Added: 0047399
Apr 9, 2018 9:47 pm BadgerBadger Assigned To keith.lamothe => Chris_McElligottPark
Apr 9, 2018 9:50 pm BadgerBadger Note Added: 0047401
Dec 7, 2021 12:12 am Chris_McElligottPark Status assigned => resolved
Dec 7, 2021 12:12 am Chris_McElligottPark Resolution open => fixed
Dec 7, 2021 12:12 am Chris_McElligottPark Note Added: 0063367