Skip to content
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

Issue: Items in inventory occasionally gray out after clicking CleanUp #272

Closed
jclock opened this issue Jan 2, 2017 · 9 comments
Closed

Comments

@jclock
Copy link

jclock commented Jan 2, 2017

Unable to interact with the item until I /rl or zone.

@shagu
Copy link
Owner

shagu commented Jan 2, 2017

@shirsig ?

@shirsig
Copy link
Contributor

shirsig commented Jan 2, 2017

shirsig/Cleanup#1
Seems to be a server bug on nost.

@shagu
Copy link
Owner

shagu commented Jan 2, 2017

Alright, thanks

Maybe slowing down the sorting process within your CleanUp library could help?
Could be dynamically set according to the client->server->client delay,
or did you already tried that, @shirsig ?

Thanks for your reply

@shagu shagu closed this as completed Jan 2, 2017
@jclock
Copy link
Author

jclock commented Jan 2, 2017

Thanks for the quick response.

@shirsig
Copy link
Contributor

shirsig commented Jan 2, 2017

Well, the way the sorting works/what makes it so fast is that it takes advantage of the fact that you can do multiple moves in one step as long as they don't involve the same items. Between those steps it is already throttled. Thottling it during a step would require a rather big change as this is done simply with a loop.

Either way, it has nothing to do with delay, it must be an issue in nost's inventory scripting. And it's most likely not blizzlike as the old retail addon bag sort also used multiple moves per step.

By the way, do you know how to report something on the elysium bugtracker?

@shagu
Copy link
Owner

shagu commented Jan 3, 2017

Thanks for the explanation, that sounds really awesome, good job! :)

By the way, do you know how to report something on the elysium bugtracker?

No, never did that.

@jclock
Copy link
Author

jclock commented Jan 3, 2017

@shirsig are you talking about this link:
https://elysium-project.org/bugtracker/issue/add

@shirsig
Copy link
Contributor

shirsig commented Jan 3, 2017

Thanks, couldn't find a link to that on the tracker.
Anyway, reported it: https://elysium-project.org/bugtracker/issue/966
Please upvote :)

@jclock
Copy link
Author

jclock commented Jan 4, 2017

No problem upvoted, thanks for all of your help. Also, I especially like:

CORRECT BEHAVIOR
They don't.

SOURCES
Doh.

@shagu shagu mentioned this issue Nov 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants