dendrograms / astrodendro

Generate a dendrogram from a dataset
https://dendrograms.readthedocs.io/
Other
37 stars 38 forks source link

Documentation fixes / review #82

Closed keflavich closed 10 years ago

keflavich commented 10 years ago

Mostly stylistic changes to the documentation, with a few 'todo' notes mixed in.

Wait to merge this.

keflavich commented 10 years ago

Ready for review.

astrofrog commented 10 years ago

@keflavich - thank you for reading over the docs and for all the fixes! See my comments above. For anything that still needs doing, you can always just start the comment with TODO so we can easily find it in future.

astrofrog commented 10 years ago

@keflavich - could you rebase? Thanks!

astrofrog commented 10 years ago

@keflavich - thanks!

@ChrisBeaumont - I will merge this tomorrow if there are no objections, and will address a couple of small points.

ChrisBeaumont commented 10 years ago

sounds good! On Oct 4, 2013 5:24 PM, "Thomas Robitaille" notifications@github.com wrote:

@keflavich https://github.com/keflavich - thanks!

@ChrisBeaumont https://github.com/ChrisBeaumont - I will merge this tomorrow if there are no objections, and will address a couple of small points.

— Reply to this email directly or view it on GitHubhttps://github.com/dendrograms/dendro-core/pull/82#issuecomment-25731141 .