Rishabh Ranjan
Rishabh Ranjan
Hi, for this to be merged, can you please attach experimental results comparing the existing model with the GAT model you implemented. Thanks
@rusty1s can you please take a look?
merged in #328
Thanks for the PR @martinjurkovic ! Can you please address some comments: - [ ] remove `entity_col` and `time_col` because they are redundant. `entity_col` can be obtained as `entity_table.pkey_col` and...
Thanks @martinjurkovic ! I am in touch with @ValterH regarding this, and plan to review and merge soon.
Hi @martinjurkovic , I am closing this PR. I have merged @ValterH 's PR (#305) which includes your work. You are listed as a coauthor on this PR (in fact,...
Thanks for pointing this out @vladislavalerievich ! Since you have proposed the improvements, can you make a quick PR with these changes? We will be happy to merge to main!
Closing since the PR is now merged.
@XinweiHe @zechengz can you please look into this?
Thanks for the contribution @PKUTHM ! @kexinhuang12345 can you please upload this database and let me know so that I can merge it? Thanks