search

Mirascope / mirascope

LLM abstractions that aren't obstructions
http://mirascope.com/
MIT License
757 stars 50 forks source link

O1 style chain #614

Closed KTS-o7 closed 1 month ago

KTS-o7 commented 1 month ago

In relation to #573 Hello @willbakst I have written a rudimentary cookbook. Currently right answers are a hit or miss. What am I doing wrong ?

also when i followed Contribution.md i think uv.lock got updated. I have currently pushed it, should i be exlcuding it ?

sorry for this hurried notebook.

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (96df57a) to head (18156ce).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## support-cohere-version-5.11.0 #614 +/- ## =============================================================== Coverage 100.00% 100.00% =============================================================== Files 351 351 Lines 10474 10474 =============================================================== Hits 10474 10474 ``` | [Flag](https://app.codecov.io/gh/Mirascope/mirascope/pull/614/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mirascope) | Coverage Δ | | |---|---|---| | [tests](https://app.codecov.io/gh/Mirascope/mirascope/pull/614/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mirascope) | `100.00% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mirascope#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

KTS-o7 commented 1 month ago

@koxudaxi I have made the changes you requested in the new commit. Please let me know if any more changes are required Edit : I dont know why checks are failing, it ran on my pc image

Am i doing something wrong ?

Edit 2: Also Github is asking to Update branch, ie merge something from main, should we be doing that ? image

koxudaxi commented 1 month ago

OK, we should wait for merged support-cohere-version-5.11.0 branch to main.

KTS-o7 commented 1 month ago

Yeah that is fine If you guys find this contribution useful any chance of considering this repo for Hacktoberfest ? If that's possible it would be really cool as it would help me reach the milestone.

koxudaxi commented 1 month ago

Thank you very much. Your contribution has been very helpful for the community. I'm not very familiar with Hacktoberfest, so I'll ask @willbakst about it.