Need help?
<- Back

Comments (28)

  • juancn
    That's usually true of all common hash table implementations (when objects don't have a defined order, if they have you can get O(1) average and O(log N) worst case), regardless of language.The O(1) is the expected average case, which usually holds.Yeah, O(N^2) is theoretically possible, but unless you're defending against some sort of denial of service attack, in practice it rarely matters.Still, if you can guess a sensible initial size for a hash table you can avoid a lot of the overhead of rehashing.
  • northisup
    Raymond Hettinger has a great talk about how much python's dict has improved over the years. So this is super interesting and will probably just make the builtin dict better eventually.The lesson of the talk is that if you are idiomatic then you will benefit as the language improves.https://www.youtube.com/watch?v=npw4s1QTmPg
  • brudgers
    But could we create a hash table that would be truly constant-time? No. As the size of your data structure grows, it requires progressively slower memory.At a large enough size to be interesting, everything is dominated by IO and because IO is slow, at any interesting size performance is a matter of tailoring the implementation to the details of the data {0}.Engineering is hard work, not naive math.[0] Data might be arbitrary but it is never random. Not being random is what makes it data.
  • aadyachinubhai
    That is why I like to use __slots__ when defining a class. Unlike dicts, using __slots__ is a tuple so using it to store class attributes is much faster.
  • ok123456
    Most algorithm analyses don't incorporate memory hierarchies. I'm not sure what the point of this post was.If you are really concerned about it, compute the empirical roofline for your machine.Also, if the point is to point out memory hierarchies, it's not 'quadratic performance.' The algorithm doesn't behave differently once it spills over. The costs just get bigger.
  • TristanDaCunha
    Which statement in this article applies only to Python?
  • ot
    O(1) doesn't mean constant, it means bounded by a constant. An algorithm can be faster with small n and converge to a horizontal asymptote as n goes to infinity, and it would still be O(1).In a real machine there is no infinity, but hundreds of GBs of memory are "infinity enough" compared to the cache size [1]. So asymptotic analysis is still a decent model.I'm surprised that even a CS professor confuses this.[1] Ok if we want to be pedantic memory access is logarithmic due to the traversal of page tables, but you can use huge pages.
  • emil-lp
    > To put it differently, saying that a hash table is O(1) or constant time is a modelNobody really says that, nor is it a model. It is the expected time complexity.
  • done_lurking
    Time complexity is something that applies to algorithms, and is determined analytically. I don't think it's useful to equivocate the definition with performance of implementations of algorithms determined through real world data. Both of these things are important, but they are not the same. The fact that they differ is not very surprising and does not necessarily mean that a misunderstanding has occurred.
  • anon
    undefined
  • stkdump
    Once a hash table has outgrown all caches, it should have linear performance. It's just that caches accelerate it at sufficiently small sizes.
  • coolThingsFirst
    Lemire back again with CS101 insights.Next, he will teach us doubles have finite precision.
  • javcasas
    Java's HashMap also has O(log(N)) complexity on hash collision, and that is before memory/cache details.https://docs.oracle.com/javase/8/docs/api/java/util/HashMap....In fact, some studying on data structures probably leads to the conclusion that it is impossible to guarantee that an unbounded set/map to have access performance under O(log(N)).
  • jldugger
    Uh, what is going on with this benchmark?Why is M so big? Why does it cross the maxint boundary? Why is constructing the list comprehension part of the benchmark? Why are we summing the set? Why are we only measuring 5 values for n?
  • 0xa2
    The map is not the territory.
  • oefrha
    This "quadratic-time performance" is incredibly disingenuous. First, it's doing n operations that are each O(n), so it's more like "can have linear time performance, but done n times so I can give you a scary title".Edit: A charitable take is constructing a set/dict from a list is indeed a common operation so it's worthwhile to think about its complexity, but it's not really one of the standard operations when discussing the performance of a hashset/hashmap, so really shouldn't be this handwavy.And instead of attacking some straw man "It is indeed widely believed that ..." claim (widely believed by who?), why not attack what's literally on docs.python.org? https://docs.python.org/3/library/time-complexity.html:> dict> The times listed for dict objects are average-case times, as they assume the hash function for the objects is sufficiently robust to make collisions uncommon. They also assume the keys are well-distributed among the set of possible keys. In the worst case, when every key hashes to the same value, each of the O(1) operations below instead takes O(n) time. They also assume that hashing and comparing a key is O(1). For more detail on the implementation, see How are dictionaries implemented in CPython?.> ...> set, frozenset> See dict as the set and frozenset implementations are similar, and the same caveats apply. In the worst case, O(1) operations instead take O(n) time, and operations that look up every element degrade accordingly. +--------------------------------------+------------+ | Operation | Complexity | +--------------------------------------+------------+ | x in s | O(1) | | Copy (s.copy()) [6] [7] | O(n) | | Add (s.add(x)) [1] | O(1) | | Discard (s.discard(x), s.remove(x)) | O(1) | | ... | ... | +--------------------------------------+------------+ You explicitly construct a list of ints that are all multiples of sys.hash_info.modulus and hence all hash to 0, no shit you get that well documented O(n) behavior.The discussion of CPU cache is good though, so why hide that behind this clickbait.
  • jaydeepg
    [flagged]
  • vyftec_wpsec
    [flagged]