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

OrderedDict for names? #19

Closed
phaverty opened this issue Jan 27, 2016 · 5 comments
Closed

OrderedDict for names? #19

phaverty opened this issue Jan 27, 2016 · 5 comments

Comments

@phaverty
Copy link

The OrderedDict type from DataStructures.jl might be a nice data structure to hold the names. OrderedDict will soon be based on a speedy implementation by Jeff Bezanson. OrderedSet might also be appropriate.

@davidavdav
Copy link
Owner

Yes, thanks, this is a good suggestion. I didn't know ordereddict was around.

@davidavdav
Copy link
Owner

Is OrderedDict faster than Dict?

@nalimilan
Copy link
Contributor

I don't think it is, though I think Jeff's implementation is quite fast (i.e. more or less as fast as Dict). If it were faster, Dict could be implemented using OrderedDict.

@phaverty
Copy link
Author

I got the idea from this pull request:
JuliaLang/julia#10116
It is faster for certain things. It seems that it was considered for use in
base julia, but discussion is ongoing.

Pete


Peter M. Haverty, Ph.D.
Genentech, Inc.
phaverty@gene.com

On Mon, Jun 27, 2016 at 6:21 AM, Milan Bouchet-Valat <
notifications@github.com> wrote:

I don't think it is, though I think Jeff's implementation is quite fast
(i.e. more or less as fast as Dict). If it were faster, Dict could be
implemented using OrderedDict.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#19 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AH02K53o77qqYYxswqH4RH8Pim0nuM2oks5qP85qgaJpZM4HM_wP
.

@phaverty
Copy link
Author

Resolved by PR #31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants