[b41-1.0.5] Talent Point Planner

A place to post your add ons and ideas for them

Moderator: Moderator

Message
Author
Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#106 Post by Zizzo »

v10f.3 fixes (or at least tries to) a persistent annoying bug with reusing "vault" plans from a previous character: they would sometimes show fewer talent points available than the player should have. In particular, I would often load an old point plan and think, "…what? Why does the planner think I have -4 category points available?" This was because the vault plan was mistakenly including certain talents and categories that it shouldn't, like rune/infusion talents and talents conferred by worn objects, and on reload, the planner was inappropriately spending points in them, leaving fewer for the player. I caught some of these ringers in a previous release, but apparently I still missed a lot of them, like talents conferred by tinkers and the internal talent categories used for prodigies (hence dropping from 2 category points to -4…). v10f.3 covers all of those that I can think of; if there are any I'm missing, I'll catch them in a later release… :oops:
"Blessed are the yeeks, for they shall inherit Arda..."

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#107 Post by Zizzo »

…So, I never noticed that (a) you can buy Technique/Combat-training from the Kruk Pride heavy armor store, and (b) the planner doesn't let you choose that. :oops: That's fixed in v10f.4, just pushed out. We Apologize for the Omission.™
"Blessed are the yeeks, for they shall inherit Arda..."

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#108 Post by Zizzo »

Lots of changes in v10g:
  • From our perspective, the new Lich race evolution is a lot easier to handle than the old-style lich handling; it simply provides the Undead/Lich talent tree and a few stat boosts.
  • We also pick up 1.7's new High Thaumaturgist and Fallen evolutions. Not entirely sure how well I'm handling the latter yet, but that can be improved as we go…
  • The Adept prodigy now gives a talent level boost instead of a tree multiplier boost, and we match that. [Actually, that comes for free, since we implement this source by just giving the planner clone the Adept prodigy. :wink: ]
  • Learned from a Steam user that the Fall of Zigur quest grants the Corruption/Hexes tree, so we support that now; it will become visible on the planner dialog as soon as you have the corresponding achievement.
  • Got a bit overzealous filtering out talents in an earlier bugfix; was accidentally filtering out "hidden" talents like Exotic Weapon Mastery, even for players that could see them. :oops: That should be fixed.
  • Something that always vaguely bugged me when restarting a character was not being quite sure where to spend my initial points because I couldn't remember what that character's point plan was. I could always cancel out of the levelup dialog, reload the character's vault plan and go back to spend the points, of course, but that's suboptimal, because talents learned during that initial on-birth dialog don't start in cooldown. Well, now I've got a workaround for that, in the form of new game option "[ZOmnibus] | Open talent point planner on character creation", which, if enabled, will fire up a planner dialog before moving on to the regular levelup dialog. This gives me a chance to reload the character's vault plan or create a new plan for the character before spending my initial talent points.
  • As with all my other addons, we add (at least an attempt at) support for 1.7's new localization infrastructure.
"Blessed are the yeeks, for they shall inherit Arda..."

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#109 Post by Zizzo »

Aaand already pushing out v10g.1 with a bugfix to that new "[ZOmnibus] | Open talent point planner on character creation" option: If you exit the new on-birth planner dialog by pressing the "Cancel" button, it'll jump you ahead to the usual on-birth levelup dialog and regular game start. If you exit the new on-birth planner dialog by typing <Esc>, it… wasn't. :oops:
"Blessed are the yeeks, for they shall inherit Arda..."

Pyros
Higher
Posts: 75
Joined: Mon Sep 14, 2020 12:37 pm

Re: [b41-1.0.5] Talent Point Planner

#110 Post by Pyros »

The fallen support currently doesn't work if you try to open it as a Sun Paladin, just errors out. Copy pasting the screen from someone else on discord: Image

Works fine once you're a fallen though.

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#111 Post by Zizzo »

Pyros wrote:The fallen support currently doesn't work if you try to open it as a Sun Paladin, just errors out.
[sound F/X: source diving] Ah, the Fallen evolution changed some since beta5, and I didn't update for it. Should be fixed in v10g.2, just pushed out.
"Blessed are the yeeks, for they shall inherit Arda..."

Pyros
Higher
Posts: 75
Joined: Mon Sep 14, 2020 12:37 pm

Re: [b41-1.0.5] Talent Point Planner

#112 Post by Pyros »

There's a bug in the current version, the Hex unlock doesn't appear. After digging a bit I'm pretty sure the issue is the addon checks for the achievement but has a typo in the pointplandialog.lua:
return is_campaign("Maj'Eyal") and has_world_achievement('ANTI_ANIMAGIC')

Should be ANTI-ANTIMAGIC then I assume it'll find the achievement and pop up.

Edit: Changed it and it does indeed fix. There's another Animagic right below too return has_achievement('ANTI_ANIMAGIC') that I also edited.

Pisastrish
Thalore
Posts: 157
Joined: Wed Sep 28, 2016 8:07 pm

Re: [b41-1.0.5] Talent Point Planner

#113 Post by Pisastrish »

Could there be a button or option for this that marks all talent sources as seen? I don't know if there is a new bug in 1.7 but the mark as seen option doesn't seem to work at all, and even in the past it seemed kind of hit and miss (plus it's very annoying to enter every single code when I've already seen everything, just not with the addon on)

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#114 Post by Zizzo »

Pyros wrote:There's a bug in the current version, the Hex unlock doesn't appear. After digging a bit I'm pretty sure the issue is the addon checks for the achievement but has a typo in the pointplandialog.lua: return is_campaign("Maj'Eyal") and has_world_achievement('ANTI_ANIMAGIC')
I actually had to stare at that for a minute before I saw the missing T… :oops: Fixed in v10h, just pushed out.
Pisastrish wrote:Could there be a button or option for this that marks all talent sources as seen?
v10h also includes a new "Show all sources" toggle, which does basically what it says. Disclaimer: The internal known() method for talent sources does the work of both "Does the player know about this source?" and "Does this character have access to this source?", so turning on show-everything will also show sources that your character can't get, like Cursed/Cursed Aura for non-Cursed characters. We Apologize for the Inconvenience.™
Pisastrish wrote:I don't know if there is a new bug in 1.7 but the mark as seen option doesn't seem to work at all, and even in the past it seemed kind of hit and miss
? [sound F/X: testing] It seems to work for me. Can you be more specific?
"Blessed are the yeeks, for they shall inherit Arda..."

Pisastrish
Thalore
Posts: 157
Joined: Wed Sep 28, 2016 8:07 pm

Re: [b41-1.0.5] Talent Point Planner

#115 Post by Pisastrish »

Cursed aura and both of the elixirs are the ones I remember not working, but I think there were more

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#116 Post by Zizzo »

Pisastrish wrote:Cursed aura and both of the elixirs are the ones I remember not working, but I think there were more
[sound F/X: source diving] Hmm, I think I may have done some work in the area of those particular sources a while back; I'll investigate.

In the meantime, trying to build a point plan for my golem revealed that I'd badly botched handling of 1.7's new Lich evolution. :oops: :oops: That's fixed in v10h.1, just pushed out.
"Blessed are the yeeks, for they shall inherit Arda..."

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#117 Post by Zizzo »

Inspired by my recent changes to Enhanced Object Compare, v10h.2 is a stab at expanding our ability to make talent point plans for sidekicks like the golem. In theory, any party member that isn't flagged internally as not getting talent points on levelup should be accessible; the main complication was what file name to use for the sidekick's "vault" plan, which we key off the sidekick's "type" in the party member list.
"Blessed are the yeeks, for they shall inherit Arda..."

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#118 Post by Zizzo »

v10h.3 is mostly a compatibility release for use with my new Interact with Sidekicks addon, but the necessary internal re-architecting also bought us some small improvements: the planner now has separate notions of "player knows about this source" and "character can access this source", so the "Show all sources" toggle will now no longer show sources that are inaccessible to your current character (like the Cursed/Cursed Aura tree for non-Afflicted characters).
"Blessed are the yeeks, for they shall inherit Arda..."

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#119 Post by Zizzo »

I knew that major rewrite in the previous version would break stuff… :oops: Here's what I've found so far, fixed in v10h.4:
  • Was only recognizing the Technomancy trees as available in the Embers campaign, rather than simply when the addon was active.
  • Basically any tallent source that's only available to a specific class or class group was misdetecting and showing as unavailable. :oops: :oops:
"Blessed are the yeeks, for they shall inherit Arda..."

Zizzo
Sher'Tul Godslayer
Posts: 2517
Joined: Thu Jan 23, 2003 8:13 pm
Location: A shallow water area south of Bree
Contact:

Re: [b41-1.0.5] Talent Point Planner

#120 Post by Zizzo »

Fired up a new Embers character recently, and found yet another thing that that massive rewrite in v10h.3 broke: the "Technique/Combat-training from the Kruk Pride heavy armor store" source that we added way back in v10f.4 got lost. That's fixed in v10h.6, just pushed out. We Apologize for the Omission.™ :oops:
"Blessed are the yeeks, for they shall inherit Arda..."

Post Reply