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.

Sign up for GitHub

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

Jump to bottom

Scale #11

Merged
merged 3 commits into from Oct 21, 2014
Merged

Scale #11

merged 3 commits into from Oct 21, 2014

Conversation

pathway
Copy link

@pathway pathway commented Oct 19, 2014

Please rework as you like. I may go further but I will like be busy for some time.

Issues:

  • NodePriorityQueue used BOTH a queue AND an array, just so that contains() and get() are cheap, but it seems wasteful
  • NodePriorityQueue remove() borken.
  • test coverage
  • coding standards

@coveralls
Copy link

Coverage Status

Coverage decreased (-61.02%) when pulling 4e0b765 on pathway:scale into f59ab60 on jmgq:feature/efficientOpenList.

jmgq added a commit that referenced this pull request Oct 21, 2014
@jmgq
Merge pull request #11 from pathway/scale
07a3da8 
Scale
@jmgq jmgq merged commit 07a3da8 into jmgq:feature/efficientOpenList Oct 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@pathway @coveralls @jmgq