Thomas Bonnin

It is better for sure but still easy to shoot yourself in the foot by using the default 'master' value. I wonder if we should find a way to set...

The solution seems to be adding `println!("cargo:rustc-link-lib=torchvision");` in `torch-sys/build.rs` main function and `#include ` in `torch-sys/libtch/torch_api.cpp` (for instance) for the torchvision operators to be registered. It would be nice indeed...

how many customers use the gong integration in prod. If number is small, can we coordinate with them to update their connection instead of making the providers definition and code...

> Looks good, I haven't run it but the logic seems good 🙏🏻. I have multiple feedback/question, nothing blocking: > > * How do we handle deletion? > > *...

> Curious about how you tested this and what is the plan for running this on staging? I have tested using prod replica as the source and localhost as target....

> @TBonnin sure, done. Any timeline for when this can be merged? Thanks Can you rebase your branch? Then I will merge it before the end of the day

> No way around RDS being self signed? couldn't find anything. The only cert authority options are `rds-ca-...`

It needs to be addressed but it is not at the top of my list. I am basically keeping it to remind me to take care of it at some...

> As I said earlier, not a fan because it add a problem on top of the problem, and we don't know the behavior of this feature. Also if we...

tl;dr: this PR objective is not to trim CLI dependencies. It is about introducing a method to bundle internal modules This PR is not addressing the massive CLI dependencies problem...