aymericingargiola / repentance-run-tracker

The Binding Of Isaac: Repentance run tracker app
MIT License
17 stars 1 forks source link

Highest Winstreak Display #54

Closed Shaniquaniminiquani closed 1 year ago

Shaniquaniminiquani commented 1 year ago

Have the highest winstreak save per rule and then display this pb next to the rule's current win streak. Example: image

Basically the app would remember the highest streak for the specific characters and bosses assigned to a rule and then display a pb accordingly when that specific rule is used again.

Ie. If I do a tLost to mother and beast streak, it will show a pb of 1 even though I've had a streak of 20 on isaac to the chest for instance and vice versa

Ps. It took me like 20 minutes to know what to title this lmao

aymericingargiola commented 1 year ago

Yes, checking this !

aymericingargiola commented 1 year ago

Update done to last release, i let you try https://github.com/aymericingargiola/repentance-run-tracker/releases/tag/v0.7.16

(I changed icon for PB to crown and current to flag, makes more sense, same for the run killer with a crown instead of a flag)

Shaniquaniminiquani commented 1 year ago

image Have only gotten a chance to try it once, will continue trying for each char since I need to do mother and the beast with every char in this save file but I don't think it's working? Maybe I'm doing something wrong but think it should show at least 1 here

(I changed icon for PB to crown and current to flag, makes more sense, same for the run killer with a crown instead of a flag)

This makes a lot of sense actually. Good change 👍

aymericingargiola commented 1 year ago

Have only gotten a chance to try it once, will continue trying for each char since I need to do mother and the beast with every char in this save file but I don't think it's working? Maybe I'm doing something wrong but think it should show at least 1 here

Oh it should works actually ! Can you give me your actual winStreaks.json and runs.json (or your last backup in the backup folder at the same place, it's compressed) ? (i see in the second game you had a character switch, Isaac, maybe i should update the code to see it as a win)

Shaniquaniminiquani commented 1 year ago

i see in the second game you had a character switch, Isaac

Yeah I saw that too, don't really remember changing characters at all but maybe it was a modded interaction I'm not familiar with. No clue.

Github was saying they didn't accept json files for upload for some reason but here is my latest backup which should be the same haha: backup_datas_12-04-22-024848PM_checked.zip

Ps: Have since tried the same with cain and the number did go up this time. It probably was the glitched transformation. The ironic thing is, that in the cain run I actually did transform. I transformed into dark judas but the app didn't track it haha. Was confused on why cain didn't have a mother mark image

aymericingargiola commented 1 year ago

@Shaniquaniminiquani

I see an other bug i just corrected, impossible to delete a winstreak 😨

Also how was you able to add an other rule on the same save ? (it will conflict, what are you exactly trying to do ? You can select multiple characters on one streak rule)

Also that's strange it says your run isn't over with Cain, mod issue ?

I still continue to check the issue with this Magdaleine streak

Edit :

Oh... The Beast boss floor is 13.1 but it only records 13.0 (Home, so dogma), anyway i'll change The Beast boss floor for 13.0 because if you win the game it means you beat Dogma AND The Beast

aymericingargiola commented 1 year ago

I made an update, you should remove the Cain winstreak, then click on edit and save on Magdalene winstreak to update, let me know !

Shaniquaniminiquani commented 1 year ago

Also how was you able to add an other rule on the same save ?

I wasn't originally allowed to so I decided to delete the previous mag rule to see if it worked on cain, but when I closed and restarted the app to give you the latest backup it pulled up both instead (Mag was already deleted) haha. Must be a bug.

Also that's strange it says your run isn't over with Cain, mod issue ?

Its weird, I actually finished the run as dark judas like I said and the app didn't recognize it. It could be a mod issue, but I'm unsure. The weird thing is that it counted as a win towards the rule but I do see that the run is still ongoing on it

aymericingargiola commented 1 year ago

I wasn't originally allowed to so I decided to delete the previous mag rule to see if it worked on cain, but when I closed and restarted the app to give you the latest backup it pulled up both instead (Mag was already deleted) haha. Must be a bug.

Yes it is now fixed you can delete winstreaks now

Its weird, I actually finished the run as dark judas like I said and the app didn't recognize it. It could be a mod issue, but I'm unsure. The weird thing is that it counted as a win towards the rule but I do see that the run is still ongoing on it

No dark juda doesn't impact, there is the date of the end of the run but we don't know if it's a win or a lose ahah, you can fix this later when you will start a new game on the same save, you can force the status of the run (delete, win or lose) from the improved broken runs pop up

Shaniquaniminiquani commented 1 year ago

image I think the transformation is because of the chase sequence for keypiece 2. None of the other times I've done it I've gotten this. My latest run shows 3, because I found a crawlspace and it behaves weird inside of there and transforms you twice

I also don't seem to have gotten an improve broken runs popup yet, but maybe after a few runs? also even if dark judas doesn't impact the rule, it should still show that I transformed as him no?

deleted the other streaks for save 2 but it didn't seem to fix the magdalene one, will try to make a new rule for a fresh character in a few

aymericingargiola commented 1 year ago

I think the transformation is because of the chase sequence for keypiece 2. None of the other times I've done it I've gotten this. My latest run shows 3, because I found a crawlspace and it behaves weird inside of there and transforms you twice

Ohhh nice catch ! I should ignore this character, when you are the lost, i should do a long debug session about this, thanks

Edit : Well i tried and i can't reproduce, i think this is because of the mods you use ?

image

Then i tried to transform has Dark Juda on death with Judas shadow, but it just replace the main character as excpected

image

I tried knife piece 2 but it dosen't trigger a transformation too 😞

image

I also don't seem to have gotten an improve broken runs popup yet, but maybe after a few runs?

No i think in your case it's not showing because it triggers the end but only with the time... i don't know where comes from the issue, maybe the logs are altered with mds, but i'll add options so you can force mark the run as win or lose in the run's options.

also even if dark judas doesn't impact the rule, it should still show that I transformed as him no?

It should not no, it just count as the same character

deleted the other streaks for save 2 but it didn't seem to fix the magdalene one, will try to make a new rule for a fresh character in a few

Did you click on edit then click on save to update it ?

Edit : ah yes it doesn't work because you did cain run on the same save after so it cancel your streak, i had to delete Cain runs

Shaniquaniminiquani commented 1 year ago

but i'll add options so you can force mark the run as win or lose in the run's options

Honestly I think this can close the issue haha. Think my pc is bugging out at this point. I can't even replicate these bugs with mods on now 💀 ty for the update tho 🙏

aymericingargiola commented 1 year ago

I saw you had issue about greed mode also before your edit, this is not the case anymore or caused by mods ? I can check this part while adding thoses buttons also, i'll check this later today !

Shaniquaniminiquani commented 1 year ago

I saw you had issue about greed mode also before your edit

After restarting the run the following tCain run seems to have tracked somewhat properly. I say somewhat because this time it tracked the characters and items properly without corrupting however I noticed the shop and ultra greed floor didn't track for some (They still counted as wins though so it wasn't a big deal, this could maybe be because I didn't pick up items in those floors (am unsure if I did or not) but usually the floors still show up, but empty haha): image

I don't have to do greedier for a bit but maybe f it was just a one time thing though, once I finish the beast mother runs I'll streak greedier w the tainted chars to check.

Will try this tom and see if it tracks properly: image

Sorry for the late reply haha, posted a banger on reddit and then got sick so haven't had too much time to play 🙏

aymericingargiola commented 1 year ago

I say somewhat because this time it tracked the characters and items properly without corrupting however I noticed the shop and ultra greed floor didn't track for some

It's strange indeed since it still finds the Greedier floor to validate the boss and the run, not sure what happenned, i should do some test and watch the game logs

Will try this tom and see if it tracks properly:

Yes let me know ! Seems pretty strong rule

posted a banger on reddit

I don't really understand but everyone seems to have fun ahah !

aymericingargiola commented 1 year ago

I'v added the option to edit run and force run status (win/lose) if then end trigger doesn't work for some reason https://github.com/aymericingargiola/repentance-run-tracker/releases/tag/v0.7.19

Shaniquaniminiquani commented 1 year ago

I'v added the option to edit run and force run status (win/lose) if then end trigger doesn't work for some reason

this works well 👍, is there a current way to check runs flagged as ongoing (Neither won or lost)?

aymericingargiola commented 1 year ago

is there a current way to check runs flagged as ongoing (Neither won or lost)?

I don't see any situation where this could be necessary, you have an idea ? Because the runs watcher will not watch a run that has ended anymore

Shaniquaniminiquani commented 1 year ago

I went back to fix the two broken runs that were still marked ongoing as cain but not corrupted and the idea crept up that there might be other runs I just hadn't noticed. I think this would only be useful if the ongoing run wasn't the latest run on that save file though

aymericingargiola commented 1 year ago

I went back to fix the two broken runs that were still marked ongoing as cain but not corrupted and the idea crept up that there might be other runs I just hadn't noticed. I think this would only be useful if the ongoing run wasn't the latest run on that save file though

Oh ok ! i though you wanted an option to mark the run as ongoing, yes i will add a filter in "win conditions"

aymericingargiola commented 1 year ago

You can now check with last update ! https://github.com/aymericingargiola/repentance-run-tracker/releases/tag/v0.7.20

From your backup file i didn't find old runs with no end status 😄