Well, I make no promice either but I have looked at TSI a bit the first time wmrojer took a little leave of absence (BTW man, thanks for all you've done).
Right now, I believe that he used a script to extract the recipes from WoWHead or something like that. I'll have a look this weekend to see if that can be replaced by PT3 classes. If someone can do it before me though, go right ahead.
As for rewrite, it might be a good idea medium or long term but for now, we should look into a way to make the existing TSI up to date and fully usage fast. I saw yesterday that some Jewelcrafting designs were missing and I'm thinking other stuff must be missing to.
Well this is excellent news from all of you. I was ready to try and find a similar mod, but I have yet to get lucky. Nothing is better than clicking an item in your inventory or anywhere (AH, Vendor, , etc.) to know exactly where and in what recipes it is used in even if i don't have the profession. The value of items is much more understandable if you have that knowledge at your fingertips.
I already ran into an issue with the changed enchanting recipes in the game that were there previously, ie enchant bracers spell power went from 8 x primal fire and 8 x primal mana to 6 a piece. When I tried to show this to someone it closed my client because it of course no longer exists this way.
I wish i knew how to code so i could help you out, but please keep us posted as to what solution you end up developing.
Well, I make no promice either but I have looked at TSI a bit the first time wmrojer took a little leave of absence (BTW man, thanks for all you've done).
Right now, I believe that he used a script to extract the recipes from WoWHead or something like that. I'll have a look this weekend to see if that can be replaced by PT3 classes. If someone can do it before me though, go right ahead.
As for rewrite, it might be a good idea medium or long term but for now, we should look into a way to make the existing TSI up to date and fully usage fast. I saw yesterday that some Jewelcrafting designs were missing and I'm thinking other stuff must be missing to.
I did look into it but there are some data not present in PT3 that is needed in TKI and I don't know how wmrojer was extrating them from and from what database.
I'll keep looking but I don't feel like entering those by hand.
TSI wasn't updating known recipes with Manufac installed because the trade skill frame was never shown so I changed TSI to rely on GetTradeSkillInfo and GetCraftSkillInfo instead of a shown frame. Works great for me so far. Diff is attached.
TSI wasn't updating known recipes with Manufac installed because the trade skill frame was never shown so I changed TSI to rely on GetTradeSkillInfo and GetCraftSkillInfo instead of a shown frame. Works great for me so far. Diff is attached.
Hope this is of use to someone
Ivan
Why don't you commit the change to the SVN server?
Don't have svn write access yet. I have found that every once and a while with the change an UpdateKnownTradeRecipes debug message is printed. I've had it show up twice in the last 3 days, haven't looked into it much yet.
Don't have svn write access yet. I have found that every once and a while with the change an UpdateKnownTradeRecipes debug message is printed. I've had it show up twice in the last 3 days, haven't looked into it much yet.
Try sending a PM to one of the moderators then, I'm sure they will commit the change for you. And thanks for the fix, by the way.
Is there something i should be looking out for. I did not receive any errors but I am not noticing anything different either seems to be functioning as it did before. Man I wish we could update teh recipes so the link would actually work in chat. :-)
I am also using Manufac of course. I assume this upadate will help determine which tradsekill recipes I already have by the TSI color scheme that I don't buy the same recipe twice or buy the recipe for another character on the same account?
I have it fixed now, a spelling error was causing TSI to try to update known recipes when a skill point in anything was gained, i.e. weapons or defense. Committing hopefully the last bug fix for the change. I will look into recipe links shortly
Is there something i should be looking out for. I did not receive any errors but I am not noticing anything different either seems to be functioning as it did before. Man I wish we could update teh recipes so the link would actually work in chat. :-)
Regarding the recipe links, would you rather have the actual recipe link put in chat instead of the recipe name and reagents or how it is now but with the name as the link? Shouldn't be to hard to do if thats what you are thinking of.
Actually when I link a recipe it is in White text when it should come out Yellow. I believe this happened for the 2.1 patch. So when I click some of the recipes that have different mats prior to 2.1, it sometimes crashed my client. An example s the enchant for spell power to bracers, the mats were reduced after the patch.
But an update to the recipes would be solution to this. Unless there is something I am missing.
Beyond that, the best part of TSI was looting something and then opening the gui that described the other recpies it was used in even in those tradeskills that i did not have. This was my favortie thing about TSI. And the longer the recipies do not get updated the longer it looses its value.
But I am not sure how to help as I am not a developer.
Thanks again for updating the mod, after testing it tonight no issues came up at all.
Regarding the recipe links, would you rather have the actual recipe link put in chat instead of the recipe name and reagents or how it is now but with the name as the link?
I'd much rather see the recipe links, since they now include a listing of required mats anyway. Or perhaps you could include both, allowing one way of linking as a default and the other with a different modifier key. Could be a simple boolean option to switch between the two.
(It's been a while since I've used TSI, as the links were constantly crashing my client, so I'm a little fuzzy on what the default behavior is now.)
Rollback Post to RevisionRollBack
To post a comment, please login or register a new account.
Well this is excellent news from all of you. I was ready to try and find a similar mod, but I have yet to get lucky. Nothing is better than clicking an item in your inventory or anywhere (AH, Vendor, , etc.) to know exactly where and in what recipes it is used in even if i don't have the profession. The value of items is much more understandable if you have that knowledge at your fingertips.
I already ran into an issue with the changed enchanting recipes in the game that were there previously, ie enchant bracers spell power went from 8 x primal fire and 8 x primal mana to 6 a piece. When I tried to show this to someone it closed my client because it of course no longer exists this way.
I wish i knew how to code so i could help you out, but please keep us posted as to what solution you end up developing.
Good luck and thank you.
That's very good news!
I'll keep looking but I don't feel like entering those by hand.
If it's poisons info, I've already collected it by hand for PT3, but the info hasn't been put in by anyone that can do it.
If it's not poison data, well, I don't mind collecting info by hand again =)
I'm happy to do mindless grunt work, if it will help someone capable of updating this mod.
Hope this is of use to someone
Ivan
I am not a dev person so any help would be grateful I would love to test this code out.
Why don't you commit the change to the SVN server?
Try sending a PM to one of the moderators then, I'm sure they will commit the change for you. And thanks for the fix, by the way.
Ivan
Was is your estimated time of update.
I LOVE THIS MOD!
Thank you for your efforts.
If you need me to test just post here and I will gladly do this :-)
Ivan
[edit]
lol first svn commit here and I mess it up, should be good now
[edit]
think I just found what was causing the error I was getting, will keep testing for a bit and commit after it doesnt show up anymore
Is there something i should be looking out for. I did not receive any errors but I am not noticing anything different either seems to be functioning as it did before. Man I wish we could update teh recipes so the link would actually work in chat. :-)
I am also using Manufac of course. I assume this upadate will help determine which tradsekill recipes I already have by the TSI color scheme that I don't buy the same recipe twice or buy the recipe for another character on the same account?
Thanks again.
Ivan
Regarding the recipe links, would you rather have the actual recipe link put in chat instead of the recipe name and reagents or how it is now but with the name as the link? Shouldn't be to hard to do if thats what you are thinking of.
Ivan
But an update to the recipes would be solution to this. Unless there is something I am missing.
Beyond that, the best part of TSI was looting something and then opening the gui that described the other recpies it was used in even in those tradeskills that i did not have. This was my favortie thing about TSI. And the longer the recipies do not get updated the longer it looses its value.
But I am not sure how to help as I am not a developer.
Thanks again for updating the mod, after testing it tonight no issues came up at all.
I'd much rather see the recipe links, since they now include a listing of required mats anyway. Or perhaps you could include both, allowing one way of linking as a default and the other with a different modifier key. Could be a simple boolean option to switch between the two.
(It's been a while since I've used TSI, as the links were constantly crashing my client, so I'm a little fuzzy on what the default behavior is now.)