This message was deleted.
# how-do-i
s
This message was deleted.
p
Hey Cameron! It should be in the header of the description field of the PR page - I know @Sven Bendel was also having issues finding this button, so unless you're part of the orgs that have specifically opted out of our AI offerings you should be able to see it!
Looping @Greg into this as well
c
This part?
p
Ah yeah it's definitely missing - do you mind PMing me what GitHub org you're a part of?
šŸ‘ 1
s
Can confirm that it’s still missing for me as well. Let me know if you need additional information to debug.
c
@Sven Bendel @Cameron Perry can you DM me what companies you’re with? We’ve had to turn off this feature for some users for compliance reasons, but you might still be getting the NUXes (the latter part being a bug probably).
āœ… 1
d
I'm looking into this and believe I know what the problem is. I'm working on a fix now that I hope to have out later today.
party parrot 1
s
Awesome, thanks David! šŸš€
d
The fix for this should now be out. Please give it a try and let me know if you continue to see issues.
c
Thanks šŸŽ‰
Ah, got a ā€œFailed to summarize try againā€ error when I clicked.
d
We currently have a limit on the size of PRs that AI summarize will work on. Looking through our logs, it looks like the PR you are trying is hitting that limit. This is something we want to improve in the future.
c
Hmm I saw the ā€œDiff too largeā€ errors on a couple PRs but this one that was a couple hundred lines was giving me a different generic error message. Is it still that limit? About how many lines can it handle?
s
Button is there for me now as well, sorry for the late reply. Though I also get the ā€œDiff too largeā€ pretty frequently. Still, thanks for making it work again! šŸ™
p
Yeah we still have a limit on the diffs we can summarize šŸ˜• will get back to you on the limit
Limitation of the way we're using the API unfortunately, but may try to find some workarounds
c
No worries. Honestly I don’t see this being a thing I use other than for the ā€œcool factorā€ of seeing what it says. Might be more interesting to see weekly digests with an AI summary of all the PRs merged over the week. I could see that making me aware of changes I didn’t look closely at in PR. Since you know what PR I did/didn’t review, would be even cooler if I got AI summary of only the PRs I hadn’t already reviewed šŸ˜„
c
@Cameron Perry we are working on this right now :)
šŸŽ‰ 1
šŸš€ 1
p
šŸ‘€ šŸ‘€
@Greg's current brain child!
c
Awesome!! Those blocked out parts are like a one sentence summary of the change?
p
Yep a brief summary based on some PR context! Blocked out because I don't want to accidentally spoil some fun features we have coming ahead :')
c
For sure. Well done! Ship it šŸ˜† šŸ‘
j
Hey, sorry to bump this thread - just wondering if the ā€œDiff too largeā€ error might be because package lock files are being included? Just encountered this on a PR šŸ˜