DDOCharPlanner / DDOCharPlannerV4

Dungeons and Dragons Online Character Planner
GNU General Public License v3.0
36 stars 12 forks source link

Resetting Trees #25

Closed DougSK closed 7 years ago

DougSK commented 7 years ago

When someone resets a tree (or all trees) in the planner, ideally it should reset from that level upward, not downward. That way, if a build requires changing the spending of AP partway through, it doesn't require two separate savefiles. Currently, no matter at what level you click "Reset Tree" or "Reset All Trees", the tree or trees are completely wiped from L1 up.

jonesth3 commented 7 years ago

I will have to think about that. Current reset works like it does in game. I will have to look, but I think can add a button that will reset the current level and above.

DougSK commented 7 years ago

Can't think of a reason why it shouldn't be the default behavior (in the planner). If you want to add extra buttons for a full reset, I suppose you could, but simply going to L1 Enhancements and doing the reset would take care of it with no additional programming or buttons.

In the game, you're always working with the current state of your character. That's why a reset in-game resets everything. In the planner, you're working with all the levels of your toon at the same time. If you are planning your enhancements level-by-level and you decide to go back a few levels and redo, you most definitely don't want to have to redo from start. And if you do want to redo from start, you're going back to L1 anyway. :)

DougSK commented 7 years ago

However, to have a button that indicates that you are respeccing your toon mid-leveling and allows you to respend ALL your AP without resetting the Enhancement points spent at lower levels.. THAT would be handy.

It's not uncommon to find a build on the forums that requires spending your AP a certain way up to a specified level and then resetting and respending differently (for example, once you have enough levels in your second class to make it worthwhile investing in its trees). If you're not familiar with such, here's one I'm currently playing: https://www.ddo.com/forums/showthread.php/487071-1-30-Reaper-Clonklock?highlight=u32+morninglord+build (The reason I'm bothering to run it through the planner, rather than slavishly following the build is because the build is based on Human and I wanted a Morninglord life. That messes up all the Feat progressions, at a minimum).

It's rather annoying to try to put such a plan in the planner. ATM your only option is to have two savefiles, one that covers up to the point of respec, and another that covers from the respec onward. And some builds respec AP more than once....

DougSK commented 7 years ago

BTW, I can't think of a reason the "Respec" button wouldn't reset all trees. Possible someone would do a respec of a single tree, but I've never seen it in a build plan. Probably rare.

jonesth3 commented 7 years ago

I have just got back from Vacation and have had a chance to look at the program a little more. I think the easiest thing to do will be to add a repec from this level button. This will require the least amount of re programming, add then new feature while leaving the current state of the existing buttons alone for people who currently do use them the way the are. The repec button will do all trees to make it easier to see which trees were changed. Still may need 2 save files as for some builds. I myself have done builds that required you to build in one tree till you reach a set level then respec the tree to spend all the points in a separate tree. I will try to have some thing up for this soon.

DougSK commented 7 years ago

Sound opposite to what would be the most useful (or most commonly used), but takes care of the need. Thanks.

Technical-13 commented 7 years ago

I have thought about the 'respec' process in the past as well because the current implementation isn't ideal. I've thought that any functionality in this direction should include spells as well. I know the gear planner also needs a lot of direction, but thought in that direction on respeccing should be considered as well. I'll add another comment later, just wanted to indicate interest before going in for my dr appointment.

jonesth3 commented 7 years ago

I have looked at the code a little and this definitely will be the preferred method. Otherwise I will have to recode the entire block of code to do it any other way. Some times when working with the program I am stuck with the vision that the original programmer had in mind and have to find ways not to brake the things they have built in the program. As far as the spells I still plan to create a spell specific page where you will be able to select all spells on one page like the feats page. I just have been running short of time to make it happen and still need to work out the best way to make it happen. I did put some mock ups on the forums some time ago. Here is the current layout I have been thinking about. spell screen

Technical-13 commented 7 years ago

I remember seeing that. It is less useful than what I had in mind. The issue with it is that it only shows what the list of spells should be end game, it doesn't allow for indication of which spells are most useful as you level up. Classic case is a sorc that is fire savant up to level 14-16 and switches to air savant. They're spell book is going to vastly change with that respec, and it shouldn't require multiple save-files or forum prints to show it. This is, of course, my opinion; others have mentioned it to me as well, so it's not "just" mine. :)

DougSK commented 7 years ago

I like the spell screen overall. Looks straightforward and easy to use.

DougSK commented 7 years ago

I get what you mean about working within the existing vision for the program. Still, mid-level respecs (feat swaps, enhancement repends, spell changes) are pretty common. It's definitely annoying how many of them require two savefiles for the same toon to plan. Obviously, I have my own vision for what would be best in terms of usability and flexibility. At the same time, I'm only one user and I know nothing about how the planner is coded. Do the best you can. Any improvement will be greatly appreciated.

jonesth3 commented 7 years ago

Currently the planner does nothing for spell planning when it comes to what spells are prepped for a run. It has in the past been only for spells that must be chosen at the time of level selection. I do see the need or should I say desire for the ability to reselect all spells at some time during the characters life. With a sorc that is a time consuming process for sure. one thing I do not have that could be added is a clear all spells button. This would at least speed things up, but with out changing the way the Character builder stores spells completely having multiple spell lists at each level is not in the cards at this time. all thou this screen would allow for that if I got over ammishist at some point in time.

jonesth3 commented 7 years ago

Resetting the trees is now complete and will be in the next update. I am closing this issues even though the spell screen is not done as it was off topic. I hope to get to that soon.