Feature/change eval to post

[emrgnt-cmplxty]

	🚀 This PR description was created by Ellipsis for commit 11abe71.

Summary:

This PR modifies the HTTP method for the /eval endpoint from GET to POST in the r2r/main/app.py file, affecting the rag_completion, _stream_rag_completion, and eval functions.

Key points:

• Changed HTTP method for /eval endpoint from GET to POST in r2r/main/app.py.
• Updated requests to /eval in rag_completion and _stream_rag_completion functions.
• Updated decorator for eval function to @app.post.

---

Generated with ❤️ by ellipsis.dev

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
r2r-docs	🔄 Building (Inspect)	Visit Preview	Mar 28, 2024 2:26am

[ellipsis-dev]

👍 Looks good to me!

• Reviewed the entire pull request up to 11abe71
• Looked at 32 lines of code in 1 files
• Took 45 seconds to review

More info

• Skipped 0 files when reviewing.
• Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.

1. r2r/main/app.py:341:

• Assessed confidence : 100%
• Grade: 0%
• Comment:
  The /eval endpoint now expects a request body due to the change from GET to POST. Please ensure that all calls to this endpoint have been updated to include the necessary request body.
• Reasoning:
  The PR changes the HTTP method from GET to POST for the /eval endpoint and the requests made to it. This is generally a good practice as POST is more suitable for requests that cause changes. However, the /eval endpoint now expects a request body (payload: EvalPayloadModel), which is not typically included in GET requests. The PR author should ensure that all calls to this endpoint have been updated to include the necessary request body.

Workflow ID: wflow_gftEOcabrM3qsGdY

---

Not what you expected? You can customize the content of the reviews using rules. Learn more here.