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

Genshin_panel #2

Open
Aoi-00 opened this issue Jul 15, 2021 · 19 comments
Open

Genshin_panel #2

Aoi-00 opened this issue Jul 15, 2021 · 19 comments

Comments

@Aoi-00
Copy link

Aoi-00 commented Jul 15, 2021

Hi, would you happen to be willing to add more english documentation? I'm trying to implement this in a damage calculator but running into some issues because I'm not sure how the calculation works and the names of certain artifact/character/weapons?

Thank you!

@wormtql
Copy link
Owner

wormtql commented Jul 15, 2021

Thanks for the issue. I'm working on it, but my English is not good enough to write very expressive documentation, and some weapon/character/artifacts name might be awkward.

@wormtql
Copy link
Owner

wormtql commented Jul 15, 2021

and that readme is quite outdated, because I'm not expecting someone will be using it, 🤣

@wormtql
Copy link
Owner

wormtql commented Jul 15, 2021

the document can be viewed here: https://wormtql.gitbook.io/genshin-panel-doc/

@Aoi-00
Copy link
Author

Aoi-00 commented Jul 17, 2021

I see! Yeah I'm using it haha☺️but would you mind explaining what is curedEffect, cureEffect, bCritical, airCritical, bBonus, airBonus, bonus, aSpeed & bSpeed? 😅 Can't seem to figure it out from the documents.

@wormtql
Copy link
Owner

wormtql commented Jul 17, 2021

cureEffect: healing bonus
curedEffect: incoming healing bonus
bCritical: charged attack critical
airCritical: plunging attack critical
bBonus: charged attack bonus
airBonus: plunging attack bonus
bonus: bonus without specific skill/talent
aSpeed: normal attack speed
bSpeed: charged attack speed

@Aoi-00
Copy link
Author

Aoi-00 commented Jul 29, 2021 via email

@wormtql
Copy link
Owner

wormtql commented Jul 29, 2021

The effect of shimenawaReminiscence is quite dynamic, in game you don't always have the bonus, to reflect this,
I use some parameters to config artifacts. You can do this:

let config = {
    configShimenawaReminiscence: {
        rate: 1, // full bonus
    }
}

let attribute = new genshin.AttributeBuilder()
    .character(character).weapon(weapon)
    .artifactsConfig(config)
    .build();

@wormtql
Copy link
Owner

wormtql commented Jul 29, 2021

By default all this kind of dynamic effects are not counted.

@Aoi-00
Copy link
Author

Aoi-00 commented Jul 29, 2021

Is there a way to make it such that all artifacts effects (those dynamic ones) are always activated in attribute? Is there any list of configs?

@wormtql
Copy link
Owner

wormtql commented Jul 29, 2021

No, you have to provide full configs to activate all effects, you can view all config items in https://github.com/wormtql/genshin_panel/blob/main/src/artifact/param.ts#L35

@Aoi-00
Copy link
Author

Aoi-00 commented Jul 29, 2021

By provide full configs to activate all effects, do you mean that if I provide the full config each time I build, it will always assume the artifact effect (the corresponding one I'm using in the build) is active? Or will it activate ALL effects at the same time (even if I am not using the set)?

Also, sorry to bother you about this! But could you explain the linked file? Don't quite get how it works

@wormtql
Copy link
Owner

wormtql commented Jul 29, 2021

It will only activate the one that is in your build.

It's a typescript interface describing the "shape" of the config object, each key represents an artifact set name, and the value represents it's configuration. I suppose you can read js code, it's better to read ts documentations. I believe it's quite straight forward.

@Aoi-00
Copy link
Author

Aoi-00 commented Jul 29, 2021

I see, thank you so much! Also may I know if you are okay with me using this panel in my app? It won't be commercialized!

@wormtql
Copy link
Owner

wormtql commented Jul 30, 2021

Of course you can use it.

@Aoi-00
Copy link
Author

Aoi-00 commented Jul 30, 2021

Hi, so sorry to disturb you again! But I get this error when I use artifact sets that are mixed. It doesn't occur if I use 4 of the same set. May I know why is this so?
image

@wormtql
Copy link
Owner

wormtql commented Jul 30, 2021

Could you provide a minimal program to reproduce this error?

@Aoi-00
Copy link
Author

Aoi-00 commented Jul 30, 2021

Sorry I found the error! It was the artifact set names that have " 's " in it, resulting in a mismatch. I've fixed it!

But now I have an error where if I retrieve the artifact data from localStorage, it isn't saved as 'Artifact' but as just an Object, resulting in an error as shown below. I suspect it is due to the fact that the Artifact object's functions are removed when I save and retrieve it from localstorage.
image

Not really able to provide a minimal program as it is mostly interlinked, but I could provide the 2 page's code
Artifacts.txt
Build.txt

@wormtql
Copy link
Owner

wormtql commented Jul 31, 2021

You'll have to build again using localStorage data

@Aoi-00
Copy link
Author

Aoi-00 commented Jul 31, 2021

I'll give it a try, thanks!

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

No branches or pull requests

2 participants