Alex Loftus

> why make this a wrapper, as opposed to an option? I thought it was a little more clunky as an option, since turning it into an option implies that...

> @loftusa what's the status of this PR, left a few comments above but wondering if we should push in or close? I think probably `binary_heatmap` is unnecessary / overengineering,...

@bdpedigo status on this? I saw approved, but I'm not authorized to merge

> tests still werent passing looks like they broke yesterday with 44f372e. One sec

> @loftusa any interest in getting this merged? if not will close for the time being Yeah - the doc-building error was weird and I wasn't sure how to fix,...

`AttributeError: type object 'gensim._matutils.array' has no attribute '__reduce_cython__' ` what the heck is this

@bdpedigo Possibly, but I'd prefer to work within the codebase itself as much as possible at the moment I think

I like this issue! I appreciate your multi-paragraph issue descriptions, Dwayne 🙂 I think (although am not sure) that we have the paid version of netlify, which might be nice...

> @loftusa I cannot understand this. can you explain a bit more if you have some free time. yeah, see #665. I mean that the sidebar on the left [here](https://microsoft.github.io/graspologic/tutorials/index.html)...

Yeah, sounds good to me, just need to add a link back to the main page as well and then we're golden