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

Kms 1.2.360 (others) #840

Open
wants to merge 13 commits into
base: patch/1.2.360
Choose a base branch
from
Open

Conversation

jj9810
Copy link
Contributor

@jj9810 jj9810 commented Feb 5, 2022

  • 소울마스터
  • 플레임위자드
  • 윈드브레이커
  • 나이트워커
  • 아란
  • 에반
  • 루미너스
  • 데몬슬레이어
  • 데몬어벤져
  • 제로

@icepeng icepeng force-pushed the hero-remaster branch 2 times, most recently from f214ef1 to ecb23c4 Compare February 6, 2022 00:57
dpmModule/jobs/flamewizard.py Outdated Show resolved Hide resolved
dpmModule/jobs/luminous.py Outdated Show resolved Hide resolved
dpmModule/jobs/windbreaker.py Outdated Show resolved Hide resolved
@@ -55,24 +36,22 @@ def get_ruleset(self):

def get_passive_skill_list(self, vEhc, chtr : ck.AbstractCharacter, options: Dict[str, Any]):
Mastery = core.InformedCharacterModifier("숙련도", mastery=90)
CommonSpec = core.InformedCharacterModifier("태도/대검 마스터리 - 공용스펙", pdamage_indep=10, crit=30)
Copy link
Owner

Choose a reason for hiding this comment

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

크확 30도 받았나요?

Copy link
Contributor Author

@jj9810 jj9810 Feb 8, 2022

Choose a reason for hiding this comment

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

원래 알파 마스터리 (AlphaMDF) 크확 +40%에 베타 마스터리 (BetaMDF) 크확 +15%였는데 평준화시키면서 둘다 +30%로 바뀌었습니다. 그래서 성능 향상을 위해 겹치는 부분을 분리했습니다.

@jj9810 jj9810 changed the base branch from hero-remaster to patch/1.2.360 February 8, 2022 15:47
@jj9810
Copy link
Contributor Author

jj9810 commented Feb 8, 2022

확인 감사합니다. 1개는 의도된 사항이라 넘겼고 나머지 3개는 수정 완료했습니다.

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.

None yet

2 participants