Inventory.dropAll weird behavior on mirror mode

[dankdrcode]

I made a script that works well on stealth mode but when using mirror mode the behavior is not the same when using Inventory.dropAll(). 

In stealth mode it takes ~10 seconds to drop a full inventory.

In mirror mode it takes +30 seconds to drop a full inventory.

Is this normal behavior? I'm reporting it as a bug because it seems so strange and robotic.

 

This is my code : 

inventory.dropAllExcept("Feather", "Fly fishing rod");

I'm not using any while loops or sleep commands...

 

If this isn't a bug, can someone please explain the discrepancy between the two? I'm still learning so any info is appreciated

[Camaro]

Completely normal. Some systems take a couple seconds to perform any inventory option on mirror mode. inventory.dropAll happens to perform many separate inventory options sequentially, so its going to take forever. You'll have better luck creating a custom dropAll method using getMouse().click(new RectangleDestination(getBot(), getInventory().getSlotBoundingBox(slotNum)))

[Fay]

1 hour ago, Camaro said:

Completely normal. Some systems take a couple seconds to perform any inventory option on mirror mode. inventory.dropAll happens to perform many separate inventory options sequentially, so its going to take forever. You'll have better luck creating a custom dropAll method using getMouse().click(new RectangleDestination(getBot(), getInventory().getSlotBoundingBox(slotNum)))

This is a great recommendation. @Camaro what is your take on the usage of the built-in functions like inventory.dropAll()? 

[Gunman]

2 hours ago, dankdrcode said:

If this isn't a bug, can someone please explain the discrepancy between the two? I'm still learning so any info is appreciated

Lower the reaction time to 50 or do what Camaro said

[dankdrcode]

Thank you! @Camaro The performance difference is huge. 

 

As a side note, I also tried this for a custom drop method but Camaro's method is still much faster. 

mouse.move(new InventorySlotDestination(getBot(), i));
mouse.click(false);

 

 

 

 

 

[dankdrcode]

1 hour ago, Gunman said:

Lower the reaction time to 50 or do what Camaro said

The reacting time is already set to 50. I played around with it and it didn't seem to make a difference.  

[dankdrcode]

2 hours ago, Fay said:

This is a great recommendation. @Camaro what is your take on the usage of the built-in functions like inventory.dropAll()? 

And if you don't mind answering this : What are some methods that you personally stay away from and what alternatives do you use instead? 

 

I wish these performance issues were documented somewhere.... 

[Fay]

On 1/9/2022 at 4:49 PM, dankdrcode said:

And if you don't mind answering this : What are some methods that you personally stay away from and what alternatives do you use instead? 

 

I wish these performance issues were documented somewhere.... 

I think it just comes down to personal experience with the use of the API. If you are building your own bots, you'll get enough runtime with them to understand ways to optimize. The built in methods IMO are ways for people to jump right in and adopt the platform.