search

deepchem / moleculenet

Moleculenet.ai Datasets And Splits
MIT License
88 stars 19 forks source link

SIDER dataset added #26

Closed yuanqidu closed 3 years ago

mufeili commented 3 years ago

Looks great in general! I left some minor comments.

yuanqidu commented 3 years ago

Thanks so much, mufei. Sorry about the errors. Some were left off while I was testing the code. I will correct them and PR again very soon. Happy Lunar New Year!

yuanqidu commented 3 years ago

I have corrected all of them, including fixing some \newline mismatch. Thanks~

mufeili commented 3 years ago

@yuanqidu Looks good to me. Happy Lunar New Year! If you have bandwidth, you may start working on the KAGGLE dataset on a new branch.

@rbharath I think this PR is ready to merge.

rbharath commented 3 years ago

Happy Lunar New Year as well!

Going to go ahead and merge