-
Notifications
You must be signed in to change notification settings - Fork 133
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
Chambers of Xeric QOL & fakemass #5970
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❌ Changes requested. Reviewed everything up to 8d13a3c in 50 seconds
More details
- Looked at
1353
lines of code in11
files - Skipped
0
files when reviewing. - Skipped posting
0
drafted comments based on config settings.
Workflow ID: wflow_OCnIlva1edmqdWcT
Want Ellipsis to fix these issues? Tag @ellipsis-dev
in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
const quantity = _quantity ?? 1; | ||
const allUsers = await Promise.all(users.map(async u => mUserFetch(u))); | ||
const fetchedUsers = await Promise.all(users.map(async u => mUserFetch(u))); | ||
let allUsers = isFakeMass ? Array(maxSizeInput).fill(fetchedUsers[0]) : fetchedUsers; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The handling of fakemass
by using the same user object for all positions in the array might not reflect the intended functionality of a mass where each user is distinct. This could affect loot distribution and point calculations. Consider revising how fakemass
is handled to ensure it aligns with the expected behavior of a mass raid.
Will merge after conflicts fixed (assuming its been tested and such) |
Updated. |
Description:
Update various aspects of CoX to closer match ingame times and boosts.
Add an option to fakemass (similar to nex/tob solo).
Changes:
fakemass
/raid cox itemboosts
to check your itemboosts for CoXisFakeMass
maxSizeInput
& update repeatStoredTripUpdated Times for CoX:
Assumptions: 1 raid trip, 100% itemboost, 100% kc boost, 100% gearscore.
solo: 16m 42s - 18m 28s (CA time: < 17m 00s)
2man: 14m 39s - 16m 12s
3man: 13m 43s - 15m 09s (CA time: < 14m 30s)
4man: 12m 55s - 14m 17s
5man: 12m 08s - 13m 25s (CA time: < 12m 30s)
6man or higher: same as 5man times
cm solo: 30m 12s - 33m 23s (CA time: < 38m 30s)
cm 2man: 20m 24s - 22m 33s
cm 3man: 21m 38s - 23m 55s (CA time: < 27m 00s)
cm 4man: 20m 24s - 22m 33s
cm 5man: 19m 09s - 21m 10s (CA time: < 25m 00s)
6man or higher: same as 5man times
Other checks:
closes: #1811
closes: #1769
closes: #2151
closes: #1850