Bug: 'Death from Above' & 'Snap Shot' allow for a free Serial-like effect, where every kill from high ground allows for another shot.

I can't seem to open my log file, as Notepad keeps crashing. I will try some more to get my log file, but I hope the bug is able to be replicated.

Not really a bug but an interaction between the 2 perks and definitely a balance concern. Neither of the perks are guaranteed to appear and both do fit on a sharpshooter nicely, but I will think about adjusting as in combination they do kinda invalidate the final perk of the tree. Thanks for the heads up.

Hi @h4ilst0rm

It said in the mod description to report non-functioning perk combinations or situations here.

I have a sniper that uses a sniper rifle, but one of his optional perks is Iron Curtain... which says it doesn't work with sniper rifles :(

Shucks!

-gargak <3

@h4ilst0rm

the perk "Cold Blooded" does not seem to work with snipers for some reason (or perhaps squadsight).

Originally posted by GargakOfTheEast:

@h4ilst0rm

the perk "Cold Blooded" does not seem to work with snipers for some reason (or perhaps squadsight).

That would be because Cold Blooded only trigger on "StandardShot" ability whereas shooting a sniper rifle uses "SniperStandardFire" ability. Will see what I can do next time I get around updating the mod.

Oh, @hailst0rm

Another bug, perhaps also a feature? It was really cool when it happened!

I have the GREMLIN auto-retaliate ability too on different soldier, which zaps enemies with chain lightning they get too close to the techspert.

However, I used Skullmine on an ADVENT dork, and the hack succeeded. For some reason, him dying from being skullmined counts as "movement". She zapped his dead body and his nearby friends with chain lightning after a successful skullmine. Wow! What a cool idea for a perk! :D

This "being skullmined to death for some reason counts as moving" is not limited to only that one GREMLIN ability. I had a different soldier trigger an overwatch-like ability during a different skullmining success. No idea why, but that's what happened.

-gargak <3

Originally posted by GargakOfTheEast:

However, I used Skullmine on an ADVENT dork, and the hack succeeded. For some reason, him dying from being skullmined counts as "movement". She zapped his dead body and his nearby friends with chain lightning after a successful skullmine. Wow! What a cool idea for a perk! :D

This is how it works in Chimera Squad for the gremlin bladestorm chain jolt ability when you go and melee someone (which is where the ability was inspired by / taken from). My best guess is these types of abilities only see if an enemy enters their radius and do not check which side is the one actually doing the movement.

The skill that puts you into overwatch after taking a standard shot doesn't appear to work with sniper rifles. If this is intended, the perk should probably mention this as its otherwise just a wasted pick on a sniper.

Originally posted by h4ilst0rm:

That would be because Cold Blooded only trigger on "StandardShot" ability whereas shooting a sniper rifle uses "SniperStandardFire" ability. Will see what I can do next time I get around updating the mod.

Most likely the same situation as this, will see what I can do when I get to it

Sounds odd, and definitely not something we touch here.

Gunslinger spec at Rank[3] has (StatType=eStat_Dodge,StatAmount=1) listed twice

I dont see gunslinger or sentinel as a class. I think I may have some issues with it displaying in GTS and even in skirmish mode. I thoguht I maybe missing some classes. I might have a conflict or I did something. Just troubleshooting here.

Not so much a bug as a feature request, but...

Would it be possible for you to replace the underscore string split in FindMostInvestedSpec with a different method? As I understand it now, specs that have an underscore in their internal name will end up breaking that logic on any classes that roll that spec.

(helping someone who is new to making specs, and just noticed they stepped on this rake without knowing about it)

yeah, the string split is not the greatest way of doing things but being regular soldier classes they are it needed to be done in that roundabout way to match the soldier to specific specializations.
Could try and make my own splitter that understands rogue underscore which has its own downsides, or at least add a comment of the flaw to the tank mod config so its documented in the "guide"

honestly I'm a bit surprised if thats the only place that breaks