Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Write a function in views.py to integrate the model with the web tool. #2194 #2366

Merged
merged 18 commits into from
Jun 29, 2024

Conversation

Sarthak5598
Copy link
Member

No description provided.

Copy link
Member Author

@Sarthak5598 Sarthak5598 left a comment

Choose a reason for hiding this comment

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

Not ready for merge now , Just want to keep it open as everyone is working on it , Would like to know everyones opinion

@Sarthak5598 Sarthak5598 changed the title added langchain adn openai to poetry Write a function in views.py to integrate the model with the web tool. #2194 Jun 20, 2024
@Sarthak5598
Copy link
Member Author

Hey, can anyone help me with this? For some reason, the AutoLabel function isn't getting the bugdescription. I've console logged everything, and it's correct from the report.html side. It's even getting the label from the function (JSON response), but it can't get bugdescription from report.html.

@Sarthak5598
Copy link
Member Author

Works now !!, no need for a review as of now, thanks

@Sarthak5598
Copy link
Member Author

The new implementation works as intended for setting the label but the issue is that the bug reports are no longer being posted on the main page as they used to be. It seems that the issue may be related to how the bug reports are handled after the label selection is done. please help

blt/urls.py Outdated Show resolved Hide resolved
website/templates/report.html Outdated Show resolved Hide resolved
website/views.py Outdated Show resolved Hide resolved
website/views.py Outdated Show resolved Hide resolved
website/views.py Outdated Show resolved Hide resolved
Sarthak5598 and others added 5 commits June 29, 2024 09:23
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>
@Sarthak5598 Sarthak5598 enabled auto-merge June 29, 2024 04:51
@Sarthak5598 Sarthak5598 requested a review from arkid15r June 29, 2024 04:51
Copy link
Contributor

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

LGTM

blt/urls.py Show resolved Hide resolved
@Sarthak5598 Sarthak5598 merged commit 97a46ba into OWASP-BLT:main Jun 29, 2024
8 checks passed
Copy link

sentry-io bot commented Jun 30, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the O... /issue-auto-label/ View Issue

Did you find this useful? React with a 👍 or 👎

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

Successfully merging this pull request may close these issues.

2 participants