randovania / open-samus-returns-rando

Randomizer patcher for Metroid: Samus Returns
GNU General Public License v3.0
5 stars 1 forks source link

DNA in auto tracker #468

Closed ThanatosGit closed 1 month ago

ThanatosGit commented 1 month ago

@dyceron what are we gonna do with the DNA in the auto tracker? In the game + in the future we will count down from the required DNA to 0. We could also place more than 39 DNA.

Should we make it consistent and also just show the DNA left in the game or are we going to count up from "collected dna / required dna" ? The latter could also mean we are doing nothing until we support placing more than 39 DNA officially.

dyceron commented 1 month ago

Probably easier just keep it counting up. Probably should remove the /39 part though, and also have it not count starting dna.

ThanatosGit commented 1 month ago

Hmmm that's a thing, I don't like. I only did the "count down" instead of "count up" because the number we are counting up is not a fixed value of 39 anymore. Could be the default of 39 but also a greater number. If you count up and remove the /39, or better to say /required_dna part, in the auto tracker, there is no way you could tell from the auto tracker to which value it actually needs to count up to.

dyceron commented 1 month ago

Could the tracker list something like 0/10, where the 10 is the required DNA? I don't think that'd be bad.

ThanatosGit commented 1 month ago

We can read all that from RDV. No idea how hacky the gui stuff will be.

dyceron commented 1 month ago

Then let's do that. It's the most intuitive to me.