search

cesium-ml / cesium

Machine Learning Time-Series Platform
Other
670 stars 101 forks source link

Fill out a few function docstrings #196

Closed acrellin closed 7 years ago

acrellin commented 7 years ago

@stefanv addressed your comments - want to take another look?

bnaul commented 7 years ago

👍 here

acrellin commented 7 years ago

@bnaul took me a minute to see my typo... thanks

acrellin commented 7 years ago

@stefanv I've updated the docstrings - please take a look when you have a moment.

bnaul commented 7 years ago

👍 from me

bnaul commented 7 years ago

oh right still waiting on feature guidelines...do those live here or will it just say "see the same file in cesium-ml/cesium"?

acrellin commented 7 years ago

It'll live here in CONTRIBUTING.md, which I've created.

acrellin commented 7 years ago

I think the cesium_web docs can just refer here for those guidelines.

acrellin commented 7 years ago

@stefanv I'd love to move on this and update CONTRIBUTING.md in a separate PR