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

Trial of the crusader: faction champions' stats are not scaled down #144

Closed
WangDianhui opened this issue Aug 3, 2023 · 12 comments
Closed
Assignees

Comments

@WangDianhui
Copy link

WangDianhui commented Aug 3, 2023

Describe your feature request or suggestion in detail

If there are characters whose reputation with Argent Crusade is friendly or more in the instance, the faction champions' stats would not be scaled.
Neutral with Argent Crusade
Friendly with Argent Crusade

@pangolp
Copy link
Contributor

pangolp commented Aug 3, 2023

How strange that he does not climb with the champions ...
I haven't really used this module, it has many configurations...
I'll have to try it.

@cdenq
Copy link

cdenq commented Aug 15, 2023

Is your party cross-faction by any change? Scaling will apply properly if everyone is the same faction.

@kjack9
Copy link
Collaborator

kjack9 commented Sep 20, 2023

I want to wait until #155 is merged and then troubleshoot this if its still happening - that will give us some more detail via the .ab mapstat and .ab creaturestat commands.

@WangDianhui
Copy link
Author

This problem still exists. I have already pulled the latest commit for both server and mod.
1
2
3
The champions' HP does not change as the num of players changes.

@WangDianhui
Copy link
Author

I think I've found the problem...
4
If the party leader's name contains chinese characters, the champions' stats would not be changed. But for English names, the mod works properly.

@kjack9
Copy link
Collaborator

kjack9 commented Sep 21, 2023

Wow, I have no idea what could possibly be causing that. The mod doesn't care about character names for making decisions. It DOES have character names in some of the announcements, but I can't think of why that would matter.
I don't even know where to begin on troubleshooting this.

Marking "Help Wanted" for now.

@WangDianhui
Copy link
Author

I've confirmed that the problem is independence with the name. But it just occurred when there are some SPECIFIC characters in the instance, no matter who the party leader is.

I'm still looking for the trigger condition.

@WangDianhui
Copy link
Author

Confirmed. If there are characters whose reputation with Argent Crusade is friendly or more in the instance, the faction champions' stats would not be scaled.
Neutral with Argent Crusade
Friendly with Argent Crusade

@kjack9
Copy link
Collaborator

kjack9 commented Oct 12, 2023

I've added this to my personal backlog.

@kjack9
Copy link
Collaborator

kjack9 commented Oct 19, 2023

@WangDianhui can you please test with the latest version of AC and AutoBalance and let me know if the issue is still happening?

@WangDianhui
Copy link
Author

@WangDianhui can you please test with the latest version of AC and AutoBalance and let me know if the issue is still happening?

Tested, the problem has gone. Champions' scaling are now irrelevant with reputation.

@kjack9
Copy link
Collaborator

kjack9 commented Oct 21, 2023

@WangDianhui can you please test with the latest version of AC and AutoBalance and let me know if the issue is still happening?

Tested, the problem has gone. Champions' scaling are now irrelevant with reputation.

Great! Closing as confirmed fixed by #157

@kjack9 kjack9 closed this as completed Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants