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
[ fields.lua ] Update 0xC8. #2338
base: dev
Are you sure you want to change the base?
Conversation
Update flags for `types.alliance_member` in `0x0C8`.
[ fields.lua ] Update 0xC8.
While the PR is good and the data is useful, this will break the code for anyone using the current |
I have a couple of questions, and some proposed changes. Mostly dealing with clarity of value meaning.
Also, my 2 cents is go ahead and break usage of |
Not opposed to a), but it's kinda obvious if you look at the type, since it can only hold values |
None:
As for the labels, it is a party identifier, not which member slot in the party you are. Personal preference to all of them really, as I find your wording confusing as well. Party just seems to refer to Grouping to me, and Party Member would refer to a singular player.
This personally leads back to your last complaint, as then you have a flag that is now only available when quartermaster is set, and when in an alliance, and to me personally, there is no label to easily identify that; would you not agree?
It doesn't indicate this, which is actually wrong. If you set Quartermaster this would not flag anything. Edit: To further clarify, one is "Only while in alliance", and the other is "the player" who is quartermaster. The alliance flag is also only set on the player who has quartermaster while in the alliance, and there for seems more logical to use it as a value as more of a quartermaster mode, than a flag.
In-game, you have a Linkshell 1 & Linkshell 2. these flags have no bearing on secondary linkshell, and thus Linkshell 1, refers to only the linkshell in the first slot; the only one which creates an indicator in the nameplate.
I chose the name scheme to follow the items names in game. You can't just have a "any shell". You have a Linkpearl, Pearlsack, or a Linkshell. Linkpearl has no bearing as it changes no data, thus I used Pearlsack, and Linkshell indicating which item effects the flag. |
Uh, is a
I'm not questioning that. What is potentially unclear is whether it is an index of the party (A party slot within an alliance) or an index in the party (a player slot within a party). (note that, obviously it's the former because it's only 2 bits, but I'm trying to think like a Dev for which that wouldn't have occurred to them)
I may need a better explanation of what these values mean then.
to mean that the first bit meant that quartermaster was on someone in the alliance at all, and that the second bit meant that this indicated player is the one that is the alliance quartermaster. just saw this ⬇️ as well so editing my response (before I post it xD)
So you're saying that
I was not stating that slot1/slot2 were likely to be confused, I was stating that it wasn't as clear as it could be that the data had only to do with their linkshell slot 1 and not my linkshell slot 1 -- meaning, a comment could help clarify that it's just whether their slot1 link* is a shell or a sack, and has nothing to do with them being in my slot 1 linkshell. Hence why I am suggesting possibly using verbage that indicates that they are a "sackholder" -- which is a "group authority" term rather than a "item name" term. Anyway, I don't have strong feelings about it, just a thought.
I'm assuming that "they have a linkpearl in slot 1", has the same value as "they do not have any link* equipped at all in slot 1" And as a side comment... I'm trying to figure out why this data is even in here at all XDD (although it would be far from the first time that there was pointless extra info included in a packet) |
Thats exactly what I thought when I found about it, and still have no idea why its there. |
Update flags for
types.alliance_member
in0x0C8
.06:01
- 2-bit value is the Index of the party the player is in.06:03
- Bit is set if the player is a party leader.06:04
- Bit is set if the player is the alliance leader.06:05
- 2-bit value indicating quartermaster. First bit is 1 if you are in an alliance with quartermaster set.06:07
- Bit is set if the player is a linkshell owner in linkshell slot 1.06:08
- Bit is set if the player is a pearlsack owner in linkshell slot 1.Thanks to @KenshiDRK for some additional info about the flags.