char quick restart with changed options

Collapse
X
 
  • Time
  • Show
Clear All
new posts
  • PowerDiver
    Prophet
    • Mar 2008
    • 2820

    char quick restart with changed options

    If you change birth options, such as ironman, the quick restart uses the same option as the previous char, ignoring the change to birth options.

    If this is the right way to do things, I think the birth options ought to be listed on the screen when it asks if you want to replay the character.

    I kind of think it should use the new options even though it is a restart. If I go through the process of getting my race/class/stats just right, then realize too late that I needed to change a birth option, I'd still like to be able to change, suicide, and quickstart. However, I'm not sure, and it makes sense the way it is.
  • Magnate
    Angband Devteam member
    • May 2007
    • 5110

    #2
    Originally posted by PowerDiver
    If you change birth options, such as ironman, the quick restart uses the same option as the previous char, ignoring the change to birth options.

    If this is the right way to do things, I think the birth options ought to be listed on the screen when it asks if you want to replay the character.

    I kind of think it should use the new options even though it is a restart. If I go through the process of getting my race/class/stats just right, then realize too late that I needed to change a birth option, I'd still like to be able to change, suicide, and quickstart. However, I'm not sure, and it makes sense the way it is.
    You are correct that quickstart retains the previous character's "adult" options and therefore ignores any changes to birth options. I am not 100% certain whether this is the best approach, but there is no dilemma because I lack sufficient knowledge of the birth code to change it. One thing I do know from working on this is that non-maximise mode is the one to watch. I spent hours debugging why non-maximise wasn't working in various permutations of quickstart/normal start and changed/unchanged. It's ok now, but if we allow changed birth options to take effect on quickstart characters, we have to make sure that one works too.

    If you want to have a go at this, CMD_FINALIZE_OPTIONS is the thing you need to insert at some point during the quickstart process.

    The alternative - listing the options on the quickstart screen - is difficult due to lack of space. You could have an additional input choice - as well as Y, N, C you could have O to view birth options. That would require a new birth command action I think, since you'd just be displaying them and not changing them. But if you're going to go that far, you may as well allow changing them.
    "Been away so long I hardly knew the place, gee it's good to be back home" - The Beatles

    Comment

    • Nomad
      Knight
      • Sep 2010
      • 958

      #3
      Originally posted by Magnate
      You could have an additional input choice - as well as Y, N, C you could have O to view birth options. That would require a new birth command action I think, since you'd just be displaying them and not changing them. But if you're going to go that far, you may as well allow changing them.
      I'd definitely like to be able to press '=' on the "New character based on previous one" screen and change options from there. When I reload a save file, I usually want to keep all the race/class/stat choices but would like to be able to check/change whether I've got randarts on, and it's a pain in the butt to have to hit start afresh and re-enter all of that stuff just to get a look at the options.

      Comment

      • Magnate
        Angband Devteam member
        • May 2007
        • 5110

        #4
        Originally posted by Nomad
        I'd definitely like to be able to press '=' on the "New character based on previous one" screen and change options from there. When I reload a save file, I usually want to keep all the race/class/stat choices but would like to be able to check/change whether I've got randarts on, and it's a pain in the butt to have to hit start afresh and re-enter all of that stuff just to get a look at the options.
        I'd be up for that, and I'm sure takkaria wouldn't mind, as long as it didn't break anything. Patches welcome - or a pull request on github will do instead.

        I have opened ticket #1259 for this. The problem is going to be recalculating the stats if maximise mode is changed: when you first see the quickstart character, the stats are based on the assumption that birth_maximize is unchanged. If you change this option, not only will the base stats change, but the sensible spending of points will also change.
        "Been away so long I hardly knew the place, gee it's good to be back home" - The Beatles

        Comment

        • PowerDiver
          Prophet
          • Mar 2008
          • 2820

          #5
          Perhaps the mistake is setting birth options from the options menu.

          The idea of changing an option so that things won't change now, but will next game, is pretty strange.

          Suppose they were only set at birth, with a screen coming up during normal birth where you can change them. Then they could not change before a quickstart, and there would be nothing to talk about. Also, the double variables could be switched to single variables.

          Comment

          • Magnate
            Angband Devteam member
            • May 2007
            • 5110

            #6
            Originally posted by PowerDiver
            Perhaps the mistake is setting birth options from the options menu.

            The idea of changing an option so that things won't change now, but will next game, is pretty strange.

            Suppose they were only set at birth, with a screen coming up during normal birth where you can change them. Then they could not change before a quickstart, and there would be nothing to talk about. Also, the double variables could be switched to single variables.
            Yes, that's pretty good too - and probably a good simplification.
            "Been away so long I hardly knew the place, gee it's good to be back home" - The Beatles

            Comment

            Working...
            😀
            😂
            🥰
            😘
            🤢
            😎
            😞
            😡
            👍
            👎