Page 2 of 2

Re: When casting display the CL of the spell in combat log

Posted: Tue May 24, 2022 9:29 pm
by mrm3ntalist
EasternCheesE wrote: Tue May 24, 2022 6:32 pm
mrm3ntalist wrote: Tue May 24, 2022 9:41 am Can you test the cl of a build with dragon disciple ? My builds with that prc get 2cl more than they should. I made a post about that months ago and I was I told it was fixed, however my builds still get the bonus cl
This one is fixed in next update, i found the offending code and changed it to working one.
On the main topic, there is no resolution yet as i seek for the way to avoid running through every spell script to check it.
I am glad. I mentioned this almost a year ago, but nothing got done. I am surprised no-one else noticed. BTW the CL message displayed at the log window already helps alot, even if it is not correct for things such as SLAs

Re: When casting display the CL of the spell in combat log

Posted: Wed May 25, 2022 5:21 am
by EasternCheesE
mrm3ntalist wrote: Tue May 24, 2022 9:29 pm
EasternCheesE wrote: Tue May 24, 2022 6:32 pm
mrm3ntalist wrote: Tue May 24, 2022 9:41 am Can you test the cl of a build with dragon disciple ? My builds with that prc get 2cl more than they should. I made a post about that months ago and I was I told it was fixed, however my builds still get the bonus cl
This one is fixed in next update, i found the offending code and changed it to working one.
On the main topic, there is no resolution yet as i seek for the way to avoid running through every spell script to check it.
I am glad. I mentioned this almost a year ago, but nothing got done. I am surprised no-one else noticed. BTW the CL message displayed at the log window already helps alot, even if it is not correct for things such as SLAs
As i checked our commit log, other Dev did actually fix it in july of 2021, but they seem have been too used to python syntactic sugar, so they wrote their code using it which kinda doesn't work in nwnscript :P

Agreed, i found it really useful. Since we already removed that feature from live, i had to manually add it for my local testing setup in order to troubleshoot which exact part was failing. Btw, you correctly identified the source of issue - it was dragon disciple giving 9 progression instead of 7 at lvl 10, because condition block never triggered it's right part since nwnscript doesn't support ( n < Var < n) style of comparisons.

Re: When casting display the CL of the spell in combat log

Posted: Wed May 25, 2022 10:18 am
by Hullack
Image

Penetrating Blasts seems to have a similar bug.

Re: When casting display the CL of the spell in combat log

Posted: Wed May 25, 2022 10:41 am
by Rhifox
Hullack wrote: Wed May 25, 2022 10:18 am Image

Penetrating Blasts seems to have a similar bug.
Not a bug. Penetrating Blasts gives +4 to CL for the purposes of spell penetration.

Although, looks like this value might be getting used on spell properties dependent on caster level rather than just penetrating SR. Hmm.

Re: When casting display the CL of the spell in combat log

Posted: Tue Jun 21, 2022 7:36 pm
by Rinzler
Has this been fixed?

Re: When casting display the CL of the spell in combat log

Posted: Wed Jun 22, 2022 4:08 am
by EasternCheesE
Rinzler wrote: Tue Jun 21, 2022 7:36 pm Has this been fixed?
Not yet. Won't be fixed in next patch as making such thing right would require me to rewrite our spellcasting framework (and it's something i plan to do). So i didn't make some bandaid and just focused on next patch. After it, i will come to this topic.

Re: When casting display the CL of the spell in combat log

Posted: Sat Jul 02, 2022 9:30 am
by mrm3ntalist
Was this removed? I no longer see the CL. It is extremely helpful for the Rcr

Re: When casting display the CL of the spell in combat log

Posted: Sat Jul 02, 2022 11:02 am
by EasternCheesE
Yes, it was removed because proper implementation requires us rewriting our magic system. Once it's done, it will be back.

Re: When casting display the CL of the spell in combat log

Posted: Sat Jul 02, 2022 3:20 pm
by mrm3ntalist
EasternCheesE wrote: Sat Jul 02, 2022 11:02 am Yes, it was removed because proper implementation requires us rewriting our magic system. Once it's done, it will be back.
Thank you