-
Notifications
You must be signed in to change notification settings - Fork 382
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
Option to use job name instead of player name #3000
Comments
I've briefly thought how to apply it, it seems not an easy one. I guess the player name is loaded directly from the log. So, there should be some table relating player with job. |
You can get any player in the party's job via let playerJob;
data.party.details.forEach((p) => {
if (p.name === partner)
playerJob = Util.jobEnumToJob(p.job);
}); My question is, how on earth would you code this? A post-trigger filter to replace player names with jobs or something? |
I guess so. I do think this would need a lot effort. |
Would it make more sense to do this based on indexed roles or something so it works even with duplicates? E.g.
I don't know if there's a way to get the party list in the order it's actually displayed in-client. |
Oh... I understand what you mean. If we can get party member's list order, this can be different by each player, so we should get it from the client data, that's not a bad idea. |
It's possible that it's the natural order of the party list returned by the FFXIV parser plugin, I'd have to check the decomp or do some tests in-game to be sure? |
You mean FFXIV parsing plugin gives us party list order data? Party list order can be changed during fight... (I did it when disconnected then join party again. rejoining mixes the member order) So... we need real-time data of the list, but does parser gives us that information? |
No, looks like that's a bust, party order in partychanged event is random from what it appears? It also doesn't fire that event mid-fight at all, only when someone is added to or removed from the party. |
Something that pushes everyone's <name, job> to shortNames temporarily upon entering an instance? This would be especially terrible in the alliance raids when you're asked to stack on 'Red Mage (6)'. |
There are few sounds from Chinese community requesting this feature, too. Although for me the current method is Okay enough. As far as I know there are extremely shorten job abbreviation which is usually in one or two character in Chinese and Japanese, such as "赤" for Red Mage and "ガ"/"枪" for Gunbreaker. Folks in CN are also prefer to call a player with their job name rather than their first name, especially when the player named in some uncommon characters that someone didn't know how to read them, unless there are duplicated jobs (And then they would call something like "那个四字黑魔"(The Black Mage with 4 characters (in his name)) rather than typing the poor BLM's name...). So what I thought is that we can add a job abbreviation after player's name. For example, "Joe(赤)"/"Joe(RDM)", or "Michelle(暗)"/"Michelle(DRK)"... |
I agree with using abbreviation. Korean server has one word(?) nicknames so mostly they are longer and just shorting the name is awkward (like calling Andrew as "And"). So, I think Korean exceptionally not use the nickname. For example my nickname "블러드트레일" is long. So just saying "용기사" (job name) is better than "블러드트레일 (용기사)" (nickname and job name in bracket). FYI. every Korean job abbreviation is maximum 3 letters. |
I think it could be possible to have |
I'm using something like this. (this script used to be broken but I fixed it in 2022-11-02) "use strict";
export const jobIDToCN = {
20: "武僧",
21: "战士",
23: "诗人",
19: "骑士",
24: "白魔",
25: "黑魔",
27: "召唤",
22: "龙骑",
28: "学者",
30: "忍者",
31: "机工",
32: "DK",
33: "占星",
34: "武士",
35: "赤魔",
36: "青魔",
37: "枪刃",
38: "舞者",
39: "镰刀",
40: "贤者"
};
const playerNicks = Options.PlayerNicks = {};
addOverlayListener("PartyChanged", (e) => {
for (const [key] of Object.entries(playerNicks)) {
delete playerNicks[key];
}
for (const party of e.party) {
const v = jobIDToCN[party.job];
if (v) {
playerNicks[party.name] = v;
}
}
});
console.log("enable name to job"); |
What about add an option in |
Do you think we should solve this? You could theoretically have 15 of the same DPS in an alliance raid in the Duty Finder. |
I think the user who choose this option should aware that there can be a situation like what you mentioned (we can also add a note on the option), what I talking about is more like a raid-oriented feature for those who don't want to use the |
The default option is kept same as now, showing player name. If user change this option, they'll naturally understand this could be a problem with "more than one player per job" situation. I think users will use "print job name" option when they think good to use only. (one player per job situation.) |
Hmm, if there are 5 RDMs in a party, does it sort them alphabetically afterward, or is the order random? If it's actually sorted alphabetically, they could be RDM1, RDM2, etc. |
Assuming you know ahead of time if there’s collisions (on instance load) you could do:
And just the job if there are no collisions.
Basically the opposite of what MaikoTan proposed above. This way you wouldn’t need to swap the configuration between content types. You could take Trim21’s example and extend it so that the record value is an array and anything with a length > 1 will set the playerNick to Alternatively, you could do like MaikoTan wrote above in the style of “Name (Job)” for collisions; I don’t know which is more appropriate for each language. |
Also, another aspect to consider here is text to speech. I don’t know if the abbreviations (formal or informal) for other languages can be easily read aloud, but I imagine ‘RDM’ or ‘GNB’ don’t sound very good through TTS, and pronouncing each letter like ‘W H M’ (4 syllables) would take longer to pronounce than ‘White Mage’. |
oh, that's good point. Korean abbreviation is fine. Korean each letter has full pronounce and call them same in speech. |
Yes, there is an issue. By the way, there is no TTS issue in Chinese abbreviations. And those folks in Chinese server often use https://github.com/Noisyfox/ACT.FoxTTS as TTS engine, which has a preprocessor for replacing text so that TTS would say the right speech. (I have a fork of it and implement Japanese support but didn't merge the preprocessor commit (/▽\)) |
I think in general, there aren't options that people are switching back and forth per content type. I think if possible I would like to have options that people can set once and then not have to adjust constantly. Maybe that means we need more options for people to pick from or some options should treat raid/trial content differently than alliance content?
Unfortunately, as far as I can tell there is no ordered party list information in ACT or OverlayPlugin at the moment. So, the best you could do is to alphabetically sort them so at least "PLD 1" is always the same PLD if you have two. Or, use job + name instead as was also suggested above. |
I would like this. Player names mean nothing to me — especially when I use TTS and it struggles to pronounce the names. It would be so much clearer to hear "x on DPS" or "x on tank" rather than "x on Ksduhsdsdhuasd". |
You can now also write `${player.job}`, `${player.role}`, `${player.nick}`, or `${player.name}` in triggers now as well if you want to override output strings individually. If the property isn't there (somebody not in your party, bugs, typos), it will default to nickname. Fixes #3000.
…gers (#5916) You can now also write `${player.job}`, `${player.role}`, `${player.nick}`, or `${player.name}` in triggers now as well if you want to override output strings individually. If the property isn't there (somebody not in your party, bugs, typos), it will default to nickname. Fixes #3000. 2d1f7d1
Sorry for getting around to this 2.5 years later. Thanks for all the work and ideas, @valarnin. The base functionality is now there, but I've filed a few extra things that I suspect people want if anybody is interested in implementing it themselves. I think it should be pretty straightforward: #5917 |
Summary
I know why triggers are calling player name, not the job name. You can have two or more player with same job.
But usually when you go latest raid content, you normally have one player per job. I think there's quite a few people who wants to know the job name, not nickname. I often hear that if there is any way to called by job name instead of nickname from community, especially from those who uses PF/RF without static.
The text was updated successfully, but these errors were encountered: