fonsleenaars / tqdb

Python parser for the Titan Quest Anniversary Edition, Ragnarok, Atlantis, Eternal Ember DLC database.
https://tq-db.net
MIT License
50 stars 15 forks source link

Affixes doesn't say which sort of items they are eligible for #39

Closed DiomedesofHelos closed 5 years ago

DiomedesofHelos commented 6 years ago

Hi. I really love the site, but if I want a green legs armor for an example, it is hard to know which prefixes and suffixes that there's even a chance of them having. Therefore not possible for me to plan, if the prefix I want can only occur on a ring or another item that isn't legs armor. Is it possible to update the affixes to say which kinds of green and yellow items they can occur on? Does anyone even know? Ps. Otherwise the site is perfect, except I don't quite understand why the Adamantine Sickle of Kronos isn't there along with all the basic non-magical pieces of armor and weapons which are the ones that can have all the affixes.

fonsleenaars commented 6 years ago

Definitely possible, it's already reported in #6 ;) Feel free to close it yourself if #6 accurately describes what you're asking for!

DiomedesofHelos commented 6 years ago

Thank you very much, cheers!

On Tue, 22 May 2018 04:51 Fons Leenaars, notifications@github.com wrote:

Definitely possible, it's already reported in #6 https://github.com/fonsleenaars/tqdb/issues/6 ;) Feel free to close it yourself if #6 https://github.com/fonsleenaars/tqdb/issues/6 accurately describes what you're asking for!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-390846380, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHPMDHc1RK6RSJVV4E-Nxcy9xkpvmks5t030-gaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

Do you have any idea at all of when the page will be updated to show it?:-)

2018-05-22 10:27 GMT+02:00 Asgerr Storgaard-Larsen <seth.storgaard@gmail.com

:

Thank you very much, cheers!

On Tue, 22 May 2018 04:51 Fons Leenaars, notifications@github.com wrote:

Definitely possible, it's already reported in #6 https://github.com/fonsleenaars/tqdb/issues/6 ;) Feel free to close it yourself if #6 https://github.com/fonsleenaars/tqdb/issues/6 accurately describes what you're asking for!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-390846380, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHPMDHc1RK6RSJVV4E-Nxcy9xkpvmks5t030-gaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Hm, hard to give an estimate at the moment, I'm trying to make time for TQDB so I'll see if I can update the release schedule to something more concrete. Maybe 3-4 weeks?

DiomedesofHelos commented 6 years ago

Sounds amazing!!!!xD Thank you, it really is the best tq site I've ever seen!!;-)

On Tue, 22 May 2018 15:55 Fons Leenaars, notifications@github.com wrote:

Hm, hard to give an estimate at the moment, I'm trying to make time for TQDB so I'll see if I can update the release schedule to something more concrete. Maybe 3-4 weeks?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-390999373, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHLyxsORcA4YPkgNyrjnihHgVB1lCks5t1BjogaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

Hello! Do you know more about which date the affixes will tell which kind of item they may belong to? :) Also I found a blue item called Bormo's Arm Ring, but it's not in the database. How can that be? Keep up the good work, I really appreciate it

2018-05-22 15:58 GMT+02:00 Asgerr Storgaard-Larsen <seth.storgaard@gmail.com

:

Sounds amazing!!!!xD Thank you, it really is the best tq site I've ever seen!!;-)

On Tue, 22 May 2018 15:55 Fons Leenaars, notifications@github.com wrote:

Hm, hard to give an estimate at the moment, I'm trying to make time for TQDB so I'll see if I can update the release schedule to something more concrete. Maybe 3-4 weeks?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-390999373, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHLyxsORcA4YPkgNyrjnihHgVB1lCks5t1BjogaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Thanks, I'll check if Bormo's Arm Ring is in the latest release I've been working on locally.

I think for the affix information, probably mid july by the latest? I've got 2 weeks holiday at the end of June that I plan to use to get a lot of issues worked through for this project, so the release following that holiday will probably have the affix fix in it! Wish I had more time to spend on this project, it's a lot of fun!

DiomedesofHelos commented 6 years ago

Sounds great! Im so glad you think its fun! It helps a lot of us!! :)

On Tue, 5 Jun 2018 19:44 Fons Leenaars, notifications@github.com wrote:

Thanks, I'll check if Bormo's Arm Ring is in the latest release I've been working on locally.

I think for the affix information, probably mid july by the latest? I've got 2 weeks holiday at the end of June that I plan to use to get a lot of issues worked through for this project, so the release following that holiday will probably have the affix fix in it! Wish I had more time to spend on this project, it's a lot of fun!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-394798920, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHIRbhBTq0veXqTn8Ju1g0ib-3ijJks5t5sN1gaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

Is it going well with patching the site? ;)

2018-06-05 20:10 GMT+02:00 Asgerr Storgaard-Larsen <seth.storgaard@gmail.com

:

Sounds great! Im so glad you think its fun! It helps a lot of us!! :)

On Tue, 5 Jun 2018 19:44 Fons Leenaars, notifications@github.com wrote:

Thanks, I'll check if Bormo's Arm Ring is in the latest release I've been working on locally.

I think for the affix information, probably mid july by the latest? I've got 2 weeks holiday at the end of June that I plan to use to get a lot of issues worked through for this project, so the release following that holiday will probably have the affix fix in it! Wish I had more time to spend on this project, it's a lot of fun!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-394798920, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHIRbhBTq0veXqTn8Ju1g0ib-3ijJks5t5sN1gaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Somewhat slowly since work is busy at the moment, after this week I have another week holiday so I expect that to be the week I can finally wrap it up. Sorry for the delay!

DiomedesofHelos commented 6 years ago

It's okay, I just appreciate your work. Thank you for keeping it up!:)

On Sun, 22 Jul 2018 14:30 Fons Leenaars, notifications@github.com wrote:

Somewhat slowly since work is busy at the moment, after this week I have another week holiday so I expect that to be the week I can finally wrap it up. Sorry for the delay!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-406863170, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHNjkWxYe32j-6a0y9i-_qfnSGdicks5uJHBkgaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Parser changes have been completed, I'm updating the website so it can handle the new information.

Maybe 1 more week until it's done.

DiomedesofHelos commented 6 years ago

Awesome, great news! Thank you :)

On Sun, 5 Aug 2018 14:42 Fons Leenaars, notifications@github.com wrote:

Parser changes have been completed, I'm updating the website so it can handle the new information.

Maybe 1 more week until it's done.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-410517742, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHLBNSGWV8cmIwnLu7SZ3ppdcnv5hks5uNugcgaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

It's gonna be so great :D

On Sun, 5 Aug 2018 14:58 Asgerr Storgaard-Larsen, seth.storgaard@gmail.com wrote:

Awesome, great news! Thank you :)

On Sun, 5 Aug 2018 14:42 Fons Leenaars, notifications@github.com wrote:

Parser changes have been completed, I'm updating the website so it can handle the new information.

Maybe 1 more week until it's done.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-410517742, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHLBNSGWV8cmIwnLu7SZ3ppdcnv5hks5uNugcgaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Oh I was looking for this issue, here's the update on staging: https://staging.tq-db.net/affix/prefixes

Once I fix a few more bugs it will go on the main website.

DiomedesofHelos commented 6 years ago

Can't thank you enough for that, you're so cool!

On Tue, 14 Aug 2018 22:02 Fons Leenaars, notifications@github.com wrote:

Oh I was looking for this issue, here's the update on staging: https://staging.tq-db.net/affix/prefixes

Once I fix a few more bugs it will go on the main website.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-412996576, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHFo8lYYKRMKAFBwcyvAV--lX9LPAks5uQyzLgaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

Btw I just noticed "Of the Mammoth" is in prefixes instead of suffixes, is it supposed to be there? And it says it's only for reference. Just if you were looking for bugs :P

2018-08-14 22:05 GMT+02:00 Asgerr Storgaard-Larsen <seth.storgaard@gmail.com

:

Can't thank you enough for that, you're so cool!

On Tue, 14 Aug 2018 22:02 Fons Leenaars, notifications@github.com wrote:

Oh I was looking for this issue, here's the update on staging: https://staging.tq-db.net/affix/prefixes

Once I fix a few more bugs it will go on the main website.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-412996576, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHFo8lYYKRMKAFBwcyvAV--lX9LPAks5uQyzLgaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

And also, do you know where then to find the items with the affixes that says they are only for reference, if they aren't found on common items?

2018-08-14 23:29 GMT+02:00 Asgerr Storgaard-Larsen <seth.storgaard@gmail.com

:

Btw I just noticed "Of the Mammoth" is in prefixes instead of suffixes, is it supposed to be there? And it says it's only for reference. Just if you were looking for bugs :P

2018-08-14 22:05 GMT+02:00 Asgerr Storgaard-Larsen < seth.storgaard@gmail.com>:

Can't thank you enough for that, you're so cool!

On Tue, 14 Aug 2018 22:02 Fons Leenaars, notifications@github.com wrote:

Oh I was looking for this issue, here's the update on staging: https://staging.tq-db.net/affix/prefixes

Once I fix a few more bugs it will go on the main website.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-412996576, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHFo8lYYKRMKAFBwcyvAV--lX9LPAks5uQyzLgaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

And lastly, aren't the requirements listed for all the Ragnarok items?

2018-08-14 23:30 GMT+02:00 Asgerr Storgaard-Larsen <seth.storgaard@gmail.com

:

And also, do you know where then to find the items with the affixes that says they are only for reference, if they aren't found on common items?

2018-08-14 23:29 GMT+02:00 Asgerr Storgaard-Larsen < seth.storgaard@gmail.com>:

Btw I just noticed "Of the Mammoth" is in prefixes instead of suffixes, is it supposed to be there? And it says it's only for reference. Just if you were looking for bugs :P

2018-08-14 22:05 GMT+02:00 Asgerr Storgaard-Larsen < seth.storgaard@gmail.com>:

Can't thank you enough for that, you're so cool!

On Tue, 14 Aug 2018 22:02 Fons Leenaars, notifications@github.com wrote:

Oh I was looking for this issue, here's the update on staging: https://staging.tq-db.net/affix/prefixes

Once I fix a few more bugs it will go on the main website.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-412996576, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHFo8lYYKRMKAFBwcyvAV--lX9LPAks5uQyzLgaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Of the Mammoth

Good catch thanks

only for reference

This basically means they cannot occur in-game, only if somebody were to write a mod that uses it, could they be found

requirements listed for all the Ragnarok items

I'll take a look, thanks for reporting!

DiomedesofHelos commented 6 years ago

Thank you so much too!:)

On Wed, 15 Aug 2018 12:18 Fons Leenaars, notifications@github.com wrote:

Of the Mammoth

Good catch thanks

only for reference

This basically means they cannot occur in-game, only if somebody were to write a mod that uses it, could they be found

requirements listed for all the Ragnarok items

I'll take a look, thanks for reporting!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-413154967, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHJZ_OA-H-eeMqJzgsHE5AsE6UdAXks5uQ_VrgaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

It might not only be the Ragnarok items, a lot of items, especially the monster infrequents, seems to be missing the stat requirements. Also some of them seems to be messed up, where the epic version of the item requires more strength fx, than the legendary version of that same item... Mysterious

2018-08-15 12:36 GMT+02:00 Asgerr Storgaard-Larsen <seth.storgaard@gmail.com

:

Thank you so much too!:)

On Wed, 15 Aug 2018 12:18 Fons Leenaars, notifications@github.com wrote:

Of the Mammoth

Good catch thanks

only for reference

This basically means they cannot occur in-game, only if somebody were to write a mod that uses it, could they be found

requirements listed for all the Ragnarok items

I'll take a look, thanks for reporting!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-413154967, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHJZ_OA-H-eeMqJzgsHE5AsE6UdAXks5uQ_VrgaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

Sorry to constantly notify you, but I noticed that the -15.00 health regeneration for the prefix "Defiled" doesn't show in the updated version of the site as of now.

On Thu, 16 Aug 2018 10:40 Asgerr Storgaard-Larsen, seth.storgaard@gmail.com wrote:

It might not only be the Ragnarok items, a lot of items, especially the monster infrequents, seems to be missing the stat requirements. Also some of them seems to be messed up, where the epic version of the item requires more strength fx, than the legendary version of that same item... Mysterious

2018-08-15 12:36 GMT+02:00 Asgerr Storgaard-Larsen < seth.storgaard@gmail.com>:

Thank you so much too!:)

On Wed, 15 Aug 2018 12:18 Fons Leenaars, notifications@github.com wrote:

Of the Mammoth

Good catch thanks

only for reference

This basically means they cannot occur in-game, only if somebody were to write a mod that uses it, could they be found

requirements listed for all the Ragnarok items

I'll take a look, thanks for reporting!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-413154967, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHJZ_OA-H-eeMqJzgsHE5AsE6UdAXks5uQ_VrgaJpZM4UHY5R .

fonsleenaars commented 6 years ago

No problem at all, I appreciate the bugreports, I'm just going to reopen this issue so I don't forget!

I'll respond later to the bugs, at work atm!

DiomedesofHelos commented 6 years ago

Yes sir!

On Fri, 17 Aug 2018 13:59 Fons Leenaars, notifications@github.com wrote:

No problem at all, I appreciate the bugreports, I'm just going to reopen this issue so I don't forget!

I'll respond later to the bugs, at work atm!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-413843021, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHPP8cRcn0_v_4wlAg_hx90X1lhCHks5uRrAGgaJpZM4UHY5R .

DiomedesofHelos commented 6 years ago

Also sets display bonuses for having their items improperly, as if there are a bonus for having more items in the set than there even are of items in it. These are all not the biggest issues, to me, the biggest issue is just that all items don't display the stat requirements needed for them. Except for the level maybe, but not the stats that you need.

On Wed, 15 Aug 2018 12:18 Fons Leenaars, notifications@github.com wrote:

Of the Mammoth

Good catch thanks

only for reference

This basically means they cannot occur in-game, only if somebody were to write a mod that uses it, could they be found

requirements listed for all the Ragnarok items

I'll take a look, thanks for reporting!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-413154967, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHJZ_OA-H-eeMqJzgsHE5AsE6UdAXks5uQ_VrgaJpZM4UHY5R .

fonsleenaars commented 6 years ago
DiomedesofHelos commented 6 years ago

Very cool (Y) :)

On Wed, 22 Aug 2018 11:43 Fons Leenaars, notifications@github.com wrote:

  • Of the Mammoth (this is tagged as a Prefix internally, I'll add an exception note at some point)
  • Defiled is lacking -15.00 HP regen
  • Requirements missing
  • Set display bonuses incorrectly

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-414975628, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHLaXZ9pagm5GXvnJqniHXkBYbJ8zks5uTSfWgaJpZM4UHY5R .

fonsleenaars commented 6 years ago

The requirements (for the MIs at least) seem to just not be set, even compared to the current tq-db.net.

I checked the files and they're not in there either. I think it might be largely in part of the MIs having affix requirements, instead of base ones. It's strange because in many instances it looks like the Normal version has a requirement but the Epic and Legendary versions don't.

Did you find any epic/legendary item without requirements? If so, link me that.

DiomedesofHelos commented 6 years ago

Dagger of basbona, hofud, hrunting, lævateinn and many more. Affixes don't have a stat requirement though, except for level do they?

On Wed, 22 Aug 2018 13:13 Fons Leenaars, notifications@github.com wrote:

The requirements (for the MIs at least) seem to just not be set, even compared to the current tq-db.net.

I checked the files and they're not in there either. I think it might be largely in part of the MIs having affix requirements, instead of base ones. It's strange because in many instances it looks like the Normal https://staging.tq-db.net/equipment/deathweavers-helmet-normal version has a requirement but the Epic https://staging.tq-db.net/equipment/deathweavers-helmet-epic and Legendary https://staging.tq-db.net/equipment/deathweavers-helmet-legendary versions don't.

Did you find any epic/legendary item without requirements? If so, link me that.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-414997333, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHL6QFtKLhQ9euZLMwj6FN6iGh9Ooks5uTTzNgaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Thanks!

Yeah that's true, I kind of need a screenshot from the current in-game for one of those examples of MIs without requirements. I think I have a high level character I can use to go do some runs to see if I can verify the in-game drop.

DiomedesofHelos commented 6 years ago

Thank you, that would be very generous of you! Do you want me to report it as well, if I come across items in the game whos stat reqs aren't shown? A little tip: A few of the items are shown as when held over by the mouse cursor, when you simply google "titan quest the name of the item", and then go to images. Although on rarer occasions, there's a link on the normal search that takes you to a debate where someone has posted a picture of the item where you can see the stats required. It could save you a little time.

On Wed, 22 Aug 2018 13:58 Fons Leenaars, notifications@github.com wrote:

Thanks!

Yeah that's true, I kind of need a screenshot from the current in-game for one of those examples of MIs without requirements. I think I have a high level character I can use to go do some runs to see if I can verify the in-game drop.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-415007428, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHItXbUcx2ge6fiHVWD2Ksu47bQUhks5uTUdogaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Yeah, for now just add them in this ticket somewhere (you can edit your original issue if you want, up to you).

That's true, but I never trust anything except the in-game screenshots anymore, and since Ragnarok a lot of those are outdated. But you're right, sometimes it's a nice way to verify!

DiomedesofHelos commented 6 years ago

Well I will send you screenshots then, each time I find any that doesn't show stat req, and I do have ragnarok :) Creds to you doe not trusting everything, I didn't think you would either. It's good to be sure that you have the right information!

On Wed, 22 Aug 2018 14:09 Fons Leenaars, notifications@github.com wrote:

Yeah, for now just add them in this ticket somewhere (you can edit your original issue if you want, up to you).

That's true, but I never trust anything except the in-game screenshots anymore, and since Ragnarok a lot of those are outdated. But you're right, sometimes it's a nice way to verify!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-415010050, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHE8Q2TIfyFbJL7l3KdnTuitSVLjuks5uTUn3gaJpZM4UHY5R .

fonsleenaars commented 6 years ago

Deathweaver's helmet in Epic.

Level 25 Strength 368

I think the level requirement comes from the Rugged prefix, so now all I have to do is figure out where 368 Strength comes from!

DiomedesofHelos commented 6 years ago

Hmm interesting.

On Wed, 22 Aug 2018 14:45 Fons Leenaars, notifications@github.com wrote:

Deathweaver's helmet https://puu.sh/Bit1G/4f71d65a10.jpg in Epic.

Level 25 Strength 368

I think the level requirement comes from the Rugged prefix, so now all I have to do is figure out where 368 Strength comes from!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-415019091, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHAOVEtKewP-xqUCRxmEVgXsU3wAYks5uTVJegaJpZM4UHY5R .

fonsleenaars commented 6 years ago

I've found one explanation, but there's absolutely no reference to this file from the Deathweaver's helmet file, but in what seems like a generic "requirement" calculation folder, a file called

itemcost.dbr has the following equation: headStrengthEquation,50+((itemLevel-1)*7.75),

The item level for Deathweaver's Helmet is 42, so plugging that in yields:

50 + (41 * 7.75) = 367.5

Now I just need to figure out how on earth it's determined that this calculation should be used!

DiomedesofHelos commented 6 years ago

Hmm good clue. Exciting..!

On Wed, 22 Aug 2018 15:02 Fons Leenaars, notifications@github.com wrote:

I've found one explanation, but there's absolutely no reference to this file from the Deathweaver's helmet file, but in what seems like a generic "requirement" calculation folder, a file called

itemcost.dbr has the following equation: headStrengthEquation,50+((itemLevel-1)*7.75),

The item level for Deathweaver's Helmet is 42, so plugging that in yields:

50 + (41 * 7.75) = 367.5

Now I just need to figure out how on earth it's determined that this calculation should be used!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-415023724, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHIx1ww5F3ErfYhPuXCmn5TqnsDKUks5uTVZZgaJpZM4UHY5R .

fonsleenaars commented 6 years ago

So what I've done is I've set this itemcost.dbr file as a fallback for any item that doesn't have a requirement.

I did limit it to equipment, not just all things in the database. I think it works now, but I only have the Deathweaver's Helmet as definitive proof. I've deployed it to staging so you can check it.

DiomedesofHelos commented 6 years ago

Awesome!! Thank you, hope it works for all the items then.:)

On Wed, 22 Aug 2018 15:52 Fons Leenaars, notifications@github.com wrote:

So what I've done is I've set this itemcost.dbr file as a fallback for any item that doesn't have a requirement.

I did limit it to equipment, not just all things in the database. I think it works now, but I only have the Deathweaver's Helmet as definitive proof. I've deployed it to staging so you can check it.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-415039333, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHGtHXHXKc9MxYRCqkvGXRCZ1wtlYks5uTWIvgaJpZM4UHY5R .

fonsleenaars commented 5 years ago

Alrighty, I've also fixed the set item bonuses (I think, let me know if you find any errors there). It's deploying to https://staging.tq-db.net to verify.

Few more issues to close before I can move everything to the main tq-db.net website with a release note.

DiomedesofHelos commented 5 years ago

It's beginning to look really amazing. Thank you my friend :-)

On Thu, 23 Aug 2018 22:22 Fons Leenaars, notifications@github.com wrote:

Alrighty, I've also fixed the set item bonuses (I think, let me know if you find any errors there). It's deploying to https://staging.tq-db.net to verify.

Few more issues to close before I can move everything to the main tq-db.net website with a release note.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-415558881, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHMW2Qm9Trnc80zx2yFzXrpudUfxLks5uTw8cgaJpZM4UHY5R .

fonsleenaars commented 5 years ago

No problem, I'll close this issue again, I think the issues you found have been address in the staging environment now right?

The 'of the Mammoth' will remain open for a while until I exclude it manually 😉

DiomedesofHelos commented 5 years ago

Yes, they have. :D

On Sat, 25 Aug 2018 15:12 Fons Leenaars, notifications@github.com wrote:

No problem, I'll close this issue again, I think the issues you found have been address in the staging environment now right?

The 'of the Mammoth' will remain open for a while until I exclude it manually 😉

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#issuecomment-415968371, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHEgGhcq3F7rqfDJcRNHGX6KMSeajks5uUU0zgaJpZM4UHY5R .

DiomedesofHelos commented 5 years ago

Hello! I just found an item "of the Mammoth"... This means that it is not unused??? What's that supposed to mean?

Den lør. 25. aug. 2018 kl. 21.29 skrev Fons Leenaars < notifications@github.com>:

Closed #39 https://github.com/fonsleenaars/tqdb/issues/39.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#event-1809109341, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHH8tIfeqywPTILn5iJwMG-EWb6vYks5uUaWzgaJpZM4UHY5R .

fonsleenaars commented 5 years ago

Oh that's great, send me a screen shot!

DiomedesofHelos commented 5 years ago

Den ons. 29. aug. 2018 kl. 19.36 skrev Asgerr Storgaard-Larsen < seth.storgaard@gmail.com>:

Hello! I just found an item "of the Mammoth"... This means that it is not unused??? What's that supposed to mean?

Den lør. 25. aug. 2018 kl. 21.29 skrev Fons Leenaars < notifications@github.com>:

Closed #39 https://github.com/fonsleenaars/tqdb/issues/39.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#event-1809109341, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHH8tIfeqywPTILn5iJwMG-EWb6vYks5uUaWzgaJpZM4UHY5R .

DiomedesofHelos commented 5 years ago

By the way, do you know if a high level item for an example a MI on Legendary can have ANY affix that fits to that sort of item? Like can a high level item like Bandari's Greaves from legendary even have a low level affix like "of Brawn" or "of Ability"?

On Wed, 29 Aug 2018 20:25 Asgerr Storgaard-Larsen, seth.storgaard@gmail.com wrote:

Den ons. 29. aug. 2018 kl. 19.36 skrev Asgerr Storgaard-Larsen < seth.storgaard@gmail.com>:

Hello! I just found an item "of the Mammoth"... This means that it is not unused??? What's that supposed to mean?

Den lør. 25. aug. 2018 kl. 21.29 skrev Fons Leenaars < notifications@github.com>:

Closed #39 https://github.com/fonsleenaars/tqdb/issues/39.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#event-1809109341, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHH8tIfeqywPTILn5iJwMG-EWb6vYks5uUaWzgaJpZM4UHY5R .

DiomedesofHelos commented 5 years ago

Or is it limited to affixes above a certain amount of level requirement?

On Wed, 29 Aug 2018 20:39 Asgerr Storgaard-Larsen, seth.storgaard@gmail.com wrote:

By the way, do you know if a high level item for an example a MI on Legendary can have ANY affix that fits to that sort of item? Like can a high level item like Bandari's Greaves from legendary even have a low level affix like "of Brawn" or "of Ability"?

On Wed, 29 Aug 2018 20:25 Asgerr Storgaard-Larsen, < seth.storgaard@gmail.com> wrote:

Den ons. 29. aug. 2018 kl. 19.36 skrev Asgerr Storgaard-Larsen < seth.storgaard@gmail.com>:

Hello! I just found an item "of the Mammoth"... This means that it is not unused??? What's that supposed to mean?

Den lør. 25. aug. 2018 kl. 21.29 skrev Fons Leenaars < notifications@github.com>:

Closed #39 https://github.com/fonsleenaars/tqdb/issues/39.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/fonsleenaars/tqdb/issues/39#event-1809109341, or mute the thread https://github.com/notifications/unsubscribe-auth/AlqwHH8tIfeqywPTILn5iJwMG-EWb6vYks5uUaWzgaJpZM4UHY5R .

fonsleenaars commented 5 years ago

Good question, I'll look into it.

I think maybe the screenshot didn't upload? I don't see any yet ;)