gagb
gagb
@rajan-chari can you please update the description and roadmap task list for CAP?
This is nice! I'll check @dkirsche might also be interested
@WaelKarkoub I am running late on this PR-- I will have time after 14th to review this but I will do this. So sorry :)
@WaelKarkoub LFG!!! This align so much with what I am used to with other libs. Feedback: 1. Add NB demonstrating your code in use 2. Add test cases Wdyt?
> @gagb Thanks for the review! I updated the docstrs, let me know what you think of them Will review today.
> > This is a great PR. Thanks so much! > > @sonichi @ekzhu @qingyun-wu @jackgerrits We had conversation about creating our own "message" format. Do you think we should...
@WaelKarkoub then it seems like other maintainers agree with proceeding with this PR. wdyt?
> @gagb could you confirm that the old way of handling context will be deprecated? Shall we remove the test and eventually remove the code and examples too? @jackgerrits there...
cc: @BeibinLi @victordibia since you are working on support for multi-modal
@dkirsche would be fun to add this feature! I'll submit a PR soon for review unless you'd like to take a stab!