-
Notifications
You must be signed in to change notification settings - Fork 46
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
Subtlety Rogue error #435
Comments
Hmm...have you tried to check out a new (clean) JPS version? |
That is using the JPS-master downloaded about 4 weeks ago. I have not tried JPS-1.3.2. I will do that this afternoon. |
I see that there have been many changes since 1.3.2 so I will assume that the JPS-master version I have right now is the new clean version. Assassination is the only Rogue rotation I have been able to make work since all I had to do was take out Slice and Dice as it is now passive for Assassination. Is there a forum to share profiles and changes like there used to be on the PG forums? |
Hmm...nope 1.3.2 is an old release,the current master should work...at least most of it. There is a forum - see issue #397 - but there isn't much talk and no current rotations. You can either use ProbablyEngine (which is actively developed - see OwnedCore for rotations) or keep using JPS but write your own rotations. I currently waste most of my time developing my bot...but maybe I'll try to rewrite JPS from scratch, I'm not that happy with PE Rotations....or I'll try to convert my warlock rotation to PE... |
I also dislike PE. There is at least one JPS rotation working for every class though some do take quite a bit of cleaning up. All three specs for hunter and DK work great out of the box iirc. I will post some other issues. If you are able to help, that's great. If not, no big deal. While I will probably not partake in your bot that you are creating since I had all my accounts banned two years ago for running a gold selling operation, I would most definitely donate $$$ for a rewriting of JPS. |
I'm still kinda working on JPS. I have a sick moonkin rotation atm that does multidotting, aoe detection, and the ability to cast spells without facing the target. I might upload it? But I dunno who's still using JPS or moonkin :P |
In my opinion JPS is far superior to PE. I think a lot of PE users would use JPS if they knew about JPS. I leveled one of my Druids to 90 as moonkin then switched to feral. I thought the stock moonkin rotation worked pretty good. But I would try anything that was uploaded if you wanted to do it. |
Well I'm using JPS...but mostly on Warlock...but your moon kin rotation would be interesting. I'm using my custom WarlockRotation... But JPS requires a major refactoring, maybe a little change in the condition statements...so simcraft rotations can be easily imported... |
Well I can tell you from experience that PE pales in comparison to JPS which makes me sad that JPS is no longer being worked on. I have a SP that need a shadow rotation BADLY but its broke, Pally that has 0 rotations because I think something is missing, Rogue that just sucks for rotations....so far EL Shaman is decent and Hunters are really good. I really wish I wasn't coding challenged otherwise I would start fixing some of these rotation myself. |
Well...I used the last few days for a complete refactoring of the JPS code. Well, actually I completely rewrote half of the JPS core and took the other half from JPS or removed it. |
Wow that's great! I wish there were something I could do to help out. I have no idea how much work that took but it sounds like a lot. |
Kirk: Are you going to fork your build? If so please let us know what and where its going to be located; |
It will be located at https://github.com/kirk24788/KPS - currently it is only available in my private Git Repository. There are still a few TODO'S in my code, and I have to do a performance analysis. |
If someone is still working on JPS thanks for the addon, I love it and have used it off and on since Cataclysm so Thanks to you guys.
Over the last couple months I have done my best to fix some of these but I just end of causing more errors because of my ignorance.
jpevents.lua:247 - Error Interface\AddOns\JPS\modules\jpauras.lua:49: bad argument #1 to 'tostring' (value expected) on OnUpdate function function: 000000003C7ABDE8
The text was updated successfully, but these errors were encountered: