Skip to content

Added Nocturne's Curtain Call#2577

Open
Matoba004 wants to merge 2 commits intogenshinsim:mainfrom
Matoba004:nocturnes-courtain-call
Open

Added Nocturne's Curtain Call#2577
Matoba004 wants to merge 2 commits intogenshinsim:mainfrom
Matoba004:nocturnes-courtain-call

Conversation

@Matoba004
Copy link
Contributor

Weapon PR checklist:

  • New weapon package
  • Config in weapon package
  • Run pipeline with added config (generates weapon curve, .generated.json files)
  • Weapon key
  • Shortcuts for weapon key
  • Add weapon package to imports
  • Weapon passive (might include an attack)
  • Update documentation

r := p.Refine

m := make([]float64, attributes.EndStatType)
m[attributes.HPP] = 0.075 + float64(r)*0.025
Copy link
Collaborator

Choose a reason for hiding this comment

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

The HP should be 0.08 + float64(r)*0.02

if atk.Info.ActorIndex != char.Index() {
return
}
if atk.Info.AttackTag < attacks.LunarReactionStartDelim || atk.Info.AttackTag > attacks.DirectLunarReactionEndDelim {
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can use !attacks.AttackTagIsLunar(atk.Info.AttackTag)

w.nocturneBuff(c, char, r)
}

c.Events.Subscribe(event.OnEnemyHit, onHitF, "nocturnes-courtain-call-buff")
Copy link
Collaborator

@Charlie-Zheng Charlie-Zheng Feb 26, 2026

Choose a reason for hiding this comment

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

I think this should be OnEnemyDamage? Fix spelling on curtain


func (w *Weapon) nocturneBuff(c *core.Core, char *character.CharWrapper, r int) {
if !char.StatusIsActive(ICDKey) {
char.AddEnergy("nocturnes-courtain-call", 15)
Copy link
Collaborator

@Charlie-Zheng Charlie-Zheng Feb 26, 2026

Choose a reason for hiding this comment

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

The energy scales by refine. 13 + float64(r). Change to nocturnes-curtain-call

}

m := make([]float64, attributes.EndStatType)
m[attributes.HPP] = 0.09 + float64(r)*0.03
Copy link
Collaborator

Choose a reason for hiding this comment

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

should be 0.12 + float64(r)*0.02

},
})

c.Events.Subscribe(event.OnEnemyHit, func(args ...any) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Move this to Init/NewWeapon. Only give the buff when the Bountiful Sea's Sacred Wine effect is active. Also need to add a subscription to OnLunarChargedReactionAttack to buff CDMG for the individual contribution to Reaction LC.

m := make([]float64, attributes.EndStatType)
m[attributes.HPP] = 0.09 + float64(r)*0.03
char.AddStatMod(character.StatMod{
Base: modifier.NewBaseWithHitlag("nocturnes-courtain-call-hp", 12*60),
Copy link
Collaborator

@Charlie-Zheng Charlie-Zheng Feb 26, 2026

Choose a reason for hiding this comment

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

Rename the buff to nocturnes-curtain-call-buff (note the spelling) and use char.StatModIsActive() for the Crit DMG bonus

"moonweaversdawn",
"mountainbracingbolt",
"mouunsmoon",
"nocturnescourtaincall",
Copy link
Collaborator

Choose a reason for hiding this comment

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

nocturnescurtaincall

MoonweaversDawn
MountainBracingBolt
MouunsMoon
NocturnesCourtainCall
Copy link
Collaborator

Choose a reason for hiding this comment

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

NocturnesCurtainCall

"mouunsmoon": keys.MouunsMoon,
"mouun": keys.MouunsMoon,
"mouuns": keys.MouunsMoon,
"nocturnescourtaincall": keys.NocturnesCourtainCall,
Copy link
Collaborator

Choose a reason for hiding this comment

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

NocturnesCurtainCall

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.

2 participants