Zer0

Yes I'm using fighting back. It doesn't  fix anything. You know very well that "attack before being attacked" only works when you're like 8 yards away from the mob, rendering it useless, especially if you're a weak ranged class, like a mage. It's so bad that we both tried to fix this issue with our respective plugins.
Look, just give his profile a try and let me know how it goes, if you don't believe me. Wait till you reach the lower barrens, thousand needles, feralas, desolace etc. you'll know exactly what I mean.
You can't be serious. When I spoke up and gave you facts, you straight up told me I was wrong. Here's proof from your bug-report discord channel:

Mob names, level portion, location. It's all there. How more factual can I be, exactly? So please, don't act annoyed, telling me I don't give you facts. Let's take this example from 2 months ago, I'm telling you that two mobs are missing from the profile, and although I don't use your profile anymore, I remember very well that this specific grind loop is infested with those 2 mobs, it's not an occasional issue, it's constant and I remember dying more than 10 times an hour there because of countless disadvantageous/multiple body pulls. I've leveled 3 characters with your profile and every time it was a pain, with always the same issues, this one just being one of them. It's so bad that I remember switching to the automaton at specific portions because it straight up worked better.
So what will it be now? My FC sucks? I can't setup my wRobot properly? I'm undergeared? Explain how when I switched to Eeny's profile,  things suddenly worked much smoother, to the point that I can finally let the bot run without me babysitting it. If at first I hoped that you would fix things thanks to feedback from your bug report channel, I quickly understood that the channel is just a facade ,and you just counter argue or straight up ignore reports instead of seriously looking into potential issues. You're still doing that right now. Go ahead. Take a normal geared lvl37 mage to desolace and run your 37-40 portion, let me know how it goes.
And if you resort to bring up your "numbers that speak for themselves", I think I've made my point. I'm very glad you're making "hundreds" of people happy. Have your next Bier in Kreutzberg on my dime, bro. Cheers! ?

What an analogy. I'm not sure what the point of it was, but it only states that you're either viewing customers as children, or that you're some kind of benevolent Santa Claus. You are selling stuff here. Trying to make a profit. Don't fool yourself. 
This is really strange. I feel like there is some sort of elitism in this community. You're either pat of the godlike creators who can do no wrong, or you're part of the filthy peasants who understand nothing and should remain silent. You have a few people here complaining about the quality of paid stuff and you just call them "entitled millennials"? Come on.
If creators here were as philanthropic as some of you pretend they are, they would be teaming up and create free quality FC/profiles for the community. It might sound crazy, but I think the solution lies there. So, if you're a developer, or a tester and you're interested in participating, join my discord and let's actually move forward.

Maybe I didn't express myself clearly when I said people are in the dark. I'm talking about casual users. Those who don't know programming and who just want to level a few characters. You seem to be saying that anyone who uses wRobot needs to be an advanced programmer. How is wRobot advertised as such? It even says on the front page "Easy to use", "...for even the most novice user...".
So he downloaded wRobot and later realized that he's been "tricked". This is what enrages people.
It doesn't matter if the OP is a code genius or an HTML beginner. Although I disagree with a lot of what he said and he was clearly in over his head, he talked about his experience as a new user and the problems he faced and everyone should hear it. I faced the exact same problems: lack of free decent third party stuff. Actually, Profiles and Fight classes shouldn't even be called "third party" since they're so core to wRobot. This is the real subject: the shady practices behind wRobot and the reasons behind them. When people put money into FCs and profiles that suck, they feel robbed, they rage, they leave and never come back. That creates a thinner user base, a recalcitrant community... and less customers for you!

A lot of people are in the dark when it comes to wRobot and its current state. Bringing its issues to light can help some people realize that they're not necessarily using the bot wrong, and it can help them voice their concerns too. What do you propose? Silence discontent people?
You don't like the thread? Fine. But you realize you can choose to just ignore it, instead of actively ridiculing it, right? Believe it or not, I am interested in hearing people's opinions. All of them.

The first video is perfectly showcasing the biggest issue I have with wRobot paid profiles: the missing mob IDs. Because the "Attack before being attacked" option doesn't work properly, it makes those profiles a nightmare.
Imagine you make a grinding path with mobs along the way, and you add them all to the profile. The bot runs around and kills everything around, one by one. All good. Then a user contacts you and say "I'm playing a warrior and those scorpids which disarm you on the path are too hard to kill, fix it". What do you do? Well, one of the big vendor here just removes the mob ID from the profile. Then what happens? Sure you don't engage those mobs anymore, now, instead, you just run straight into their face while pulling other mobs. And when I reported the issue, the vendor told me he wouldn't fix it, because "it's much better like that" and "his profiles has leveled HUNDREDS (yeah, right) of characters without problem", basically putting the blame on me. And that's not even mentioning the enemy packs, the failure to take a zep, to go to Outlands etc...
How is that even tolerated?

With my Horde Remake Release - ive decided to relase Every single piece of code ive ever written, discovered, edited, messed with etc.
I put... MANY .. MANY Manyyy hours into this list. Because i have a disibility with my memory @elitecasaj00 Reccomended that i "write it all down". Using VS to create the code for the specific quest and adding to my cheatsheet.

Youll notice i have specific names for the // commands -- like // Gather, harvest, xgather - runcode - so i can CTRL-F stuff easier.
Not all credit was given proper to the authers of some of this code, however the ones i do know off the top of my head are @Droidz, @Matenia, @Smokie  @reapler 
and on some i tried copying the exact url of where i found it.
 
 
@Droidz Can you Pin this topic - I think it is sup sup helpfull !

I disagree. Nothing indicates that the bot tried to engage combat with the elemental. It just went straight for the bird. The elemental is ignored either because it's blacklisted or not in the profile.
This is most likely the profile's fault and it's incredibly common.

I don't know man, my experience has been a bit different from that. I've had glaring issues with a big grinding profile. I've reported bugs (things that I KNOW can be fixed) to a dedicated channel and I was just met with disbelief. It was a bit disheartening to realize that nothing would be fixed.
Reporting bugs is part of my real job, and I know much too well that if you can't put your ego aside when a report comes in, you're in for a bad time. I understand the frustration for big vendors. Maintaining all those profiles sounds like a nightmare, and a lot of users probably don't know what they're doing, but it seems, sometimes, vendors are quick to put the blame on the buyers, too.

If it's the first character you level up on the server, it will be hassle at first because of the lack of gold. On early expansions, it's not uncommon for some classes to eat/drink every 2 fights, especially if you're undergeared. Once you start racking up a lot of gold, you can have your next level 1s start with big bags and a big stack of gold. At this point, it won't matter if your toon buys 1 or 100 stacks of food/drinks.
I'd advise you start leveling with an easier class, like a hunter. If you're playing on TBC, I can provide you with a free fight class.

Please don't. It's obviously not a very healthy conversation, but it's raising valid points all around and i'm interested in hearing them. If controversial threads are instantly hammered down, then that means there is no possible conversation.

That's what I thought. I don't know how IsAttacked handles this, maybe the GoToTask used to get to the vendor inside the ToTown state has a exitCondition for getting in combat. 
Usually the exit condition for GoToTask is checking if the bot is running (not pause) and not being attacked. So that makes sense.

The bot SHOULDN'T check for other states while one state is still running. That's how FiniteStateMachine works. 
Maybe remove the ToTown state from the machine and build your own wrapper state that will abort the ToTown state (which should work by calling MovementManager.StopMove) if you pass/change a variable to your ToTownWrapper.

What you want to do won't really work without hacks. All of our code is more or less hacked together because we're changing wRobot's default behavior. @Marsbar, for his training plugin even removes wRobot's training state and adds his own.

Hi..
People often attempt to get into making stuff in C# for wrobot but often don't know where to begin. In this video tutorial I attempt to explain some of the basics of fightclass development in visual studio.
I've attached the solution I was showing in the video to this post.
ps. Sorry about my voice and my coughing, I'm currently a bit ill ? watching it back I also didn't touch on quite a few points. Let me know if you have questions and I can either do another video or just write out the answers.
I'll do a video on plugin creation (which I find more interesting) soon™
FightClassTutorial.zip

I guess that makes sense. IsAttacked checks for enemies targeting you (or you being on theit threat list in higher expansions) and tries to fight back. Maybe activate "don't fight back" in wRobot settings?:

Also I believe NPCScanState is started with the "newThread()" method (static) on the class, so it would make sense it's running in an additional thread to start a fight. Unfortunately I don't know all the inner workings of wRobot.

I'd say states are the way to go, that way other things won't interfere. Have a look at the grinder product example. Since it's a product when it loads the Bot it adds all the relevant states and has it's own grinder state. That should be enough of an example to craft your own.

Maybe just add your radar as its own state. That would, however, make it so the ToTown state running wouldn't allow your state to be triggered.
Like Mars said, look at the grinder product example to see how exactly Droidz' FSM works. Then take a look at my free PartyHelper plugin on how to add states to wRobot.
 

Hi!
There are often questions regarding the wrobot api and what functions it has, generally you're told to either decompile the dlls or reference them in your visual studio project and use the object browser.
Some people only want to know a certain function name and don't wish to go into the deeper realms of development. There isn't a resource they can use to obtain that info without some effort on their part, that's hopefully where this site I generated will come in handy:

Unofficial WRobot API Docs
It was created by decompiling the vanilla wmanager and robotmanager dlls and generating an api web template using docfx.
Currently it doesn't give you anything more than you'd have by decompiling the dlls yourself but I'm hoping to update this with method, property, constructer etc. descriptions and potentially basic code snippets for a better understanding of the wrobot api.
If you're interested in contributing let me know and I can add you to the gitlab project (sorry if you don't like gitlab but it was quick and easy with free private repos).

After further investigation, turns out there's a simpler way to handle the issue. Much cleaner, and works way better. It is confirmed that when you try to aggro a NPC that is not explicitly listed in the profile, AND when you're in a movement loop, the bot just tries to get back in the loop. The 3D radar helped a lot to understand what was going on. I still have a few (hopefully) minor bugs, and then I can release the plugin. Thanks again for your help, everyone.
// Pull hook robotManager.Events.FiniteStateMachineEvents.OnBeforeCheckIfNeedToRunState += (engine, state, cancelable) => { if (state.DisplayName == "Movement Loop" && inRadarCombat) { cancelable.Cancel = true; } };  

Hello, blacklist events is in "wManager.Events.OthersEvents"
And try
 

For some reason, even turning wManagerSetting.CurrentSetting.BlackListIfNotCompletePath to false (and in the UI) doesn't do the trick and the vendor still gets blacklisted.
I've found a workaround though. It's a bit shaky but so far it's working as intended. Works for both Vendor and Repair. Maybe there's a way to hook blacklist events directly? If not I'll clean up the code and use this for now.
private string vendorNpc = ""; // Manage blacklist (inside Initialize()) robotManager.Events.LoggingEvents.OnAddLog += delegate (Logging.Log log) { ManageNpcBlacklist(log); }; private void ManageNpcBlacklist(Logging.Log log) { if (log.Text.Contains("[ToTown] Go to vendor")) { vendorNpc = log.Text.Substring(22); vendorNpc = vendorNpc.Remove(vendorNpc.Length - 9); } if (inRadarCombat && log.Text.Contains("[ToTown] Unable to reach the vendor, blacklist it 120 minutes (you can disable this NPC in NPC DB tab 'Tools').")) { foreach (var n in NpcDB.ListNpc) { if (n.Name == vendorNpc) { Logging.WriteDebug("Removing " + n.Name + " from vendor Blacklist"); n.BlackList(-1); } } } }  

Would something like this work?
if(ObjectManager.GetWoWUnitAttackables().Any(x=> x.Position.DistanceTo(ObjectManager.Me.Position) < 20)) { MovementManager.StopMove(); Fight.StartFight(ObjectManager.GetWoWUnitAttackables().OrderBy(x => x.Position.DistanceTo(ObjectManager.Me.Position)).FirstOrDefault().Guid); } Edit: Untested, just a suggestion, also changed getwowunit to attackables
Edit2: looking at the above the if statement i have is bad, you probably already have a filter though