Jinu Sunil

Results 10 comments of Jinu Sunil

I added the token as a secret to `wsad1/pytroch_tabular` repository, I guess you should be able to see that. Then I used this secret as the `password` for deploying to...

Ah I see. So a secret can be tied to an environment or a repository, as explained [here](https://docs.github.com/en/actions/security-guides/encrypted-secrets). If its tied to the repository , like I did for this...

@rusty1s, we need to resolve a few things before merging. Will look at it this week and get back here.

> Hi! > > Thanks for the great effort with this PR. Is there any update on getting it verified and merged? > > Also, in the `GNNExplainer` class there...

@rusty1s , this is ready to merge. please take a look.

Thanks for the Issue. 1. This is a good idea. I think it should straightforward to add an argument `edge_list` to [subgraph](https://pytorch-geometric.readthedocs.io/en/latest/modules/data.html#torch_geometric.data.HeteroData.subgraph) and filter out edges not in `edge_list`. 2....

@otaviocx Thanks for this, its a great plan. One thing to think of after we are done with the initial 5 steps you mentioned, is to think of how to...

I think we should add this point to the roadmap * Implement a concrete `FeatureStore` using some "popular" storage backend. This will help us "test" the interface, and also demonstrate...

Also we could add * Since `FeatureStore` and `MaterializedGraph` are independent. It would be nice to have `validate(FeatureStore, MaterializedGraph)` which checks things like 1. `MaterializedGraph` only connects `node_type` present in...

Thanks for sharing @advil64. We don't have the bandwidth to implement this currently. But happy to accept PRs.