Carlos Paradis
Carlos Paradis
I just noticed this last week message now. I am assuming you clarified this on call!
@daomcgill Heads-up: I am reviewing this one now, so don't commit to this PR to avoid merge conflict.
Oops, it is #320 that I need to review first, these are the execs. Not locked anymore until I come back around this.
@daomcgill hi Dao, I am unfortunately out of time to fix these. Could you add the get here? @crepesAlot @nicoelee123 @RavenMarQ please make sure as you code review each other,...
Working on this PR now, please hold on doing commits.
@daomcgill i suggest you fix this PR first with the get and the other notes, assuming this is the one that came before the exec PR. Then you can merge...
@daomcgill could you confirm the notebook display the parsed tables so the users can see what each function outputs as i did with parse_mbox on your mail notebook?
@daomcgill you sure did! It is very much in line with the changes I did on mail and that you see on itm0, there is no better way to showcase...
Thank you!!
Seems there are still merge conflicts.