HeroForge Bug Tracking Page
Mantis Bug Tracker


View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000230HeroForgeBugpublic11-17-2009 10:2406-05-2010 16:44
ReporterRainer Åhlfors 
PrioritynormalSeverityminorReproducibilitysometimes
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version6.1.0.1Fixed in Version6.1.0.1 
Summary0000230: Gestalt Problem (and a feature request)
DescriptionFirst, the problem. Sometimes a gestalt character will gain +2 BAB at a given level.

Next, the request (which is more like a pseudo bug). Currently a 9th level gestalt might be listed as Ftr/Rog/Wiz/Rgr (with levels 5/6/3/4) ... when, in reality, the classes should be listed as either:

a) Ftr/Rgr//Rog/Wiz (with levels 5/4//6/3)
b) Ftr/Rgr||Rog/Wiz (with levels 5/4||6/3)

Personally I prefer the latter (with || as the gestalt divider). I already have this change in my personal copy and can easily update the dev copy accordingly. It's a simple copy-paste.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0000557)
Rainer Åhlfors (manager)
11-17-2009 10:24

The BAB bug has been fixed already.
(0000563)
Christophe Kowalski (developer)
11-17-2009 13:52

|| is perhaps a clearer separator than //. The construction of Class could also be improved with the use of rangeconcat().
(0000564)
Rainer Åhlfors (manager)
11-17-2009 14:14
edited on: 11-17-2009 15:00

On the subject on clarity -- that was my thought as well when I implemented it in my own sheet.

Construction of class is slightly different for gestalt and cannot use the present system. Currently, a Ftr||Rog -> Mnk||Rog -> Ftr||Mnk will simply be listed as Ftr2/Rog2/Mnk2 when it should be Ftr2/Mnk1||Rog2/Mnk1. But, I digress ...

So, support for proper gestalt class listing from you at least?

(0000565)
Christophe Kowalski (developer)
11-17-2009 15:00

Definitely. That might require splitting column F (lvls) in gestalt/non-gestalt. Btw, column F somehow duplicates column E (most PCs live at level <20, isn't it?) so re-injecting E in F would reduce the number of expensive counts by a factor 4: F=E+if(hitdice>20,countif(class,lvls 21-60))
(0000566)
Rainer Åhlfors (manager)
11-17-2009 15:07

Column F ... on which sheet are you talking? "Class Info" I presume ...

On my sheet I put this on "Class Info Aux". This isn't reducing the number of expensive counts (at all). But, this can definitely be expanded to handle gestalt/non-gestalt characters.

I am tempted to get it working first, improve later.
(0000567)
Rainer Åhlfors (manager)
11-17-2009 15:09

What does "Au" in "LvlsAu20" stand for, by the way?
(0000568)
Christophe Kowalski (developer)
11-18-2009 01:07

Don't know about Au, and yes, I was talking about Class Info.

Class Info Aux has a hybrid structure, and I think most of its upper part (the 60 top rows) could be moved to Classes (and eventually Class Info for a few items, as well as Skills for the skill points logic). The nice 60 levels vertical structure is already present on Classes with the necessary data, namely level, primary and gestalt class index. HD calculation logic would also fit nicely there, since this is precisely where it is used.
Class Info Aux would then be dedicated to Familiar, Mounts and Companions (an deventually relabelled.)
(0000648)
Christophe Kowalski (developer)
12-02-2009 08:46

Out of curiosity, I accidentally found out about this "Au": it's the French word for at.
(0000706)
Shawn Perry (administrator)
01-06-2010 16:12

Objections with putting this back to 6.2?
(0001054)
Christophe Kowalski (developer)
05-04-2010 14:50

As I was touching the class list anyway, I took the liberty to implement this request, following the /||/ model. The whole class list logic has been moved from class info aux to class info, and split into primary and secondary classes for a proper handling of gestalt.

Inserted 7 columns on Class Info (X:AD)

Ex-rule:
Xn default to FALSE, TRUE if condition or ex-class fulilled (e.g. =(LEFT(Alignment;6)="Lawful") or barbarian)

Primary classes:
Yn==IF($AAn=0;"";IF($Xn;"Ex-";"")&$Bn)
Zn=IF($AAn=0;"";IF($Xb;"Ex-";"")&$Cn)
AAn=COUNTIF(TblClassesAtLvlCS;$An)

Secondary classes:
ABn==IF($ADn=0;"";IF($Xn;"Ex-";"")&$Bn)
ACn=IF($ADn=0;"";IF($Xn;"Ex-";"")&$Cn)
ADn=COUNTIF(TblClassesAtLvlCS;$An)

ClassList=RANGECONCAT("/";TRUE;$Y$5:$Y$726)&IF(IsGestalt;"||"&RANGECONCAT("/";TRUE;$AB$5:$AB$726);"")
AbrList=RANGECONCAT("/";TRUE;$Z$5:$Z$726)&IF(IsGestalt;"||"&RANGECONCAT("/";TRUE;$AC$5:$AC$726);"")
LvlList={RANGECONCAT("/";TRUE;IF(AA5:AA726=0;"";AA5:AA726))&IF(IsGestalt;"||"&RANGECONCAT("/";TRUE;IF(AD5:AD726=0;"";AD5:AD726));"")}
ClassLvlList={RANGECONCAT("/";TRUE;IF($AA$5:$AA$726=0;"";$Z$5:$Z$726&$AA$5:$AA$726))&IF(IsGestalt;"||"&RANGECONCAT("/";TRUE;IF($AD$5:$AD$726=0;"";$AC$5:$AC$726&$AD$5:$AD$726));"")}


- Issue History
Date Modified Username Field Change
11-17-2009 10:24 Rainer Åhlfors New Issue
11-17-2009 10:24 Rainer Åhlfors Status new => assigned
11-17-2009 10:24 Rainer Åhlfors Assigned To => Rainer Åhlfors
11-17-2009 10:24 Rainer Åhlfors Note Added: 0000557
11-17-2009 12:41 Rainer Åhlfors Status assigned => feedback
11-17-2009 13:52 Christophe Kowalski Note Added: 0000563
11-17-2009 14:14 Rainer Åhlfors Note Added: 0000564
11-17-2009 15:00 Christophe Kowalski Note Added: 0000565
11-17-2009 15:00 Rainer Åhlfors Note Edited: 0000564
11-17-2009 15:07 Rainer Åhlfors Note Added: 0000566
11-17-2009 15:09 Rainer Åhlfors Note Added: 0000567
11-18-2009 01:07 Christophe Kowalski Note Added: 0000568
12-02-2009 08:46 Christophe Kowalski Note Added: 0000648
01-06-2010 16:12 Shawn Perry Note Added: 0000706
01-06-2010 16:12 Shawn Perry Severity major => minor
01-17-2010 18:23 Shawn Perry Target Version 6.1 => Next Release Topics
04-09-2010 11:58 Shawn Perry Target Version Next Release Topics => 6.1.0.1
05-04-2010 14:50 Christophe Kowalski Note Added: 0001054
05-04-2010 14:50 Christophe Kowalski Status feedback => resolved
05-04-2010 14:50 Christophe Kowalski Resolution open => fixed
05-04-2010 14:50 Christophe Kowalski Fixed in Version => 6.1.0.1
06-05-2010 16:44 Yonner Status resolved => closed


MantisBT 1.2.0[^]
HeroForge and the HeroForge Logo are Trademarks of HeroForge Software, LLC - Copyright 2008 - 2009 HeroForge Software, LLC. All rights reserved.
MantisBT - Copyright © 2000 - 2010 MantisBT Group
Powered by Mantis Bugtracker