While trying to grind for the ChaR I noticed the game stats counts the iqhan master twice thus wasting a precious space of the monster count top 5.
I assume this could be due to their robes in red and purple?
While we're at it, could this top5 be expanded to top7 or even Top10? this would be an enhancement of course, no bug
v0.8.10 (BWM fill) released to Google Play!
Useful links
Source code of the game - Contribution guide - ATCS Editor - Translate the game on Weblate - Example walkthrough - Andor's Trail Directory - Join the Discord
Get the game (v0.8.10) from Google, F-Droid, our server, or itch.io
Useful links
Source code of the game - Contribution guide - ATCS Editor - Translate the game on Weblate - Example walkthrough - Andor's Trail Directory - Join the Discord
Get the game (v0.8.10) from Google, F-Droid, our server, or itch.io
Game stats: iqhan master shows Up twice
-
- Posts: 16
- Joined: Sun Oct 25, 2020 7:11 pm
- android_version: 9.0 - Pie
Game stats: iqhan master shows Up twice
You do not have the required permissions to view the files attached to this post.
Riccious, lvl 71
- rijackson741
- Posts: 4451
- Joined: Tue Aug 20, 2013 2:04 am
- android_version: 10 - Android 10
- Location: Somewhere in Dhayavar
- Contact:
Re: Game stats: iqhan master shows Up twice
Hmm. Yes, there are two NPCs with the name "Iqhan chaos master", and different internal IDs. The name is just a label you see in the game, and is ignored for most purposes. Seeing them show up twice is a little annoying, but fixing it would be a lot of work.
Yes, we could extend the list. Every kill is recorded in the savegame.
Yes, we could extend the list. Every kill is recorded in the savegame.
Level:71, XP:6493739, PV:608, FQ:84
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
-
- Posts: 16
- Joined: Sun Oct 25, 2020 7:11 pm
- android_version: 9.0 - Pie
Re: Game stats: iqhan master shows Up twice
thanks for the quick reply.
Just wanted add, Same issue with Iqhan chaos evoker as the second "entity"of this monster made it to the top 5 as well
understood there will be no change here given the effort and as it's a kind of 'corner case'
Just wanted add, Same issue with Iqhan chaos evoker as the second "entity"of this monster made it to the top 5 as well
understood there will be no change here given the effort and as it's a kind of 'corner case'
Riccious, lvl 71
- Antison
- Posts: 5284
- Joined: Mon Mar 28, 2011 11:33 pm
- android_version: 14 - Android 14
- Location: A home without a beagle is just a house
Re: Game stats: iqhan master shows Up twice
It's not a "corner case", because it's not really a bug. Think of it this way: You got two baseball teams playing a game. Each one is playing in a different uniform, but both sets of players are considered "players". Just because they wear different clothes doesn't make them different things.
"A home without a beagle is just a house"
-
- Posts: 16
- Joined: Sun Oct 25, 2020 7:11 pm
- android_version: 9.0 - Pie
Re: Game stats: iqhan master shows Up twice
Sorry but I don't get your logic.
where is the point of having same titled monsters 2 times in the game stats. 4 of 5 slots are occupied with 2 types of monsters, Chaos master and evoker respectively. there is no benefit / sense for me as player in this behaviour.
where is the point of having same titled monsters 2 times in the game stats. 4 of 5 slots are occupied with 2 types of monsters, Chaos master and evoker respectively. there is no benefit / sense for me as player in this behaviour.
Riccious, lvl 71
-
- VIP
- Posts: 38
- Joined: Thu Feb 21, 2019 4:46 pm
- android_version: 9.0 - Pie
Re: Game stats: iqhan master shows Up twice
Is changing the label realy a big deal?
- rijackson741
- Posts: 4451
- Joined: Tue Aug 20, 2013 2:04 am
- android_version: 10 - Android 10
- Location: Somewhere in Dhayavar
- Contact:
Re: Game stats: iqhan master shows Up twice
The name is the same so that not every monster with that name is quite the same. It's deliberate, to give variety (and there's a lot of such NPCs), so changing the name is not an option. Changing the ID to be the same is also not an option, because that's impossible unless the multiple NPCs become only one. So the only option would be to change the code to look at not only ID, but also name. In which cases would we do that? So sorry, but it is not so simple to fix this.
Level:71, XP:6493739, PV:608, FQ:84
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
-
- Posts: 27
- Joined: Sun Jan 24, 2021 8:22 pm
- android_version: 13 - Android 13
Re: Game stats: iqhan master shows Up twice
I haven't looked at the code, but how about generating the kill-count display based on the cumulative kills for monsters with the same labels? I have no idea how the data is stored, but I have to think it should be fairly straightforward to iterate through the kill-count array/list/database/whatever and sum up counts for each unique label, then generate the list based that.
- rijackson741
- Posts: 4451
- Joined: Tue Aug 20, 2013 2:04 am
- android_version: 10 - Android 10
- Location: Somewhere in Dhayavar
- Contact:
Re: Game stats: iqhan master shows Up twice
The only thing that's tracked is the internal ID, which is always unique. There are many cases where the name is the same, but there is more than one internal ID (for example, there are nine versions of Burhczyd). The only real option would be to change the name so that the names of killable monsters are always unique.
Level:71, XP:6493739, PV:608, FQ:84
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
-
- Posts: 27
- Joined: Sun Jan 24, 2021 8:22 pm
- android_version: 13 - Android 13
Re: Game stats: iqhan master shows Up twice
Right, but the player doesn't know or care that there are multiple internal versions of the same monster. They just want to know the total number of times they killed the same name. So, just create list that tallies the kills for each name (in php or perl you'd use an associative array with the monster name as index; I guess in Java you'd use a HashMap), then sort that list by descending kills and take the top 5.
I'm no good at java, but here's the idea in pseudocode...
Looking at getTop5MostCommonlyKilledMonsters() in model/GameStatistics.java, I think it would be just an extra three or four lines. I've been meaning to get my feet wet in Java and Android development, and this would be a good exercise. Anyone mind if I try to contribute a fix like this?
I'm no good at java, but here's the idea in pseudocode...
Code: Select all
for each monsterId in killedMonstersByID
{
killsByName{monsterId.name}+=monsterId.kills
}
int i=0
for each monsterName in sortByDescendingValues(%killsByName)
{
if (i++ >= 5) break;
addToOutputString(monsterName, killsByName{monsterName})
}