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

AI: chatting with papers #11427

Closed
wants to merge 136 commits into from
Closed

AI: chatting with papers #11427

wants to merge 136 commits into from

Conversation

InAnYan
Copy link
Collaborator

@InAnYan InAnYan commented Jun 25, 2024

Mandatory checks

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

…' into ai-1

# Conflicts:
#	src/main/java/org/jabref/gui/preferences/ai/AiTab.java
#	src/main/java/org/jabref/gui/preferences/ai/AiTabViewModel.java
#	src/main/java/org/jabref/preferences/AiPreferences.java
#	src/main/java/org/jabref/preferences/JabRefPreferences.java
# Conflicts:
#	src/main/java/org/jabref/gui/preferences/ai/AiTab.fxml
#	src/main/java/org/jabref/gui/preferences/ai/AiTab.java
#	src/main/java/org/jabref/gui/preferences/ai/AiTabViewModel.java
#	src/main/java/org/jabref/preferences/AiPreferences.java
#	src/main/java/org/jabref/preferences/JabRefPreferences.java
# Conflicts:
#	src/main/java/module-info.java
#	src/main/java/org/jabref/gui/ai/components/JabRefMarkdownView.java
#	src/main/java/org/jabref/gui/ai/components/aichat/AiChatComponent.fxml
#	src/main/java/org/jabref/gui/ai/components/aichat/AiChatComponent.java
#	src/main/java/org/jabref/gui/ai/components/chatmessage/ChatMessageComponent.fxml
#	src/main/java/org/jabref/gui/ai/components/chatmessage/ChatMessageComponent.java
# Conflicts:
#	src/main/java/org/jabref/gui/DefaultInjector.java
#	src/main/java/org/jabref/gui/JabRefGUI.java
#	src/main/java/org/jabref/gui/LibraryTab.java
#	src/main/java/org/jabref/gui/entryeditor/EntryEditor.java
#	src/main/java/org/jabref/gui/externalfiles/ImportHandler.java
#	src/main/java/org/jabref/gui/frame/JabRefFrame.java
#	src/main/java/org/jabref/gui/frame/JabRefFrameViewModel.java
#	src/main/java/org/jabref/gui/frame/MainToolBar.java
#	src/main/java/org/jabref/gui/importer/actions/OpenDatabaseAction.java
#	src/main/java/org/jabref/gui/openoffice/OpenOfficePanel.java
#	src/main/java/org/jabref/gui/preferences/PreferencesDialogViewModel.java
#	src/main/java/org/jabref/gui/sidepane/SidePaneContentFactory.java
# Conflicts:
#	build.gradle
#	src/main/java/module-info.java
@Siedlerchr
Copy link
Member

So the checkstyle has issues with the switch statement in the MvStoreembeddings
at com.puppycrawl.tools.checkstyle.grammar.java.JavaLanguageParser.switchLabeledRule(JavaLanguageParser.java:7155)

return switch (filter) {
case null -> embeddingsMap.keySet().stream();

case IsIn isInFilter when Objects.equals(isInFilter.key(), "linkedFile") ->
Copy link
Member

Choose a reason for hiding this comment

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

When I saw the error messages correctly, this is the part where checkstyle fails.

Can you verify this locally with running checkstyle?

https://docs.oracle.com/en/java/javase/21/language/pattern-matching-switch-expressions-and-statements.html#GUID-A5C220F6-F70A-4FE2-ADB8-3B8883A67E8A

I found related issues

checkstyle/checkstyle#15044
checkstyle/checkstyle#15082

@Siedlerchr
Copy link
Member

The out of memory seems to be related to linux arm only in jlink.

Sorry to tell ya, but can you please push the branch to upstream (jabref/jabref) repo and recreate the PR ? Then we can get the binaries uploaded and test them for any module errors

@InAnYan InAnYan closed this Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants