ToME: the Tales of Maj'Eyal

Everything about ToME
It is currently Wed Oct 17, 2018 7:53 am

All times are UTC




Post new topic Reply to topic  [ 320 posts ]  Go to page Previous  1 ... 17, 18, 19, 20, 21, 22  Next
Author Message
PostPosted: Sat May 19, 2018 7:47 pm 
Offline
Halfling

Joined: Mon Mar 05, 2012 10:35 pm
Posts: 118
Few odd things I've noticed with the mod now that I'm back to playing.

Eating the Temporal Wyrm Heart gives you Chronomancy/Blink Drake regardless of class. It should probably only do that if you're playing a Wyrmic.

Blink Drake is listed as Chronomancy but isn't in any way.

Blood Drake doesn't act like Corruption at all either.

Given that Scourge Drake and Undead Drake do use Insanity and Souls respectively that doesn't feel right.

Also it looks like you were trying to make Wild Drake, Blood Drake and Blink Drake unlock from birth if you've already gotten them but couldn't get the carryover logic working from what I'm seeing of the Blink Drake and Wild Drake acquisition code (yeah, I went code diving after seeing a Writhing One get Blink Drake).

Anyway nekoarcos figured out multiple ways to do that in Odyssey Of The Summoner.

I'm still trying to parse what he did to make doing certain things enable custom class avatars that then enable alt-modes for classes at birth if selected, but there's a very simple setup he's using to make things from one playthrough modify an event in later ones:

event that sets condition
game:setAllowedBuild("<Whatever>", false)

conditional to see if the event has fired in any playthrough
if profile.mod.allow_build["<Whatever>"] then

It's similar to the standard unlock method used in the base game: Only difference is using false instead of true in the setAllowedBuild and calling a check in gameplay instead of having the birther do it.

Ah, what the heck, I'll do a dummy test in a few minutes and see if it works the way I think it does.

Edit: Yeah, it works the way I think it does.

The false "just" suppresses notification messages.

putting
game:setAllowedBuild(<unlock tag x>, false)

in each of the three talent unlock sequences and changing

getBirthDescriptor("subclass", "Wyrmic").unlockable_talents_types[<unlock x>]={false, 0.3}
to
getBirthDescriptor("subclass", "Wyrmic").unlockable_talents_types[<unlock x>]={false, 0.3, <unlock tag x>}

will almost certainly work and definitely won't crash anything.

Can't say it works with 100% certainty because I can't test the transfer without uploading the modified version of Verdant I used to make sure it doesn't crash anything and that wouldn't be kosher.


Top
 Profile  
 
PostPosted: Mon May 21, 2018 2:06 am 
Offline
Perspiring Physicist

Joined: Sun Sep 09, 2012 7:43 am
Posts: 5759
Its working mostly as intended.
Blink is Chrono and Blood is Corruption to make it absolutely clear that they aren't going to work with antimagic anythings. Making them use paradox/vim just adds semi pointless complexity. (More Willpower boosts your use of Paradox and Equilibrium and using Vim raises Equilibrium anyway.)

They are not intended to be permanent unlocks for future characters. What you saw in the birth file is simply there to grant the categories at a higher mastery level for Wyrmic.

_________________
My feedback meter decays into coding. Give me feedback and I make mods.


Top
 Profile  
 
PostPosted: Mon May 21, 2018 8:38 am 
Offline
Perspiring Physicist

Joined: Sun Sep 09, 2012 7:43 am
Posts: 5759
Also: I'm seriously considering removing the Antimagic Balance mechanic from Druid, because it requires me to overwrite a couple of core game mechanic functions, which breaks things every major game update.

For anyone looking blank at the moment: Antimagic Balance is a tweak to the code which stop arcane disruption applying to any talent tagged as a spell and a wild-gift, eg all Druid talents.
This also allows them to use arcane disrupting equipment without the spellpower penalty or sustains deactivating.

_________________
My feedback meter decays into coding. Give me feedback and I make mods.


Top
 Profile  
 
PostPosted: Mon May 21, 2018 4:25 pm 
Offline
Halfling

Joined: Mon Mar 05, 2012 10:35 pm
Posts: 118
HousePet wrote:
Its working mostly as intended.
Blink is Chrono and Blood is Corruption to make it absolutely clear that they aren't going to work with antimagic anythings. Making them use paradox/vim just adds semi pointless complexity. (More Willpower boosts your use of Paradox and Equilibrium and using Vim raises Equilibrium anyway.)

They are not intended to be permanent unlocks for future characters. What you saw in the birth file is simply there to grant the categories at a higher mastery level for Wyrmic.

Ah.

Was kind of confused by seeing all the cases that can never trigger in that case in the code. I assumed that birth unlock was planned because there wasn't any reason to handle the player already having it if there's no birth unlock and 2/3 of the unlock events exclude Adventurers.

Is it WAD that Blink-Drake is available to everyone but the others aren't?


Top
 Profile  
 
PostPosted: Tue May 22, 2018 12:15 am 
Offline
Perspiring Physicist

Joined: Sun Sep 09, 2012 7:43 am
Posts: 5759
Yeah, its a Dragon Heart which typically grants Harmony or Vile Life from the other ones. So I just allowed the Temporal Heart to grant the category to anyone as an optional sidegrade.

_________________
My feedback meter decays into coding. Give me feedback and I make mods.


Top
 Profile  
 
PostPosted: Tue May 22, 2018 3:03 pm 
Offline
Halfling

Joined: Mon Mar 05, 2012 10:35 pm
Posts: 118
HousePet wrote:
Yeah, its a Dragon Heart which typically grants Harmony or Vile Life from the other ones. So I just allowed the Temporal Heart to grant the category to anyone as an optional sidegrade.

Ah.

Fits the dragon heart pattern, breaks the class ability unlock pattern.

I can't think of any other global class talent unlock. A few unlockable class talents are given to multiple classes, but no globals.

----

Edit: Deleted derpy idea.


Top
 Profile  
 
PostPosted: Fri Jun 01, 2018 1:05 am 
Offline
Perspiring Physicist

Joined: Sun Sep 09, 2012 7:43 am
Posts: 5759
Currently upgrading this addon to work nicely with Cults.
So if anyone has any bugs, suggestions or requests, now is a good time to give them.

_________________
My feedback meter decays into coding. Give me feedback and I make mods.


Top
 Profile  
 
PostPosted: Fri Jun 01, 2018 9:10 am 
Offline
Higher

Joined: Sun Aug 24, 2014 7:09 am
Posts: 69
Location: A Holding Pattern on the Eidolon Plane
Yeah, one issue comes to me for the Woodsman. They have Archery Prowess as a locked category, which is (ever since the Archer rework) built around the use of Marks. The Woodsman has no way I saw to apply those, making those skills significantly less useful (and the first isn't usable at all).


Top
 Profile  
 
PostPosted: Fri Jun 01, 2018 10:07 am 
Offline
Perspiring Physicist

Joined: Sun Sep 09, 2012 7:43 am
Posts: 5759
Oh good point. I need to review which archery categories it has.

_________________
My feedback meter decays into coding. Give me feedback and I make mods.


Top
 Profile  
 
PostPosted: Fri Jun 01, 2018 10:53 pm 
Offline
Uruivellas

Joined: Mon Sep 21, 2015 8:45 pm
Posts: 656
Location: Middle of Nowhere
My biggest beef with this pack has always been Claw Block.


Top
 Profile  
 
PostPosted: Sat Jun 02, 2018 5:22 am 
Offline
Perspiring Physicist

Joined: Sun Sep 09, 2012 7:43 am
Posts: 5759
Yeah I'll see if the code has changed enough to let me do that without overwriting a massive chunk of the combat code.
Otherwise I'll look into changing it to something else.

Edit: The code change I need hasn't happened yet. I'll check if 1.6 will have something useful.

_________________
My feedback meter decays into coding. Give me feedback and I make mods.


Top
 Profile  
 
PostPosted: Sat Jun 02, 2018 10:23 am 
Offline
Higher

Joined: Sun Dec 10, 2017 12:27 am
Posts: 79
HousePet, don't you have an account for the Tome git repository? Wouldn't it make sense to make the change and post a merge request?

Going out on a limb, I suspect you're wanting to do the same thing as I am doing: instantiating repelled, crit, evaded and deflect prior to the hook in attackTargetWith and passing them in? It's an obvious and useful change and I already chatted to Shibari about various code tidying I'd like to contribute over the summer, but don't really have time at the moment to finish the addon I was making, let alone refactor core code.


Top
 Profile  
 
PostPosted: Sat Jun 02, 2018 10:50 am 
Offline
Perspiring Physicist

Joined: Sun Sep 09, 2012 7:43 am
Posts: 5759
I could do it, but I've had changes not get in before. Its frustrating to do code, test it and it not go anywhere.

_________________
My feedback meter decays into coding. Give me feedback and I make mods.


Top
 Profile  
 
PostPosted: Wed Jun 06, 2018 1:43 pm 
Offline
Halfling

Joined: Mon Mar 05, 2012 10:35 pm
Posts: 118
HousePet wrote:
Currently upgrading this addon to work nicely with Cults.
So if anyone has any bugs, suggestions or requests, now is a good time to give them.

Could you do something to make Ember Scales's level 5 trail have IFF like the damage aura does?

If you have it active and bump-switch in town you aggro the person you switched with. It's also unusable on escort missions. Having to turn it off when I got to town, or before level changing on maps that spawn escorts is annoying micro.

Level 5ing a talent should not add micromanagement.


Top
 Profile  
 
PostPosted: Thu Jun 07, 2018 12:46 am 
Offline
Perspiring Physicist

Joined: Sun Sep 09, 2012 7:43 am
Posts: 5759
Probably going to be painful to implement, but I'll give it a go.

_________________
My feedback meter decays into coding. Give me feedback and I make mods.


Top
 Profile  
 
Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 320 posts ]  Go to page Previous  1 ... 17, 18, 19, 20, 21, 22  Next

All times are UTC


Who is online

Users browsing this forum: No registered users and 2 guests


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
Powered by phpBB® Forum Software © phpBB Group