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

[SQL] Level correct Gyre-Carlin #5710

Merged
merged 1 commit into from
May 13, 2024

Conversation

ampitere
Copy link
Contributor

@ampitere ampitere commented May 13, 2024

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

Gyre-Carlin is currently set the same min/max as the Nyzul version which isn't appropriate for Pso'Xja. The wiki lists his level as 50, and a capture video also shows the same so. Referencing a similar NM in the same zone (Nunyunuwi) we'll tweak the min to 56 and max to 58. HP is estimated to 5883 in the capture. Resolves #5700

https://www.bg-wiki.com/ffxi/Gyre-Carlin
https://www.youtube.com/watch?v=UYTrBzyYaXI

I am not entirely how to glean the real level from captures (if that's even possible). If there is a way please let me know!

Steps to test these changes

  1. !zone Pso'Xja
  2. !mobhere 16814331

@almuth150
Copy link

almuth150 commented May 13, 2024

Oh, I actually managed to find an old EXP gain table dating back to 2010 here to go along with the old testimonial of getting 53 EXP at level 64, To gain 53 EXP from a kill at that level, the mob would have to be 14 levels lower than the player, which would put that particular spawn at level 50, though that doesn't cover the full potential level range. Which makes sense for a NM in an area that, at the time it was added, was in an area that restricted the level of players to 40.

That being said, I don't know if anything about the NM was changed over the years. I'd be surprised if there were any changes to the NM itself though. Also, I had to dig up an old EXP gain table because it seems that EXP gain was changed in 2011?

Edit: Nunyunuwi is in the part of Pso'Xja that wasn't under level restriction back when it was originally added, unlike Gyre-Carlin.

@WinterSolstice8
Copy link
Member

The widescan packet, for some reason, has mob levels in it. the windower capture suite captures that in a table

however, I'm not convinced its always correct. JoL doesn't give enough exp to match it's widescan level for instance.

@ampitere
Copy link
Contributor Author

It does look like the widescan capture also shows 50. What do y'all think? Keep it 56-58 or tune down to 50-50?

@WinterSolstice8
Copy link
Member

well, given the two data points we have 50 makes sense

@ampitere ampitere force-pushed the fix_level_correct-gyre-carlin branch from 1bdec6b to 91146c4 Compare May 13, 2024 05:36
@ampitere
Copy link
Contributor Author

Tweaked to 50-50.

@claywar claywar merged commit bd3169a into LandSandBoat:base May 13, 2024
11 checks passed
@ampitere ampitere deleted the fix_level_correct-gyre-carlin branch May 13, 2024 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 Gyre-Carlin is probably set to too high of a level
5 participants