-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
AI: chatting with papers #11427
Conversation
…' 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
So the checkstyle has issues with the switch statement in the MvStoreembeddings |
return switch (filter) { | ||
case null -> embeddingsMap.keySet().stream(); | ||
|
||
case IsIn isInFilter when Objects.equals(isInFilter.key(), "linkedFile") -> |
There was a problem hiding this comment.
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?
I found related issues
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 |
Mandatory checks
CHANGELOG.md
described in a way that is understandable for the average user (if applicable)