Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

only show notification achievement progress on some node #45

Open
gameclamp opened this issue Sep 17, 2024 · 10 comments
Open

only show notification achievement progress on some node #45

gameclamp opened this issue Sep 17, 2024 · 10 comments
Labels
achievements achievements related stuff

Comments

@gameclamp
Copy link

Now we can just choice show notification or not show notification for achievement progress.
I know the game call the notification everytime we got progress.
I suggest show notification only when the new progress exceeds the previous one by 10% or 20%.Than we can keep this feature without being overwhelmed by too many notifications.

@Detanup01 Detanup01 added the achievements achievements related stuff label Sep 17, 2024
@Rustbeard86
Copy link

Rustbeard86 commented Sep 18, 2024

In configs.overlay.ini

disable_achievement_progress=1

See configs.overlay.EXAMPLE.ini in \release\steam_settings.EXAMPLE\configs.overlay.EXAMPLE.ini for examples.

I realize this doesn't solve the issue, but because others may read this and wonder how to work around the spam notification that we see currently, I am putting this here.

@Detanup01 Detanup01 mentioned this issue Sep 25, 2024
@Detanup01
Copy link
Owner

No idea if the PR would be viable for you, please check.
If not I drop it, if you want this feature i add into it

@gameclamp
Copy link
Author

No idea if the PR would be viable for you, please check. If not I drop it, if you want this feature i add into it

Thank you very much.But it seem the code logic not true.If I set min_diff_progress = 3.When kill a zombie in game,the achievement progress not increase anymore.
Should it be this? :
1.add a new property to save the progress value last notified. e.g:last_notified_progress
2.set min_diff_progress in float like 0.1.
3.every time the game call the notification, check the ratio like this: (progress - last_notified_progress) / max_progress >=min_diff_progress ?

code run:
1: kill first zb. last_notified_progress==0? ~ true. show notification . last_notified_progress=progress=1
2: kill second zb. (2- 1) / 5000 >=0.1? ~ false. last_notified_progress: 1,progress: 2
3: kill third zb. (3- 1) / 5000 >=0.1? ~ false. last_notified_progress: 1,progress: 3
......
500: kill 500th zb. (500-1) / 5000 >=0.1? ~false. last_notified_progress: 1,progress: 500
501: kill 501th zb. (501-1) / 5000 >=0.1? ~ true. show notification. last_notified_progress: 501=progress=501
502: kill 500th zb. (502-501) / 5000 >=0.1? ~false. last_notified_progress: 501,progress: 502
.....

@Detanup01
Copy link
Owner

are you sure the % one is good?
Maybe something like (progress - last_notified) >= min_diff
(Set notif every 5)
5 - 0 = 5 | 5 >= 5 , set last_not to 5
6 - 5 = 1 | 1 >= 5 , nothing to do.

for floats:
set notif for 0.5
0.1 - 0 = 0.1 | 0.1 >= 0.5 | skip
0.5 -0 = 0.5 | 0.5 >= 05. set last notif to 0.5
0.6 - 0.5 = 0.1 | 0.1 >= 0.5 | skip

@gameclamp
Copy link
Author

Because some games have big differences in it's achievement values.
e.g:
kill 20 boss
obtain 100000000000 gold
I think the % one is better in this situation.

@Detanup01
Copy link
Owner

Hmm
Yeah sure!
Doing % based after this.
Thanks!

@Detanup01
Copy link
Owner

Updated the PR please test the dll

@gameclamp
Copy link
Author

Thank you for the update.
It seems there is no change in the behaviour.And the user's achievements.json didn't save the last_notified_progress.

@Detanup01
Copy link
Owner

sorry but have 0 idea where is it saving and how would i add that.
It works while you in game i think

@gameclamp
Copy link
Author

I download the file emu-win-api_experimental-release-x64-42a5c18c888b219c776d8390e328c45362bfc503.zip,and set stat_min_diff_progress=0.1 in configs.main.ini [main::stats].
But the notification still show everytime.
20241001225812

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
achievements achievements related stuff
Projects
None yet
Development

No branches or pull requests

3 participants