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

fiexd the issue (added the sample questions in html file). #12756

Closed
wants to merge 1 commit into from

Conversation

SamyakBhaware
Copy link

@SamyakBhaware SamyakBhaware commented Mar 16, 2025

changed AiChatComponent.fxml and AiChatComponent.java files

Added the following questions as written in the issue :
(https://github.com/JabRef/jabref/blob/4dea5fae711f9dd61c94f6a170b475b9e0633e22/src/main/java/org/jabref/gui/ai/components/aichat/AiChatComponent.fxml#L25-L24)

  1. What is the goal of the paper?
  2. Which methods were used in the research?
  3. What are the key findings?

LINK : #12702
Closes (link) OR Closes #12702

I have added the sample questions required to ai chat bot, as posted in the issue

Marked as: Done

changed AiChatComponent.fxml and AiChatComponent.java
Copy link
Contributor

Your code currently does not meet JabRef's code guidelines. We use Checkstyle to identify issues. Please carefully follow the setup guide for the codestyle. Afterwards, please run checkstyle locally and fix the issues.

In case of issues with the import order, double check that you activated Auto Import. You can trigger fixing imports by pressing Ctrl+Alt+O to trigger Optimize Imports.

@koppor koppor marked this pull request as draft March 17, 2025 11:26
@palukku
Copy link
Contributor

palukku commented Mar 17, 2025

You should use the template when opening a pull request and follow the steps/checks there.

Also @kaushikaW is already working on this, for more details you can see his current status here: #12747

@SamyakBhaware
Copy link
Author

You should use the template when opening a pull request and follow the steps/checks there.

Also @kaushikaW is already working on this, for more details you can see his current status here: #12747

I am not aware how to use template, sorry for that, can get a help in this?

@subhramit
Copy link
Member

subhramit commented Mar 18, 2025

Closing as issue is already assigned.
@SamyakBhaware Please follow the project's contributing guidelines before blindly filing PRs. See if the issue is assigned, if not, assign yourself to it. Your quality of code is also not up to the mark, so avoid using AI in cases where you don't know what you want to achieve. The title is misleading as well.

Regarding your doubt:

I am not aware how to use template, sorry for that, can get a help in this?

When filing the PR and writing the PR description, read what is written before removing the text - that is the template and should be followed/adapted. You can look at other PRs for examples.

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.

Add example questions to AI chat
3 participants