jastey

Hi Guest mindrot, please post the contents of your weidu.log here inside a spoiler.

39 minutes ago, The_Baffled_King said:

(1) The number of top-level states is not really a problem.

At least i didn't run into one so far, and never heard of any. As I said, If you look at some talkative NPC mods I would assume that's more dialogue states (also with triggers) than you'd need for your BG1 - flexible - story mod idea. One thing I don't know is if you'd make the order inside the dlg arbitrary and WEIGHT all dialogue states, whether that would have any influence. What I mean is that most NPC mods just give the dialogue states unique triggers so the engine searches from top to bottom until it finds the true one. Maybe it takes longer if the whole file needs to be checked because there are WEIGHTed states. But then, I guess the engine would have to check the whole dlg always to find WEIGHted states, so it probably looks through the whole file anyway. - I'm not a good source for these kind of questions, though. I'm rather ignorant with regard to which actions get processed first or instantly or whatever, as long as it works in the game.

44 minutes ago, The_Baffled_King said:

(2) The problem didn't come from the number of states at all (ie. going from Top-level state > state > state > state > etc down through a large number of states.

(3) The problem came from having a lot of reply options attached to a single state.

(4) The problem in particular came from having a lot of reply options with different response triggers attached to one state.

Is that accurate?

Exactly. If you have a look at Ajantis' PID in BG1NPC (bg1npc/pid/x#ajpcfl.d) and imagine all those different states combined to (almost) one (I think I separated between flirt and engaged), you get an idea about the scope of what made the classic engine hickup.

28 minutes ago, The_Baffled_King said:

149 state triggers in BG1 NPC AJANTJ - including many single triggers with over 10 lines. Puts things in perspective a bit.

Are you concerned with dialogue states per dlg or with reply options in one state? For the former I don't see a problem (I don't think any dlg hit a wall and some mod NPCs have a lot to say). For the latter - I did recode Ajantis' PID because before hand it was one state and then all reply options with different conditions lead to the dialogue hang for a second or two before it opened - that was the old engine (Tutu), though. Split it up in several dialogue states sorted by conditions still having a lot of triggered reply options made it work fine.

Thank you very much for this!

8 hours ago, The_Baffled_King said:

Are there circumstances where one method of making a journal entry should be preferred over the other method?

UNSOLVED_JOURNAL works in a dialogue state (I put it usually at the end before the EXIT but it can also be used in different order with the other actions I think). It does not work in  a CHAIN as a local NPC specific transaction, i.e. if you put a

DO ~transactions~

behind the line of the one NPC talking, so here you'd need to use AddJournalEntry().

8 hours ago, The_Baffled_King said:

What about in cases where multiple journal entries are made and/or erased in the same transition?

UNSOLVED_JOURNAL only adds one/works only for one. If you have mutliple, you need to use AddJournalEntry().

Congratulations on the release!

How do you realize lore bonus for reading a book? As far as I know it's not possible to detect whether the player had a look at an item description, so I am curious.

bg1ub updates to v16.4!
v16.4 comes with an updated Spanish translation by OmegaZeroVIII and some improvements.

Changelog:

- Changed syntax of Kivan's interjection with Tazok to make it more stable (I_C_T instead of I_C_T2).
- Spanish version updated and completed by OmegaZeroVIII.
- Corrected entry for "Beregost_House08_L2" in cpmvars.tpas.
- Readme updated.

I like the idea.

What assassins? You mean as a reply option? What do you want Duke Eltan to reply?

Cheat your way into the city and talk to the characters, then you'll get an idea. My experience is that wihtout changes, all character inside the city are talking as if the bandit thread is over.

If there is answers to updates this quickly I ususlly expect a "does not install" bug report.

Thanks for the feedback!

Most recent changelog see last post!

BG1NPC updates to v26 with bugfixes. Thank you to Coland (Italian) and Arkie (Russian) for updating the language versions to the v26 changes!

Version 26 also comes with a new component by @Angel !
Thank you very much to Angel for providing this component. Now the legacy spear quest reward for Kivan makes more sense if Kivan can have weapon pips in spears instead of halberd in BG:EE!

Description from the readme:
The BG1 NPC Project: Kivan's Proficiency

This component will give a choice to alter Kivan's weapon proficiency pips to either halberds or spears. This is interesting insofar, as the quest item Kivan receives from Thalatyr is a spear, whereas he comes with pips in halberd in BG:EE.

Option one: Kivan uses spears. This option will give Kivan 2 pips in spears, and also replace two halberds in teh game with spears (one in the cave in Dryad Falls, another in the Cloakwood Mines level 3.

Option two: Kivan uses halberds. This option will give Kivan 2 pips in halberds.

Changelog v26:

- New component: "Kivan's Proficiency Tweak", by Angel
- Kivan should count Gerde's ankheg kills correctly.
- Kivan should not reference Prism in Mutamin's garden if he didn't meet Prism.
- Ajantis should not ask Shoal to revive the companion if they didn't die.
- Faldorn should correctly banter with Kagain.
- Halbazzer's dialogue for upgrading the armor should not block his dialogue.
- "Read Gorion's letter" will also trigger in Candlekeep Catacombs.
- "KnowBhaal" variable will only set after letter was read aloud if Gorion's letter is in party's inventory.
- Journal entry for Garrick's quest should close properly for all language versions.
- EET: Tana'ri should be copied to override.
- Compatibility with Shades of the Sword Coast: Bentley will only react to Eldoth if he can see him.
- revised Ajantis' shield colors and BAMs from Sir Kill.
- corrected entry for "Beregost_House08_L2" in cmpvars.tpa.
- Russian version now uses setup.tra out of English language folder.
- added check to forbid installation after EET_End.
- Updated bg1npc.ini with regard to install order syntax.
- Typo corrections.

The mod updates to v7.1. All language versions are complete again.

Polish: by memory

Russian: by Arkie

Thank you very much for the translations!

I'm especially fond of polite anonymous guest posters. Kudos.

Hahaha! Sounds like a plan!

@Lauriel sorry! I thought you are firm with it, lol. Making a pull request is easy if you know how. Until I got there I found it completely un-intuitive, though. I don't know who designed the routine but it is *not* intuitive. (Sorry guys).

@Laurielfeel free to make pull requests. It's probably the fastest way and also makes sure the link goes to where you want it.

1 hour ago, Guest Xantro said:

But after entering in Duke Palace, all Flaming Fists, Belt and Liia attack me and my PC die without reasons after few seconds (even if I come back to the street).

This is a known issue of (at least) the older mod versions for the 2.6 patch. Before entering the Palace after killing Sarevok set per cheat:

C:SetGlobal("SarevokBehavior","GLOBAL",5)

The mod updates to v7 with bugfixes!

Changelog:

-Ophyllis should reply correctly after Imoen's interjection
-Imoen should start her correct dialogue at the Sewer exit
-more interjections in SoD
-more lines altered from "searching for Imoen" to "going after Irenicus"
-typo corrections
-added UnderstandMouseMagic to credits
-Internal restructure: texts will be copied to "autotra" folder before used in game (and converting them to utf8 for EE) and using marker file so text conversion is done only once; Russian version now uses English setup.tra directly.
-added check to tp2 to forbid installation after EET_End
-corrected entry for "Beregost_House08_L2" on cpmvars.tpas

Correction to the above: the line @28 changed in x#ajint.tra (not x#ajantis.tra)!

Grey the Dog updates to v6 with bugfixes and improvements!

Changelog:

-traified and translated all lines for PID
-fixed text format error for EE (non-English versions)
-"7th party members" for some item checks considered
-corrected entry for "Beregost_House08_L2" in cpmvars.tpas
-resttalk crossmod with Brandock should also fire if Brandock is "7th party member"
-more SoD "interjections" (woof!)
-journal entry about Brage's gauntlets should be removed when Brage's quest is finished
-improved transition to BGII
-added check to forbit install of mod after EET_End
-changed internal structure so temporary "autotra" foldr is used for text formatting
-typo corrections