som-shahlab / survlearners

Metalearners for Survival Data

Home Page:https://som-shahlab.github.io/survlearners

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Remove @utils directive

erikcs opened this issue · comments

This line: https://github.com/som-shahlab/survmetabench/blob/master/survlearners/R/scoxph.R#L1 can be removed from all files @CrystalXuR, if you want to try and do a simple branch for that

Sounds good. Trying it now

Remeber to do a search in the entire repo for @utils and remove it from all files

I removed all the ones under the R folder, but do I also need to remove the one at
https://github.com/som-shahlab/survmetabench/blob/master/simulation_hpc.R#L25

Are you going to send new PRs for these issues? you closed and deleted all the open ones?

I am going to start over and send new PRs for each issue. I intended to merge my changes under #44 to master after you approved it, but it did not happen. I still do not know why, and I think I lost all the changes I made since last night :(

Ok, it sounds like you learned a lesson the hard way = less likely to make the same mistake twice! your changes are most likely still on your local git

really, Is there an easy way to check it