[issue39634] Incorrect heapq heapify naming

2020-02-14 Thread Harsh Patel
Harsh Patel added the comment: This ticket is not intended to rename heapify method in the stdlib (its too late for that). This is to merely add a clarification in the docs to state that heapify here has a different connotation than what's used in standard algorithmic literature (CLRS, Sedgewic

[issue39634] Incorrect heapq heapify naming

2020-02-14 Thread Steven D'Aprano
Steven D'Aprano added the comment: The "-ify" or "-fy" suffix in English means "to make". For example, "amplify", "magnify", "terrify", etc. https://en.wiktionary.org/wiki/-ify So heapify literally means "make into a heap". Not only is the name correct, but it is a better name for an in-pla

[issue39634] Incorrect heapq heapify naming

2020-02-14 Thread Harsh Patel
New submission from Harsh Patel : heapify method is a misnomer in that it is actually the make-heap or build-heap procedure from textbooks -- components: Library (Lib) messages: 361996 nosy: hp685 priority: normal pull_requests: 17888 severity: normal status: open title: Incorrect heap