IR v4 Beta (last update 20 June 2017)

[subtledoctor]

Actually I can quite easily split up the changes into 10 different commits; only the last one will be multi-issue.  The biggest impediment to multiple single-issue pull requests is, I just don't know how to do it.  If done sequentially, wouldn't each one also include all of the changes from the earlier commits?  I'm just not practiced with Github.

I'll do 10 commits in my fork, and then a single multi-issue pull request, and you should be easily able to refer to the commits to see which files are changed for each issue.  (In most cases it is only one or two files.)

EDIT - okay, I actually made 12 different single-issue commits (actually 13 - I uploaded "4b10sd4" twice), you can inspect them over here.

Some of these are questionable - the second 4b10sd4 sets cold iron weapons per kreso's comment; 4b10sd5 also sets that flag for weapons with the 'starmetal' property - in this case, the "World's Edge" 2-handed sword and the Starmetal Cudgel if that spell is installed by SR or IWDification.  I like the idea that you can hit the loup garou with something other than bastard swords, but you may see this as scope creep.  At any rate it's only a few lines in main_component.tpa, feel free to ignore it.

EDIT - I forgot, I also added the flag to the “Root of the Problem” club, since it talks specifically about affecting unnatural creatures and “monsters.”  So to recap, if these changes are accepted, then the effective weapons will include a dagger, a long sword, a bastard sword, a 2-handed sword, a club usable by druids, and a mace summoned by a cleric spell from another mod. Personally I think this is reasonable; YMMV.

The change to the Bracers of Speed - honestly I'm not convinced there is a bug there.  The .eff uses opcode 146 with param2 = 1 (cast instantly, ignore level), which I think is the proper way to do it, and the subspell has casting speed 0.  I suspect the bug reporter might have had some other mod change one of those.  Maybe.  This change just sets the header target on the subspell to 7, on EEs only (I don't know if header target 7 works on the old engine), just to be sure it is cast instantly.  It may not be needed, but anyway the 'fix' shouldn't cause any harm.

Everything else is per the earlier discussion in the bugs thread.  For Daystar I didn't want to mess with the vanilla game's EVILDAM2.EFF file, even if it may be broken.  So I created a new HOLYDAM2.EFF, modeled after the several other HOLYDAM_.EFF files added by SR, and applied it in the sword's existing 177 equipping effect.

EDIT - these HOLYDAM .eff files might be worth a discussion. Currently it adds a global equipping effect giving the wielder a damage bonus vs. evil creatures, via opcode 179. This means if you have Daystar or Foebane in your off-hand, that bonus will likely apply to your main-hand attacks with a different weapon. (I don’t know how damage is typed for opcode 179... presumably it is the same as the underlying attack?) My first attempt at a fix for Daystar was to add a 177 on-hit effect targeting evil creatures, applying an .eff that uses opcode 12 to do 2 slashing damage. This is what I’m using in my current game, and I could make an argument at all of IR’s categorical damage bonuses (‘holy’ and ‘starmetal’ etc.) should work this way. OTOH it would be a not-insignificant amount of work, and wouldn’t make that much difference...

Edited May 22, 2020 by subtledoctor

[AL|EN]

2 hours ago, subtledoctor said:

Actually I can quite easily split up the changes into 10 different commits; only the last one will be multi-issue.  The biggest impediment to multiple single-issue pull requests is, I just don't know how to do it.  If done sequentially, wouldn't each one also include all of the changes from the earlier commits?  I'm just not practiced with Github.

Yes. If you would like to send multiple changes that don't include previously commited changes (commit 1 - only fix for item1, commit 2 - only fix for item2, without item1 changes) then some amount of git/github trickery with branches would be needed.

But you did good job with single commits.

[Bartimaeus]

On 5/22/2020 at 9:19 AM, subtledoctor said:

EDIT - these HOLYDAM .eff files might be worth a discussion. Currently it adds a global equipping effect giving the wielder a damage bonus vs. evil creatures, via opcode 179. This means if you have Daystar or Foebane in your off-hand, that bonus will likely apply to your main-hand attacks with a different weapon. (I don’t know how damage is typed for opcode 179... presumably it is the same as the underlying attack?) My first attempt at a fix for Daystar was to add a 177 on-hit effect targeting evil creatures, applying an .eff that uses opcode 12 to do 2 slashing damage. This is what I’m using in my current game, and I could make an argument at all of IR’s categorical damage bonuses (‘holy’ and ‘starmetal’ etc.) should work this way. OTOH it would be a not-insignificant amount of work, and wouldn’t make that much difference...

Yeah, I've been aware of this issue for years, but I was never quite sure if I wanted to fix it. The result is if you are dual-wielding, you get those damage/THAC0 bonuses with it, which is definitely not intended. However, my problem with fixing it is:

1. You can't make the same fix for THAC0(?).

2. More message log spam...a few weapons will three message entries every time they attack their target creatures (e.g. Flametongue).

3. A lot of work to fix.

4. There may be an issue of damage resistances for flat damage reductions outright negating a bonus because it's a separate source of damage that gets its own separate reduction (not sure how many creatures/characters have flat damage reductions in the BG games, though).

5. Maybe something else.

P.S. I like giving Root of the Problem the cold iron property. These are, as best as I can tell, the only silver/cold iron-enabled weapons in vanilla:

Flame Tongue = Silver
Sword of Balduran = Silver + Cold Iron
Kondar = Silver + Cold Iron
Albruin = Silver
Sword of Flame = Silver + Cold Iron
Angurvadal = Silver + Cold Iron
The Answerer = Silver + Cold Iron
Carsomyr = Silver + Cold Iron
Werebane = Silver + Cold Iron
Azurewrath = Silver + Cold Iron
Silver Sword = Silver

Oddly, Icingdeath mentions being made of silver, but is not silver-enabled.

Edited May 23, 2020 by Bartimaeus

[hardric]

Sorry for the noob question cause I am very new to this mod.

Potions of Genius and the one that gave a +3 to Intelligence were removed. In other words, it's important to have a mage that has 17+ Intelligence to scribe Level 9 arcane spells (with some Intelligence bonus from Tomes and items)?

[Guest Piquero]

4 hours ago, hardric said:

Sorry for the noob question cause I am very new to this mod.

Potions of Genius and the one that gave a +3 to Intelligence were removed. In other words, it's important to have a mage that has 17+ Intelligence to scribe Level 9 arcane spells (with some Intelligence bonus from Tomes and items)?

The designer of the mod certainly plays with 100% success chance to scribe scrolls and unlimited space in the spell book tweaks (there is also no way of increasing your WISDOM through potions, and there are a few situations in the game where you could use such resource),

You can install that tweak in your game or revert the change to the potion of mind focusing by deleting the .itm file from your Override folder I think.

[subtledoctor]

I suspect Demi just accepted the risk of spell learning failure. This is after all a D&D game, it’s all about rolling dice, and sometimes the dice go against you!

On 5/23/2020 at 4:51 PM, Bartimaeus said:

I like giving Root of the Problem the cold iron property. These are, as best as I can tell, the only silver/cold iron-enabled weapons in vanilla:

Flame Tongue = Silver
Sword of Balduran = Silver + Cold Iron
Kondar = Silver + Cold Iron
Albruin = Silver
Sword of Flame = Silver + Cold Iron
Angurvadal = Silver + Cold Iron
The Answerer = Silver + Cold Iron
Carsomyr = Silver + Cold Iron
Werebane = Silver + Cold Iron
Azurewrath = Silver + Cold Iron
Silver Sword = Silver

1) Does the “silver” property have any effect in the game?

2) Do the BG2 weapons matter? Are there any monsters in BG2 that require cold iron to hit?

[subtledoctor]

Oh yeah, I think I just found another relatively serious bug, in the interaction between Store Revisions and the new SR. Maybe. If I confirm it, I’ll make one more pull request with a fix. 

[hardric]

2 hours ago, Guest Piquero said:

The designer of the mod certainly plays with 100% success chance to scribe scrolls and unlimited space in the spell book tweaks (there is also no way of increasing your WISDOM through potions, and there are a few situations in the game where you could use such resource),

You can install that tweak in your game or revert the change to the potion of mind focusing by deleting the .itm file from your Override folder I think.

Thanks. Now to figure out how to get back the Potion of Mind Focusing.

[subtledoctor]

33 minutes ago, hardric said:

Thanks. Now to figure out how to get back the Potion of Mind Focusing.

Just navigate into your override folder, find potn37.itm, and delete it or move it somewhere else.

That will remove IR's Potion of Restoration - you'll need the spell or temple services for restoration.

[Chitown Willie]

Subtledoctor,

I noticed that you updated your Item Revisions to "4b10sd13".

Sorry if this has been asked, but how does your IR differ from the Gibberlings3 version - and is it intended to be used with IR Revised?

Thanks!

[subtledoctor]

I've added a 14th commit (unfortunately labeled "sd13" since apparently me no count good), fixing how Store Revisions handles temple services, to account for the fact that SR now uses ADD_SPELL.  Apparently since the pull request with the first 13 commits has not yet been merged, this new commit has automatically added to the pull request.  I think?  @Mike1072 if you have any questions or the way Github handles it is not as clean as it seems, just let me know.

[Chitown Willie]

So latest / greatest combo =

Subtledoctor Item Revisions 4b10sd13 + IR Revised v1.2.7?

[subtledoctor]

27 minutes ago, Chitown Willie said:

So latest / greatest combo =

Subtledoctor Item Revisions 4b10sd13 + IR Revised v1.2.7?

I would only mix IRR with whatever IR foundation Bartimaeus has tested it with.  Honestly I'm not sure how it works, but I would worry that something I've changed in IR might be different from what IRR expects and something might be screwy.  I assume when IR4b11 is released Bartimaeus will incorporate that into his project, but keep in mind, not all of my changes might be accepted into IR 4b11!  My general advice:

• If you want IR, use the latest version of IR. 
• If you want to use my fork of IR to get these fixes until 4b11 is released, that's okay with me.
• If you want IRR, use the latest version of IRR, however it was formulated and whatever its instructions are. 

Edited May 29, 2020 by subtledoctor

[Bartimaeus]

There should be no incompatibility between subtledoctor's b11 and IRR, but also, there's (at least currently, from the changes I just reviewed that he did on github) no difference at all between using b10 + IRR and using b11 + IRR. Though I should note that there's currently...two fixes from his b11 that I should and will integrate into IRR shortly, they will be overwritten once you put IRR on top of IR and wouldn't take effect anyways - I have to mirror them in IRR for it to take effect. Once/if b11 is released, I'll increment the version number as well.

[pochesun]

Any ETA on b11?