Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[FEAT]: Winstonai new plagiarized system #266

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Philrobots
Copy link
Contributor

No description provided.

@Philrobots Philrobots changed the title Feat(): Plagiat new system [FEAT]: Winstonai new plagiarized system Sep 11, 2024
@Daggx Daggx self-requested a review October 2, 2024 15:34
@@ -1 +1,2 @@
WINSTON_AI_API_URL = "https://api.gowinston.ai/functions/v1"
WINSTON_AI_PLAGIA_DETECTION_URL = "https://api.gowinston.ai/plagiarism/v2"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The current URL appears to be incorrect, resulting in a 404 error. Should be updated to :
https://api.gowinston.ai/v2/plagiarism

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True, my fault

raise ProviderException(response.json())

plagiarism_score: int = result.get("score")
sources = result.get("sources")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sources variable should be retrieved from original_response and not result.
sources = original_response.get("sources")


plagia_detection_candidate = PlagiaDetectionCandidate(
url=source_url,
plagia_score=source_score,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The plagia_score should be a value between 0 and 1, we need to divide source_score/100

@Philrobots
Copy link
Contributor Author

Whats up with this pull request ? @Daggx

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants