| Anonymous | Login | Signup for a new account | 05-25-2013 02:31 MDT | ![]() |
| All Projects | HeroForge | ModernForge | MonsterForge | SpellForge |
| Main | My View | View Issues | Change Log | Roadmap |
| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update | |
| 0000230 | HeroForge | Bug | public | 11-17-2009 10:24 | 06-05-2010 16:44 | |
| Reporter | Rainer Åhlfors | |||||
| Priority | normal | Severity | minor | Reproducibility | sometimes | |
| Status | closed | Resolution | fixed | |||
| Platform | OS | OS Version | ||||
| Product Version | ||||||
| Target Version | 6.1.0.1 | Fixed in Version | 6.1.0.1 | |||
| Summary | 0000230: Gestalt Problem (and a feature request) | |||||
| Description | First, 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. | |||||
| Tags | No 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 |