mims-harvard / TDC

Therapeutics Commons (TDC-2): Multimodal Foundation for Therapeutic Science
https://tdcommons.ai
MIT License
1.02k stars 174 forks source link

pd.append -> pd.concat #318

Closed drewnutt closed 1 month ago

drewnutt commented 2 months ago

Problem: Pandas append has been deprecated since version 1.4 and has been removed since version 1.6. create_group_split still uses pandas.append to split the train_val set into the train and val sets.

Solution: Updates create_group_split to utilize pandas.concat to create the train and val sets.

amva13 commented 1 month ago

hi @drewnutt , we can merge this when it passes the yapf linter. You can do this yourself or I can take care of it at a later time. Thanks for the contribution! Feel fee to tag me if you make the yapf change yourself