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

I'm trying to understand why many spell names have duplicate entries in spells_us.txt #54

Open
eddyystop opened this issue Nov 28, 2022 · 1 comment

Comments

@eddyystop
Copy link

eddyystop commented Nov 28, 2022

The cleric Fortitude spell was not resulting in a spell timer bar. I found there were 2 entries for Fortitude in spells_us.txt. The first entry has the right casting time, the second entry has 0. The way the code works, the second entry will be used and insta-cast spells are not displayed.

I changed the spell name of the second entry so the first entry is used, and I now get my spell timer bar.

I'm trying to understand the situation. There are many such duplicate names, almost all after line 1,000. Have spells changed with expansions and the new defns added to the end of the file? For P99, can I just delete all lines 1,000+ or do I have to change duplicate spell names one by one?

Very useful app BTW.

@rm-you
Copy link
Collaborator

rm-you commented Apr 15, 2024

Yes, the expansions are part of it. You can see at the very end of the line that some spells will say which era they apply to. If there's a specific spell (like Fortitude) that is parsing incorrectly, I can take a look at fixing it which is really just done by changing the name like you did. When I have a minute I'll check this one to verify it's broken and fix it globally. For now I would not recommend truncating the file.

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