JuliaCollections/DataStructures.jl

BinaryHeap constructor performs differently from heapify

Aperta

#639 aperta il 29 giu 2020

 (3 commenti) (2 reazioni) (0 assegnatari)Julia (261 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (745 stelle)
Metriche merge PR
 (Merge medio 1g 18h) (3 PR mergiate in 30 g)

Descrizione

I noticed that BinaryHeap like BinaryMinHeap constructs a heap that just insert every element from an array to a empty heap. And the following x and y have the same order:

julia> nums = rand(1:20000, 2000);

julia> x = MutableBinaryMinHeap(nums);

julia> y = BinaryMinHeap(Int.([]));

julia> for i in nums 
           push!(y, i)
       end

However, there is an O(N) heap-building algorithm instead of O(NlogN), and function heapify is an implement. So why not use heapify instead of insert successively?

Sincerely.

Guida contributor