109 replies to this topic

[Roxanne]

I have converted Chloe for EET and I am currently testing it. I may add ToB contents at a later stage.

 

In case nobody else publishes anything with regards to the mod, it will be a matter of minutes to make it BG2EE and EET compatible in the end.

 

I would concentrate on contents/additional contents but not on improving her kit.

I always liked her and my own EET NPC goes along well with her (and had a lot of crossmod contents with her from old BGT that I do not want to loose). This way I have a *selfish* reason to see her again in the EE world.

 

Problem will be translations for new, revised/updated text, but first things first.

 

I initially had not planned to publish the EET version, rather support somebody who wants to keep her alive. But if there is nobody doing it, my version would be as good as anything else and at a minimum be maintaining the original content and making it useable in EE.

[Roxanne]

Progress report and invitation to test

 

Like told here http://www.shsforums...e/#entry591034, I have started to convert Chloe for EET (hopefully also useable for BG2EE as a by-product).

 

This is what is done until now

 

Spoiler

Version 1.6 EE
------------------------------------
Base EET version
 Mod adopted by Roxanne@SHS
- compatibility with BG2EE and EET
- Revised personal items useable only by Chloe
- Repaired 1st meeting cuts
- New portrait for Chloe
- Journal adaptation for EE
- Kit adapted to EE
- Personal weapons repaired
- minor bug fixes + removal of unused code
- Imoen banters fixed for EET
- Weidu setup updated to 2.40

Extra
 New contents (currently only available in English)
- additional SoA banters
- Chloe works for Lathander against the beholder cult and may gain the temple stronghold on behalf of the party (if Player does not qualify in the first place)
- ToB contents (to be expanded)

 

Part one is to maintain and adapt the original mod

Part two is to develop new content that is in the spirit of the original mod (hopefully) - this part does not have translation.

 

I do the development in EET context, the installation has the diverging files for BG2EE and will detect the correct game.

 

I am now looking primarily for testers to look that the BG2EE is working.

 

For those interested, the mod is here

https://github.com/R...hive/master.zip

Github allows constant updates of the latest tested stuff I have, means that all detected defiecencies can be corrected very fast.

PS - Github download has a master-folder, to install the mod, you extract the zip to somewhere on your computer, then copy only the setup.exe and the "Chloe" folder into your game directory and run the setup.

Edited by Roxanne, 24 November 2016 - 02:26 PM.

[agb1]

Will you support this version also for original BG2 + BGT?

[Roxanne]

Will you support this version also for original BG2 + BGT?

No.

 

There is the fully functioning and developed version 1.5 for the original game. I have played it several times in the BGT context myself.(Any known issues are in the fixpack for those games.)

 

It was made available to any interested modder over three years ago (http://www.shsforums...oe/#entry559633) without anyone picking it up.

 

So the issue is in upgrading it for EET (where Chloe also has a lot of interaction with my own NPC). The item upgrades etc are not even backwards compatible with the old game due to the use of new opcodes.

 

So, it will be status quo for the old game and my adaptation for the EE. (Like already done for a number of other mods that are no longer maintained by the original author).

 

PS - I split the mod installation into two parts

1. install converted original contents.

2. install new contents (optional)

Edited by Roxanne, 08 February 2018 - 11:25 AM.

[agb1]

Understood.  Chloe NPC version 1.6 EE has been added to BiG World Setup's BG2EE/EET selection with a link to this thread; BG2/BGT will still use version 1.5.

[Roxanne]

Understood.  Chloe NPC version 1.6 EE has been added to BiG World Setup's BG2EE/EET selection with a link to this thread; BG2/BGT will still use version 1.5.

NO NO NO !!!!!!!!!!!!!!!!!!!!

 

I am now looking primarily for testers to look that the BG2EE is working.

 

This was just an invitation to test what is already there! The mod is still worked on and not even tested at all in BG2EE.

It will get its proper mod-page, readme etc once the work is fished.

[Argent77]

I've done a quick test install in vanilla BG2EE and found two issues:

1. A spelling error in Chloe.tp2 at line 44:
COMPILE ~Chloe/EEPlain/CChloeDia.d~ -> COMPILE ~Chloe/EEPlain/ChloeDia.d~

2. The mod complains about an error in DecCR.bat where it doesn't find the label "sounds_add". I would remove the batch files completely. Enhanced Edition games support Ogg Vorbis encoded sound files natively. Simply rename the file extension from .ogg to .wav.

[Roxanne]

I've done a quick test install in vanilla BG2EE and found two issues:

1. A spelling error in Chloe.tp2 at line 44:
COMPILE ~Chloe/EEPlain/CChloeDia.d~ -> COMPILE ~Chloe/EEPlain/ChloeDia.d~

2. The mod complains about an error in DecCR.bat where it doesn't find the label "sounds_add". I would remove the batch files completely. Enhanced Edition games support Ogg Vorbis encoded sound files natively. Simply rename the file extension from .ogg to .wav.

Thanks for that.

Strangely enough I did not see the sound error in EET. And the files work fine.

 

Means to rename them to wav and just copy them into override?

Edited by Roxanne, 22 November 2016 - 02:25 PM.

[agb1]

Understood.  Chloe NPC version 1.6 EE has been added to BiG World Setup's BG2EE/EET selection with a link to this thread; BG2/BGT will still use version 1.5.

NO NO NO !!!!!!!!!!!!!!!!!!!!

 

I've marked it Expert.  Assuming you continue to use GitHub, it will be easy to transition the link and download once you declare it fit for general use.  If you don't want it there at all, I can remove it, but I don't see the harm?  To clarify, Chloe was not available for EE before, so nothing changes except people who use BWS will now see a red Chloe NPC for SoA/ToB line and if they select it they will get a big warning saying beware - this might break your game.

Edited by agb1, 22 November 2016 - 02:28 PM.

[Argent77]

It would be great if you could integrate support for "Skip Chateau Irenicus" directly into the mod. That way you would have full control over the dungeon skipping process regarding Chloe.

Spoiler

Add this code somewhere into Chloe.tp2:

ACTION_IF (DIRECTORY_EXISTS ~SkipChateauIrenicus/npc~) BEGIN
  COPY ~%MOD_FOLDER%/Addons/chloe_sci.tph~ ~SkipChateauIrenicus/npc/chloe.tph~
END

 

The addon file itself ("Chloe/Addons/chloe_sci.tph"):

ACTION_IF (FILE_EXISTS_IN_GAME ~RCHLOE.DLG~) BEGIN
  OUTER_SET num_npc_actions = 1

  OUTER_TEXT_SPRINT npc_script_0 ~AR0709.BCS~

  OUTER_TEXT_SPRINT npc_trigger_0
    ~!InParty("RChloe")
     !BeenInParty("RChloe")
     Global("MeetingChloeAreaThingiesCR","GLOBAL",0)~

  OUTER_TEXT_SPRINT npc_action_0
    ~SetGlobal("MeetingFirstTalkCR","GLOBAL",3)
     SetGlobal("FakeMercenaryCR","GLOBAL",2)
     SetGlobal("FirstTimeIJoinCR","GLOBAL",1)
     MoveGlobal("AR0709","RChloe",[480.445])
     GiveItemCreate("CHLOSW1","RChloe",0,0,0)
     ActionOverride("RChloe",DestroyItem("MISC4C"))
     ActionOverride("RChloe",FillSlot(SLOT_WEAPON))
     ActionOverride("RChloe",EquipMostDamagingMelee())
     ActionOverride("RChloe",Face(S))~

  // prevent intro talk outside of Irenicus dungeon
<<<<<<<< .../inlined/chloe_patch.d
  ADD_STATE_TRIGGER ~rchloe~ 0 ~AreaCheck("AR0601")~ 1 UNLESS ~AreaCheck("AR0601")~
>>>>>>>>
  COMPILE ~.../inlined/chloe_patch.d~

  COPY_EXISTING ~ar0602.bcs~ ~override~
    DECOMPILE_AND_PATCH BEGIN
      REPLACE_TEXTUALLY ~\(ActionOverride("RChloe",SetDialog\(ue\)?("RChloe"))\)~ ~ActionOverride("RChloe",MakeGlobal()) \1~
    END
  BUT_ONLY
END

You can of course modify the file to your heart's content.

 

 

Means to rename them to wav and just copy them into override?

Correct. You can rename them directly in the copy process (e.g. COPY ~%MOD_FOLDER%/Sounds/chloe0.ogg~ ~override/chloe0.wav~). And that's it.

Edited by Argent77, 22 November 2016 - 02:32 PM.

[Roxanne]

It would be great if you could integrate support for "Skip Chateau Irenicus" directly into the mod. That way you would have full control over the dungeon skipping process regarding Chloe.

Spoiler

Add this code somewhere into Chloe.tp2:

ACTION_IF (DIRECTORY_EXISTS ~SkipChateauIrenicus/npc~) BEGIN
  COPY ~%MOD_FOLDER%/Addons/chloe_sci.tph~ ~SkipChateauIrenicus/npc/chloe.tph~
END

The addon file itself ("Chloe/Addons/chloe_sci.tph"):

ACTION_IF (FILE_EXISTS_IN_GAME ~RCHLOE.DLG~) BEGIN
  OUTER_SET num_npc_actions = 1

  OUTER_TEXT_SPRINT npc_script_0 ~AR0709.BCS~

  OUTER_TEXT_SPRINT npc_trigger_0
    ~!InParty("RChloe")
     !BeenInParty("RChloe")
     Global("MeetingChloeAreaThingiesCR","GLOBAL",0)~

  OUTER_TEXT_SPRINT npc_action_0
    ~SetGlobal("MeetingFirstTalkCR","GLOBAL",3)
     SetGlobal("FakeMercenaryCR","GLOBAL",2)
     SetGlobal("FirstTimeIJoinCR","GLOBAL",1)
     MoveGlobal("AR0709","RChloe",[480.445])
     GiveItemCreate("CHLOSW1","RChloe",0,0,0)
     ActionOverride("RChloe",DestroyItem("MISC4C"))
     ActionOverride("RChloe",FillSlot(SLOT_WEAPON))
     ActionOverride("RChloe",EquipMostDamagingMelee())
     ActionOverride("RChloe",Face(S))~

  // prevent intro talk outside of Irenicus dungeon
<<<<<<<< .../inlined/chloe_patch.d
  ADD_STATE_TRIGGER ~rchloe~ 0 ~AreaCheck("AR0601")~ 1 UNLESS ~AreaCheck("AR0601")~
>>>>>>>>
  COMPILE ~.../inlined/chloe_patch.d~

  COPY_EXISTING ~ar0602.bcs~ ~override~
    DECOMPILE_AND_PATCH BEGIN
      REPLACE_TEXTUALLY ~\(ActionOverride("RChloe",SetDialog\(ue\)?("RChloe"))\)~ ~ActionOverride("RChloe",MakeGlobal()) \1~
    END
  BUT_ONLY
END

You can of course modify the file to your heart's content.

Would that not require to install Chloe after your skip? I would think that such tweaks would be installed after the NPCs? (or at least the directory must already be in the game when Chloe is installed).

 

What I do at this moment is just to evaluate a global and if that is not present later on in the game it would tell that she was not in CI. I have done this because even with the skip mod installed, the player may decide on different ways through the dungeon after all.

Edited by Roxanne, 22 November 2016 - 02:38 PM.

[Argent77]

It would be great if you could integrate support for "Skip Chateau Irenicus" directly into the mod. That way you would have full control over the dungeon skipping process regarding Chloe.

Spoiler

Add this code somewhere into Chloe.tp2:

ACTION_IF (DIRECTORY_EXISTS ~SkipChateauIrenicus/npc~) BEGIN
  COPY ~%MOD_FOLDER%/Addons/chloe_sci.tph~ ~SkipChateauIrenicus/npc/chloe.tph~
END

The addon file itself ("Chloe/Addons/chloe_sci.tph"):

ACTION_IF (FILE_EXISTS_IN_GAME ~RCHLOE.DLG~) BEGIN
  OUTER_SET num_npc_actions = 1

  OUTER_TEXT_SPRINT npc_script_0 ~AR0709.BCS~

  OUTER_TEXT_SPRINT npc_trigger_0
    ~!InParty("RChloe")
     !BeenInParty("RChloe")
     Global("MeetingChloeAreaThingiesCR","GLOBAL",0)~

  OUTER_TEXT_SPRINT npc_action_0
    ~SetGlobal("MeetingFirstTalkCR","GLOBAL",3)
     SetGlobal("FakeMercenaryCR","GLOBAL",2)
     SetGlobal("FirstTimeIJoinCR","GLOBAL",1)
     MoveGlobal("AR0709","RChloe",[480.445])
     GiveItemCreate("CHLOSW1","RChloe",0,0,0)
     ActionOverride("RChloe",DestroyItem("MISC4C"))
     ActionOverride("RChloe",FillSlot(SLOT_WEAPON))
     ActionOverride("RChloe",EquipMostDamagingMelee())
     ActionOverride("RChloe",Face(S))~

  // prevent intro talk outside of Irenicus dungeon
<<<<<<<< .../inlined/chloe_patch.d
  ADD_STATE_TRIGGER ~rchloe~ 0 ~AreaCheck("AR0601")~ 1 UNLESS ~AreaCheck("AR0601")~
>>>>>>>>
  COMPILE ~.../inlined/chloe_patch.d~

  COPY_EXISTING ~ar0602.bcs~ ~override~
    DECOMPILE_AND_PATCH BEGIN
      REPLACE_TEXTUALLY ~\(ActionOverride("RChloe",SetDialog\(ue\)?("RChloe"))\)~ ~ActionOverride("RChloe",MakeGlobal()) \1~
    END
  BUT_ONLY
END

You can of course modify the file to your heart's content.

Would that not require to install Chloe after your skip? I would think that such tweaks would be installed after the NPCs? (or at least the directory must already be in the game when Chloe is installed).

Not at all. This code simply places the chloe.tph into the respective subfolder of "Skip Chateau Irenicus" where it is automatically processed when that mod is installed.

[agb1]

Can you replace the audio .bat files with this?

// convert .ogg files to .wav
LAF HANDLE_AUDIO
  STR_VAR
    audio_path = ~mymod/sounds~
    oggdec_path = ~mymod/sounds~
    //sox_path = ~mymod/sounds~
END
// convert .ogg files to .acm (BGEE) or .wav (other games)
LAF HANDLE_AUDIO
  INT_VAR
    music = 1 // if this variable is not 0 and the game is BG:EE, the audio files are given the extension .acm instead of .wav
  STR_VAR
    audio_path = ~mymod/music~
    oggdec_path = ~mymod/sounds~
    //sox_path = ~mymod/sounds~
END

 
The mod isn't supported for BGEE anyway, so you could get rid of the second block by moving the music files into the sounds folder.  Note that either way HANDLE_AUDIO only runs oggdec on .ogg files, so the .crm files in the music folder would need to be renamed to .ogg.

Edited by agb1, 22 November 2016 - 02:48 PM.

[Roxanne]

It would be great if you could integrate support for "Skip Chateau Irenicus" directly into the mod. That way you would have full control over the dungeon skipping process regarding Chloe.

Spoiler

Add this code somewhere into Chloe.tp2:

ACTION_IF (DIRECTORY_EXISTS ~SkipChateauIrenicus/npc~) BEGIN
  COPY ~%MOD_FOLDER%/Addons/chloe_sci.tph~ ~SkipChateauIrenicus/npc/chloe.tph~
END

The addon file itself ("Chloe/Addons/chloe_sci.tph"):

ACTION_IF (FILE_EXISTS_IN_GAME ~RCHLOE.DLG~) BEGIN
  OUTER_SET num_npc_actions = 1

  OUTER_TEXT_SPRINT npc_script_0 ~AR0709.BCS~

  OUTER_TEXT_SPRINT npc_trigger_0
    ~!InParty("RChloe")
     !BeenInParty("RChloe")
     Global("MeetingChloeAreaThingiesCR","GLOBAL",0)~

  OUTER_TEXT_SPRINT npc_action_0
    ~SetGlobal("MeetingFirstTalkCR","GLOBAL",3)
     SetGlobal("FakeMercenaryCR","GLOBAL",2)
     SetGlobal("FirstTimeIJoinCR","GLOBAL",1)
     MoveGlobal("AR0709","RChloe",[480.445])
     GiveItemCreate("CHLOSW1","RChloe",0,0,0)
     ActionOverride("RChloe",DestroyItem("MISC4C"))
     ActionOverride("RChloe",FillSlot(SLOT_WEAPON))
     ActionOverride("RChloe",EquipMostDamagingMelee())
     ActionOverride("RChloe",Face(S))~

  // prevent intro talk outside of Irenicus dungeon
<<<<<<<< .../inlined/chloe_patch.d
  ADD_STATE_TRIGGER ~rchloe~ 0 ~AreaCheck("AR0601")~ 1 UNLESS ~AreaCheck("AR0601")~
>>>>>>>>
  COMPILE ~.../inlined/chloe_patch.d~

  COPY_EXISTING ~ar0602.bcs~ ~override~
    DECOMPILE_AND_PATCH BEGIN
      REPLACE_TEXTUALLY ~\(ActionOverride("RChloe",SetDialog\(ue\)?("RChloe"))\)~ ~ActionOverride("RChloe",MakeGlobal()) \1~
    END
  BUT_ONLY
END

You can of course modify the file to your heart's content.

Would that not require to install Chloe after your skip? I would think that such tweaks would be installed after the NPCs? (or at least the directory must already be in the game when Chloe is installed).

Not at all. This code simply places the chloe.tph into the respective subfolder of "Skip Chateau Irenicus" where it is automatically processed when that mod is installed.

Maybe I did not express my concern correctly.

You probably are anticipating an install where the user first unpacks all the mods into the game directory and then runs the install (like BWS or BWP would do). I am talking of the case when there is no respective subfolder of "Skip Chateau Irenicus" by the time Chloe is installed. (Like I do it due to non-existing BWS support for EET).

[Roxanne]

Can you replace the audio .bat files with this?

// convert .ogg files to .wav
LAF HANDLE_AUDIO
  STR_VAR
    audio_path = ~mymod/sounds~
    oggdec_path = ~mymod/sounds~
    //sox_path = ~mymod/sounds~
END
// convert .ogg files to .acm (BGEE) or .wav (other games)
LAF HANDLE_AUDIO
  INT_VAR
    music = 1 // if this variable is not 0 and the game is BG:EE, the audio files are given the extension .acm instead of .wav
  STR_VAR
    audio_path = ~mymod/music~
    oggdec_path = ~mymod/sounds~
    //sox_path = ~mymod/sounds~
END

 
The mod isn't supported for BGEE anyway, so you could get rid of the second block by moving the music files into the sounds folder.  Note that either way HANDLE_AUDIO only runs oggdec on .ogg files, so the .crm files in the music folder would need to be renamed to .ogg.

I found

COPY ~Chloe/Sounds~ ~override~

easier - after I converted to ogg to wav.

[agb1]

You probably are anticipating an install where the user first unpacks all the mods into the game directory and then runs the install (like BWS or BWP would do). I am talking of the case when there is no respective subfolder of "Skip Chateau Irenicus" by the time Chloe is installed. (Like I do it due to non-existing BWS support for EET).

 

 

That problem could be solved by placing the SkipChateauIrenicus tph files in the override folder instead; they would do nothing if the mod isn't installed and could be retrieved in the SCI mod itself using a directory-file-regexp like this:

 

ACTION_BASH_FOR ~override~ ~^.*_sci.tph$~ BEGIN

  INCLUDE ~%BASH_FOR_FILESPEC%~ // import *_sci.tph files

  DELETE ~%BASH_FOR_FILESPEC%~ // clean up after loading

END

Edited by agb1, 22 November 2016 - 02:57 PM.

[Argent77]

Yeah, "Skip Chateau Irenicus" (SCI) has to be unpacked before Chloe is installed. I'd guess the majority of players are unpacking all their mods into the game folder before starting the mod installation process.

Alternatively you can simply change the code into

ACTION_IF (NOT DIRECTORY_EXISTS ~SkipChateauIrenicus/npc~) BEGIN
  MKDIR ~SkipChateauIrenicus/npc~
END
COPY ~%MOD_FOLDER%/Addons/chloe_sci.tph~ ~SkipChateauIrenicus/npc/chloe.tph~

so that players who are unpacking SCI later will still be able to use it. (It shouldn't matter either way for players who don't use SCI at all.)

[Roxanne]

Yeah, "Skip Chateau Irenicus" (SCI) has to be unpacked before Chloe is installed. I'd guess the majority of players are unpacking all their mods into the game folder before starting the mod installation process.

Alternatively you can simply change the code into

ACTION_IF (NOT DIRECTORY_EXISTS ~SkipChateauIrenicus/npc~) BEGIN
  MKDIR ~SkipChateauIrenicus/npc~
END
COPY ~%MOD_FOLDER%/Addons/chloe_sci.tph~ ~SkipChateauIrenicus/npc/chloe.tph~

so that players who are unpacking SCI later will still be able to use it. (It shouldn't matter either way for players who don't use SCI at all.)

OK, even though this method would cope for all cases, I will use the one proposed initially - I think it is not adequate for one mod to create another mod's directory for the player if the player does not install the second mod by own choice.

I will use your code but I let Chloe keep the statue, i,e  this line will be deleted

ActionOverride("RChloe",DestroyItem("MISC4C"))

Thus I will have a story trigger that she was in the dungeon but missed the PC at that time. There are too many references to her knowing Irenicus, the dungeon and what is inside in the mod itself, so she must have been there herself.

 

This should solve it for all of us.

[Argent77]

OK, even though this method would cope for all cases, I will use the one proposed initially - I think it is not adequate for one mod to create another mod's directory for the player if the player does not install the second mod by own choice.
I will use your code but I let Chloe keep the statue, i,e this line will be deleted
ActionOverride("RChloe",DestroyItem("MISC4C"))
Thus I will have a story trigger that she was in the dungeon but missed the PC at that time. There are too many references to her knowing Irenicus, the dungeon and what is inside in the mod itself, so she must have been there herself.

This should solve it for all of us.

Thanks. I'll remove the original file from "Skip Chateau Irenicus" in the next release.

Edited by Argent77, 23 November 2016 - 02:48 AM.