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.

Sign up for GitHub

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

Jump to bottom

Add AFFIXES Loot Tables to special uniques ? #24

Open
Ctrl-Alt-Suppr opened this issue Jan 4, 2018 · 4 comments
Open

Add AFFIXES Loot Tables to special uniques ? #24

Ctrl-Alt-Suppr opened this issue Jan 4, 2018 · 4 comments
Labels
loot parser website

Comments

@Ctrl-Alt-Suppr
Copy link

(sorry for my bad english)
Is it possible to add Affixes Loot Tables ?
For exemple with a 3st tab :
PROPERTIES | LOOT TABLE | AFFIXES (loot table)

https://www.tq-db.net/equipment/black-pearl-ring
and all : Jade / Pythia / Crystal / Truesilver / Black Pearl ...
@fonsleenaars
Copy link
Owner

I think you don't need to distinguish between Uniques and Monster Infrequents.

The parser (and website) can simply say: If there is an affix possible; show Affix tab.

@Ctrl-Alt-Suppr
Copy link
Author

I do not know the new extension, but with (IT) only a dozen unique objects (out of 1100) had affixes
(I speak well of blue and pink objects)
Not sure it is necessary to add a third tab to all unique if only some are concerned ...

@fonsleenaars
Copy link
Owner

It won't, the tab will only appear if an affix is possible :)

@fonsleenaars fonsleenaars mentioned this issue Oct 17, 2018
Open
@fonsleenaars
Copy link
Owner

Initial discovery in milestone 1.4.3 as a result of rethinking how to parse affixes.

It now uses the actual loot tables which allowed us to encounter the uniques mentioned in the original ticket. For now they'll be included in the source data but we'll need a good way to track how to parse them into actual equipment mentions.

fonsleenaars added a commit that referenced this issue May 30, 2022
@fonsleenaars
refactor: update how affixes are parsed
4a9afd3 
Aside from some black formatting and debug message fixes in this commit
the main change is around parsing affixes.

The parser now looks for the affix tables and then indexes which affix
files to actually parse.

This now includes the Tinkerer and prepares the work for the unique
affixes that only occur on some items like the Black Pearl.

Fixes #70, #60
Refers #24, #51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
loot parser website
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fonsleenaars @Ctrl-Alt-Suppr