Dave Eargle

This is a hack, but I'm calling the apparmor cookbook immediately after creating the mysql_service: ``` mysql_service 'default' do initial_root_password "#{node[:mysql][:root_password]}" bind_address '0.0.0.0' port '3306' action [:create, :start] end include_recipe...

Poor consolation, but FWIW there exists https://sandbox.zenodo.org to make trial-and-error easier with dummy repos

I'm not sure this is a good idea anymore. On the github readme, we want to have a link to the documentation. But if we use the github readme as...

I missed that. Yes, I think that would be sufficient. Are you okay with adding a new dependency to the docs build? [`m2r2`](https://github.com/crossnox/m2r2) would permit us to import a markdown...

@sauln can you review this?

> If you cover the interval [0,1] with 2 cubes with 50% overlape the cover you should get is [0,2/3] and [1/3, 1]. I am not a smart man, let...

Thank you for the very, detailed, well-reasoned, and well-sourced answer! I'll let @sauln respond next. And I don't think we have a formal policy yet about "open an issue first,...

It looks like you were the one who authored the commit that added the fixed `min_cluster_samples = 2` for precomputed? Why was fixing that necessary? I haven't used kepler-mapper recently,...

If I loosely understand you commit message https://github.com/scikit-tda/kepler-mapper/commit/2ccce2d891165eead157f320fbc0a052c54c5540 , then it's possible that the clusterer not have `n_clusters` set. The logic here: ``` min_cluster_samples = cluster_params.get( "n_clusters", cluster_params.get( "min_cluster_size", cluster_params.get("min_samples",...

Want to issue a new PR that still works for your usecase which was breaking, and which re-adopts that more flexible logic?