Jump to content

Photo

What's needed for SCS v31?


71 replies to this topic

#16 subtledoctor

subtledoctor
  • Modders
  • 2622 posts
  • Gender:Male

Posted 16 August 2018 - 08:06 AM

Starting off the v30 base, I've included DS v3.95 from RR as well as K4thos' EET compatibility fixes.


Any idea off-hand whether it would be allowed, and how difficult it would be, to make a fork based on kreso's version and add the DS fix to it? (I think it already had EET compatibility.) That version was the gold standard for 2.3, with every fix that I was aware of plus extra SR compatibility, and it was used and thus tested extensively for the last 6-12 months. IMHO going back to v30 and re-implementing the fixes again, and casting aside the SR features, is a mistake.

Faiths & Powers: Spell spheres and kit pack for priests and paladins
Might & Guile: Tweaks and kits for warriors and rogues
Scales of Balance: Game tweaks and rule overhauls
NPC_EE: More options for NPCs in BGEE, SoD, & BG2EE


#17 Cahir

Cahir
  • Members
  • 227 posts
  • Gender:Male
  • Location:Poland

Posted 16 August 2018 - 08:36 AM

How exactly Kreso's version was made when it comes to SR compatibiliy? I mean, is this version valid only when SR is also installed or is this compatibility added on top of standard v30 and one can play Kreso's version without SR too and benefit from all other fixes applied?


Then they summoned me over to join in with them
To the dance of the dead
Into the circle of fire I followed them
Into the middle I was led

#18 CamDawg

CamDawg

    ALL GLORY TO THE HYPNOTOAD

  • Gibberling Poobah
  • 9661 posts
  • Gender:Not Telling

Posted 16 August 2018 - 08:48 AM

It's on Github right now, so fork away. The largest problem--unless I'm misunderstanding--is that Revised only works if SR is installed (or works under that assumption).

 

I'm working through the Revised changes now. So far the majority are to make the Dispelling Screen/Spell Deflection targeting work; most of the rest are the EET compatibility changes that are already included, and it's lacking a couple of the fixes from BW.

 

It can't form the baseline SCS, though. One consistent thing I've been noticing is that it removes a lot of target-has-SI:X-enabled checks, which is a byproduct of SR removing SI.


Why is this Hypnotoad video so popu... ALL GLORY TO THE HYPNOTOAD.


#19 K4thos

K4thos
  • Modders
  • 1186 posts
  • Gender:Male

Posted 16 August 2018 - 09:01 AM

I've noticed on github that the EET related commit modifies shout.baf. It's unrelated to EET. Looks like it's there to fix the problem mentioned in this old topic, which also affected SCS. May be a good idea to review this particular change since there were several patches since 2014 to EE games, maybe it's no longer needed (or could be coded better).


Edited by K4thos, 16 August 2018 - 09:04 AM.


#20 ALIENQuake

ALIENQuake
  • Modders
  • 564 posts
  • Gender:Male
  • Location:Poland

Posted 16 August 2018 - 09:33 AM

Juuuuuust about an hour too late.

It's not a race. Since SCS is very important mod, I will still push my local repository and when you see how nice it's done, I'm sure you will appreciate it  :cool:

 

EDIT: You don't need to revert commit to add tag to older commit.


Edited by ALIENQuake, 16 August 2018 - 10:00 AM.


#21 subtledoctor

subtledoctor
  • Modders
  • 2622 posts
  • Gender:Male

Posted 16 August 2018 - 10:35 AM

How exactly Kreso's version was made when it comes to SR compatibiliy? I mean, is this version valid only when SR is also installed or is this compatibility added on top of standard v30 and one can play Kreso's version without SR too and benefit from all other fixes applied?

The largest problem--unless I'm misunderstanding--is that Revised only works if SR is installed (or works under that assumption).

 

My understanding was that it could be used just fine without SR - lots of people were doing that for several months before the 2.5 patch, and I saw no complaints...
 

It can't form the baseline SCS, though. One consistent thing I've been noticing is that it removes a lot of target-has-SI:X-enabled checks, which is a byproduct of SR removing SI.

 

Huh. Are they RES checks? SR doesn't actually remove the SI:x spells, it just removes the shell spell and moves the effects/subspells to other places (SI:Div moved to Nondetection, SI:Ench added as Mind Blank, SI:Evoc moved to ProEnergy, etc.) So I imagine it should still handle those immunity conditions, perhaps just with different kinds of checks. Unless kreso cut some corners or something. :laugh:

In any event the base SCS does not handle some SR stuff properly, so if the mod's AI coding cannot handle both environments, I guess the question is whether we can still maintain a separate SR-specific fork, and make sure these fixes get into it?


Faiths & Powers: Spell spheres and kit pack for priests and paladins
Might & Guile: Tweaks and kits for warriors and rogues
Scales of Balance: Game tweaks and rule overhauls
NPC_EE: More options for NPCs in BGEE, SoD, & BG2EE


#22 ALIENQuake

ALIENQuake
  • Modders
  • 564 posts
  • Gender:Male
  • Location:Poland

Posted 16 August 2018 - 12:13 PM

This is full list of authors and fixes from BWFixpack:

Spoiler

 

@Mike1072 I know that you wanted "each patch inside one commit" but it's simply too much work. I'm sure that DawidW can review patches via nice diff's. But I added kresos RevisedSCS as separate branch as you wanted. Now, I've pushed my repository: https://github.com/B...CoastStratagems moved to https://github.com/G...CoastStratagems

 

- best practices regarding gitattributes for IE mods

- best practices regarding gitignore for IE mods

- best practices regarding gitconfig from Google engineers

- full SCS history from SCS 22 with tags

- author of the commits is David.Wallace, not myself

- commit dates for SCS version are real dates of the releases

- Fixpack Fixes + list with links and authors

- EET compatibility from K4thos

- RevisedSCS branch with kreso as author

- Fixpack Fixes applied onto RevisedSCS branch because some of them were missing, now it's possible to have a nice comparison of changes

- CamDawg "DS v3.95 from RR, with one minor change" commited as CamDawg

 

Now, this is something which I hope make Mike1072 and DawidW happy.

 

@CamDawg Would you consider takeover this repository and put it into https://github.com/Gibberlings3 ?


Edited by ALIENQuake, 16 August 2018 - 01:58 PM.


#23 Ardanis

Ardanis

    A very GAR character

  • Modders
  • 2601 posts
  • Gender:Male
  • Location:Saint-Petersburg, Russia

Posted 16 August 2018 - 12:22 PM

Turns out, SCS was choking on those files for several people, but they had been corrupted by an earlier mod - RR.  I can't find the thread where the problem was finally diagnosed... it might have been on SHS, which is currently down.  But Wisp figured out that something in DS was failing when it tried to read the information in the 'resource' field in effects in the 2.5 Spell Deflection spells.  And he updated RR.  I think it's in 4.93?  But I only see 4.92 on the RR Github page.  The 4.92 readme says "Updated detectable spells" as of May 19th... but I don't know if that was to fix this problem.  I thought this issue only popped up in the last couple weeks.

Ah. If what he has fixed is what has been causing issues, then it should be already fixed in the version I linked above. I knew about the potential issue, but figured it wouldn't normally happen.


Retired from modding.


#24 CamDawg

CamDawg

    ALL GLORY TO THE HYPNOTOAD

  • Gibberling Poobah
  • 9661 posts
  • Gender:Not Telling

Posted 16 August 2018 - 12:51 PM

 

It can't form the baseline SCS, though. One consistent thing I've been noticing is that it removes a lot of target-has-SI:X-enabled checks, which is a byproduct of SR removing SI.

 
Huh. Are they RES checks? SR doesn't actually remove the SI:x spells, it just removes the shell spell and moves the effects/subspells to other places (SI:Div moved to Nondetection, SI:Ench added as Mind Blank, SI:Evoc moved to ProEnergy, etc.) So I imagine it should still handle those immunity conditions, perhaps just with different kinds of checks. Unless kreso cut some corners or something. :laugh:

In any event the base SCS does not handle some SR stuff properly, so if the mod's AI coding cannot handle both environments, I guess the question is whether we can still maintain a separate SR-specific fork, and make sure these fixes get into it?

 


It's in the SSL itself, so you'd have to look at the end product. E.g. here are some examples from shadel.ssl (the Shade Lord scripting):
 

//////////////////////////////////////////////////////////////////////
////	Finger of Death
//////////////////////////////////////////////////////////////////////

IF TRIGGER
	ConditionalTargetBlock(PCsPreferringStrong;!CheckStatGT(scstarget,50,RESISTMAGIC))
	TargetBlock(EnemiesInOrderShort)
	TriggerBlock(MR100|SpellTurn|Slay|SINecromancy|Enemy)
	TriggerBlock(DirectDamageSafe)
THEN DO
	Action(Power,WIZARD_FINGER_OF_DEATH,finger,24|150|50)
END

//////////////////////////////////////////////////////////////////////
////	Orbs
//////////////////////////////////////////////////////////////////////


IF TRIGGER
	ConditionalTargetBlock(PCsPreferringWeak;!CheckStatGT(scstarget,50,RESISTMAGIC))
	TargetBlock(EnemiesInOrderShort)
	TriggerBlock(DirectDamageSafe)
	TriggerBlock(MR100|SIEvocation|MinorGlobe|SpellTurn|Enemy|L1safe)
THEN DO
	Action(Power,WIZARD_CHROMATIC_ORB,orb,6)
END

In Revised the SIEvocation and SINecromancy are removed from the TriggerBlocks, among other changes. I believe this falls in line with kreso's description of the 'dispelling screen', where SI-style spells would soak up one spell and then go away. A target having an SI (or its equivalent) is no longer a don't-waste-your-time casting in SR/SCSR because you will, at least, knock out the protection. However these castings are unequivocally wasted without the SI check in a non-SR game.

 

This is full list of authors and fixes from BWFixpack:

Spoiler

 

@Mike1072 I know that you wanted "each patch inside one commit" but it's simply too much work. I'm sure that DawidW can review patches via nice diff's. But I added kresos RevisedSCS as separate branch as you wanted. Now, I've pushed my repository: https://github.com/B...CoastStratagems

 

- best practices regarding gitattributes for IE mods

- best practices regarding gitignore for IE mods

- best practices regarding gitconfig from Google engineers

- full SCS history from SCS 22 with tags

- author of the commits is David.Wallace, not myself

- commit dates for SCS version are real dates of the releases

- Fixpack Fixes + list with links and authors

- EET compatibility from K4thos

- RevisedSCS branch with kreso as author

- Fixpack Fixes applied onto RevisedSCS branch because some of them were missing, now it's possible to have a nice comparison of changes

- CamDawg "DS v3.95 from RR, with one minor change" commited as CamDawg

 

Now, this is something which I hope make Mike1072 and DawidW happy.

 

@CamDawg Would you consider takeover this repository and put it into https://github.com/Gibberlings3 ?

Holy crap, I could have saved myself hours of work. Thank you Alien, I'd be delighted to--you've set this up better than I could.


Why is this Hypnotoad video so popu... ALL GLORY TO THE HYPNOTOAD.


#25 ALIENQuake

ALIENQuake
  • Modders
  • 564 posts
  • Gender:Male
  • Location:Poland

Posted 16 August 2018 - 02:23 PM

So the new repository link is: https://github.com/G...CoastStratagems   :thumbsup: I've just installed all SCS components into BG2EE 2.5 and no errors occur  :thumbsup:


Edited by ALIENQuake, 16 August 2018 - 02:25 PM.


#26 Ardanis

Ardanis

    A very GAR character

  • Modders
  • 2601 posts
  • Gender:Male
  • Location:Saint-Petersburg, Russia

Posted 16 August 2018 - 02:51 PM

author of the commits is David.Wallace, not myself

 

Github says "DS v3.95 from RR, with one minor change" is commited by "dawidw", but searching for users only shows one such person from Poland, Poznan. Did David change his name, or github is playing tricks on me?


Edited by Ardanis, 16 August 2018 - 02:54 PM.

Retired from modding.


#27 ALIENQuake

ALIENQuake
  • Modders
  • 564 posts
  • Gender:Male
  • Location:Poland

Posted 16 August 2018 - 03:02 PM

 

author of the commits is David.Wallace, not myself

 

Github says "DS v3.95 from RR, with one minor change" is commited by "dawidw", but searching for users only shows one such person from Poland, Poznan. Did David change his name, or github is playing tricks on me?

 

It's one oversight, will fix. Fixed. It has to do with the fact that the AUTHOR of the commit and the person who commits can be two different person in git.


Edited by ALIENQuake, 16 August 2018 - 03:08 PM.


#28 CamDawg

CamDawg

    ALL GLORY TO THE HYPNOTOAD

  • Gibberling Poobah
  • 9661 posts
  • Gender:Not Telling

Posted 16 August 2018 - 04:10 PM

I'm going to package up the current repo and send it out into the wild as a release candidate. There's only so much testing I can do alone for a 130-component mod, much less someone else's. :)

 

Again, big thanks to Alien for the repo, and all the folks who contributed the fixes.


Why is this Hypnotoad video so popu... ALL GLORY TO THE HYPNOTOAD.


#29 DavidW

DavidW
  • Gibberlings
  • 4421 posts
  • Gender:Male

Posted 16 August 2018 - 10:53 PM

At some point I'm going to need someone to explain to me how I release my own updated version in this Github framework ... as always I'm very grateful to people who release updates when I'm away, but when (if?) I get out from under work and toddler I'd prefer to use my own previous build as the starting point rather than build on others' code. (Among other issues, I would *strongly* prefer to handle SR compatibility through RequireBlock(Demivrgvs) in the SSL rather than through a separate fork.)

 

Not, to put it mildly, an urgent issue! And many thanks to CamDawg and others for doing a v31 in my absence.



#30 ALIENQuake

ALIENQuake
  • Modders
  • 564 posts
  • Gender:Male
  • Location:Poland

Posted 17 August 2018 - 03:12 AM

I've noticed on github that the EET related commit modifies shout.baf. It's unrelated to EET. Looks like it's there to fix the problem mentioned in this old topic, which also affected SCS. May be a good idea to review this particular change since there were several patches since 2014 to EE games, maybe it's no longer needed (or could be coded better).

Link to commit for shout.baf is not working anymore. How we can compared https://github.com/G...p/baf/shout.baf with the  shout.baf from you topic?. You posted two files, while SCS has only one and I'm assuming that it apply the same code changes for BG1EE and BG2EE. 





Reply to this topic



  


0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users