-
Notifications
You must be signed in to change notification settings - Fork 169
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
Doesn't work with big numbers (probably long!) #40
Comments
It's because priority values are floats, which have a precision of 23 bits (about 7 digits). I have an idea for the future on how to fix this, but I haven't found time to implement it yet. [Edit] Oh, I just noticed you're using |
Oh, I didn't know that. I was using SimplePriorityQueue and it doesn't say that limitation on Wiki page or any other place I looked for, maybe you should consider to add that to let people know beforehand. |
It's mentioned in the XML documentation which should pop up when you use the class in Visual Studio |
` SimplePriorityQueue priorityQueue = new SimplePriorityQueue();
`
I tried to change the numbers from example, and you dequeue them in order you queue them like normal queue, It's expected that you should dequeue them as 1, 2, 3.
I don't know it's related to the numbers being big (not support long etc). Or maybe I tried to use it wrong way. I should be able to use it with these numbers because they're timestamp values and I want to order the queue with them
Thanks
The text was updated successfully, but these errors were encountered: