DICE PACKS BUNDLE
Page 12 of 15 First ... 21011121314 ... Last
  1. #111
    Quote Originally Posted by MeAndUnique View Post
    Personally, I do not think that CA and AWDE have hit the point of concluding incompatibility. However, given that CA works on its own and with roughly 100 other extensions of varying capabilities including additions to the modifier stack and dammage application, if I am to be able to address the present conflict in CA I require either more constructive guidance than being advised to rewrite it, or a better understanding of exactly how AWDE deviates from the ruleset.
    AWD interacts with the applyDamage function that you also change, which breaks the 'assumption' in your extension that you can hold the decodedDamageText as static. Because it is not.

    Since applyDamage is calling the decodeDamageText the inputs to applyDamage can be 'adjusted' in extension. Like you do in your extension. I just dont assume that it is 'static' or that no other extension could possibly be in the calling stack of overrides for applyDamage call chain.

    The issue is that your extension 'assumed' that once you had processed 'applyDamage' and then called down into the 'chain' of possible other extensions that could overload this function, that they would not also change something in the call process. ( Which since you wrap this function to apply your own changes, you can not assume other will not also apply changes before it hits the actual ruleset function. ) Your extension is holding the value from your extension level call, and then passing it back into the 5e ruleset version and assumes nothing will ever happen to it or be between you and the ruleset function.

    AWD also scans the damage lines before calling the ruleset applyDamage, and can change the lines. Which makes this a dynamic item. Your extension should not be assuming it is static which it will be if no other extension is in the stack that access/changes the applyDamage function.

    I dont think the suggestion change I've posted will cause you any issue, because AWD does not change the decoded outputType which is what you are holding from the stored value. You could reduce what you store in the local value to just that one item. Then not have caused this conflict.

    The reason I can not fix it in my extension, is because yours sits on top of mine and is pushing back in the stored values. Without changing my load priority I can not resolve it. As far as I know my extension does not break your extra codes as they pass through my extension. But in theory we could have some extra interaction like this. ( Like the 'transfer' damage code I already talked about. )

    I also do not think you should be doing 'extra' work in the 'messageDamage', as people will not be expecting this. I know you do it so you do not have to adjust the applyDamage function in a mid code point, which causes a possible massive conflict point. So I understand why you do this. But I suggest you look to see if you can do this work before the applyDamage or post the applyDamage function. But this would be a more of the re-write I think you should do.

    I do think you should just store the 'decodeResult.sType' which is all you use outside the applyDamage function, then you could store this at the point you 'decodeDamageText' in your applyDamage function, and then totally remove your access to the 'decodeDamageText'. Since you link into messageDamage you can use this to change your sTypeOutput to "maximum hit points'. I've not looked into the 'effectmanage/actiondamage' calls in applyDamage so it might not be possible to remove the decodeDamageText 'sTypeOutput' change.
    Last edited by bratch9; September 16th, 2021 at 20:57.

  2. #112
    I'm not sure if this has been addressed, but when I use this extension, ongoing damage (for example, "DMGO: 1d4 fire") always becomes untyped damage. I've tested this with only this extension on and it still happens. Has there been a fix for this, or has anyone else had this problem?

  3. #113
    Quote Originally Posted by BlazingAzureCrow View Post
    I'm not sure if this has been addressed, but when I use this extension, ongoing damage (for example, "DMGO: 1d4 fire") always becomes untyped damage. I've tested this with only this extension on and it still happens. Has there been a fix for this, or has anyone else had this problem?
    I can replicate this, and will take a look. It should not be causing an issue like this.

    EDIT: I've got a fix for this issue working, but want to look at what I can sort with the CA extension above before doing a new release.
    Last edited by bratch9; October 5th, 2021 at 15:15.

  4. #114
    v2.2 is now uploaded, should fix both CA and DMGO issue.

  5. #115

  6. #116
    The issue is fixed! Glad to see that resolved. I noticed it happening months ago but didn't track down which extension it was until this week. Glad it could be fixed so quickly!

  7. #117
    Whoops, posted in wrong thread. My bad!
    Last edited by RedmondStache; November 7th, 2021 at 19:17. Reason: Posted in wrong thread.

  8. #118
    Quote Originally Posted by RedmondStache View Post
    When I first enabled this extension, it set two characters' max HP to different values.

    A level 5 Artificer with +1 Con should have well over 9 max HP, and a level 5 wizard with +2 Con should have more than 8. The other four characters in the party were untouched, and I'm not sure why the two in question had their values changed.

    Typed the correct values into their HP Max entries, and everything's working well since, but it seems Constitutional Amendments is treating their HP values as "adjusted". Is there a way to change the HP values that would make CA respect them as the correct, normal amounts?
    Not sure on this, Advance Weapon Damage, does not interact with character max HP.... so it changing these on a couple of characters is probably not the extension. ( Might be some odd interaction between extensions. )

    Did you have these characters at 5th level before adding AWD ? ( Or create them with AWD )

    Did you already have CA enabled before AWD ?

    If CA is thinking these are 'adjusted', I'm not familiar with CA to know what mechanism it used to detect this. ( I think some extension I read about, allow for per level dice roll entry and manual entry and 'standard' value, which might be CA.. I seem to recall you could open some part of the character sheet to see what it thought and correct these... ie go back and say for level 2 'I rolled 5' for my HP increase.. )

    When I get chance I'll take a look, but character history and order of when extensions were added etc... would be good information to know about.

    Any extra information you can provide would be good to help re-produce this.

    Thanks, Pete

    [ I assume you posted on CA thread as well ? ]

  9. #119
    Followed a link from the CA thread to see about an incompatibility, wasn't aware I was being taken to a different thread. My bad!

  10. #120
    Quote Originally Posted by RedmondStache View Post
    Followed a link from the CA thread to see about an incompatibility, wasn't aware I was being taken to a different thread. My bad!
    Not an issue, easy done.

    ( The link was probably when CA and AWD was miss calculating. Which should be fixed. So was worried that it might have come back . )

    -pete

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Tags for this Thread

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
STAR TREK 2d20

Log in

Log in