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

[BCNM] Implement BCNM20 Shooting Fish #6220

Open
wants to merge 1 commit into
base: base
Choose a base branch
from

Conversation

The-Aerec
Copy link
Contributor

@The-Aerec The-Aerec commented Sep 6, 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?

Implements the Aqua Ball Mechanic for BCNM20 shooting fish.

Basis and evidence for following changes:
https://youtu.be/V0Qp99O1AkA
https://drive.google.com/open?id=1ZpZ9FEQ3TYeCc9lPig-y2p6pCRLNkdyF
https://ffxiclopedia.fandom.com/wiki/Shooting_Fish

image

Ability 313 it is - renamed from counterspore for clarity

Steps to test these changes

Enter BCNM20 shooting fish and see the mobs properly only use their knockback aquaball as a auto-attack.

@The-Aerec The-Aerec force-pushed the ShootingFish branch 5 times, most recently from e5842f7 to 8fdb08d Compare September 6, 2024 22:41
@The-Aerec The-Aerec marked this pull request as ready for review September 6, 2024 22:41
-- Next available ID: 2014
INSERT INTO `mob_skill_lists` VALUES ('Shooting_Fish',2014,313); -- knockback aquaball - BCNM20 Shooting Fish

-- Next available ID: 2008
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can remove this, 2008 is used

Copy link
Contributor

@jmcmorris jmcmorris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just missing the new type info for the skill.

-----------------------------------
require('scripts/globals/mobskills')
-----------------------------------
local mobskillObject = {}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

---@type TMobSkill

@zach2good
Copy link
Contributor

I can't comment on the exact line, but is everything in place for this to not be considered experimental? Mob levels, mob hp, resistances, the mechanics, the loot, the entry, etc.

@LacunaXI
Copy link
Contributor

LacunaXI commented Sep 8, 2024

The loot in the .lua is set to the exact rates of Clopedia with limited samples. Given the limited drop variance, Platoon weapons for example are more likely each an equal weight with the others in the slot. Would it make sense for the weights to be more equalized for some of these drops?

@jmcmorris
Copy link
Contributor

Yes, ideally we use loot with the weights defined in xi.loot.weight. https://github.com/LandSandBoat/server/blob/base/scripts/globals/battlefield.lua#L64-L72

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.

5 participants