GP Saggese

Results 193 comments of GP Saggese

@Shaunak01 do you have enough permissions for test driving this? Let me know if I need to do something. BTW we can write a blog with our experience on using...

I think it's set up already This is what I see on a per-project basis ![Image](https://github.com/user-attachments/assets/c8109071-ff2e-409c-a4ea-f399fe1f1d98) This is what I see at org level ![Image](https://github.com/user-attachments/assets/ca1e19ec-2bdf-4281-b2b4-f480fcdaad85) When I go to a...

We can try to feed https://github.com/causify-ai/helpers/blob/master/docs/coding/all.coding_style.how_to_guide.md as it is but I'm not sure if we need to make that file more LLM-friendly. Can you experiment with starting a new file...

I've also put together several targets in `llm_transform.py` that get this job done. I'll do a quick demo today so we can start using it

Copilot is ok, but not great. I've filed - https://github.com/causify-ai/helpers/issues/685 - https://github.com/causify-ai/helpers/issues/716 - https://github.com/causify-ai/helpers/issues/717 to use a mixed of automation and human stuff

Sure, let's give it to Copilot and see what happens. @Shaunak01 can you post where it needs to go and any link to relevant documentation for Copilot? E.g., is there...

I've tried but it's very high level and only 600 chars. ![Image](https://github.com/user-attachments/assets/885eb2e6-2e0b-4aff-b732-19554361bd7d) I think the ai_reviewer.py approach is better. Closing this

Very good. Let's push on the comments so we can start using that script.

For now let's just skip executing unit tests with `pytest.importorskip()` There are various solutions: 1) The try-install approach is not great since every time somebody imports that module (even pytest...