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

"Save" button for preferences not functioning #11663

Closed
2 tasks done
wujastyk opened this issue Aug 21, 2024 · 7 comments · Fixed by #11669 or #11703
Closed
2 tasks done

"Save" button for preferences not functioning #11663

wujastyk opened this issue Aug 21, 2024 · 7 comments · Fixed by #11669 or #11703
Assignees
Labels
component: ai Related to AI Chat/Summarization component: internationalization i18n component: preferences [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs
Milestone

Comments

@wujastyk
Copy link

JabRef version

Latest development branch build (please note build date below)

Operating system

GNU / Linux

Details on version and operating system

JabRef 5.16--2024-08-20--6b8c5a3 Linux 6.8.0-40-generic amd64 Java 21.0.2 JavaFX 22.0.2+4

Checked with the latest development build (copy version output from About dialog)

  • I made a backup of my libraries before testing the latest development version.
  • I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

JabRef 5.16--2024-08-20--6b8c5a3
Linux 6.8.0-40-generic amd64
Java 21.0.2
JavaFX 22.0.2+4

  1. Open preferences
  2. Change some stuff
  3. Press "save". Nothing happens. Preferences are not saved.

Appendix

...

Log File
Paste an excerpt of your log file here
@LoayGhreeb
Copy link
Member

Fixed today: #11661

@InAnYan
Copy link
Collaborator

InAnYan commented Aug 23, 2024

@wujastyk Hi! Did you also have agreed to the AI privacy policy?

@wujastyk
Copy link
Author

@wujastyk Hi! Did you also have agreed to the AI privacy policy?

The change I made to preferences was in Preferences/Entry Editor , deselecting the two "Show AI ..." tickboxes.

@wujastyk
Copy link
Author

JabRef 5.16--2024-08-22--4ab11ca
Linux 6.8.0-40-generic amd64
Java 21.0.2
JavaFX 22.0.2+4

PS Unticking the "Show AI Prefs" box doesn't stick. See attempts below.

jabref-2024-08-23_10.55.31.mp4

@InAnYan
Copy link
Collaborator

InAnYan commented Aug 24, 2024

Oh, I see, so turning off "Show AI Summary" button doesn't work.

Thank you for pointing this out! I'll fix it right away

@ThiloteE
Copy link
Member

ThiloteE commented Sep 2, 2024

@wujastyk @InAnYan The issue is not yet fixed. We found another cause for it.
When you change the language settings from English to another language (e.g. German), some values in the AI settings are not transformed to the new language's grammar and then fail to have a default setting.

This is what is in my log:

C:\Prog\Development\JabRef_JabRef-JabRef\JabRef-JabRef\jabref>gradle run

> Configure project :
Project : => 'org.jabref' Java module

> Task :run
WARNING: Using incubator modules: jdk.incubator.vector
Messages are not initialized before accessing key: Display help on command line options
2024-09-02 20:44:08 [JavaFX Application Thread] org.jabref.gui.theme.ThemeManager.addStylesheetToWatchlist()
INFO: Watching css C:\Prog\Development\JabRef_JabRef-JabRef\JabRef-JabRef\jabref\build\resources\main\org\jabref\gui\Base.css for live updates
2024-09-02 20:44:08 [JavaFX Application Thread] org.jabref.gui.theme.ThemeManager.updateThemeSettings()
INFO: Not updating theme because it hasn't changed
2024-09-02 20:44:08 [JavaFX Application Thread] org.jabref.gui.theme.ThemeManager.updateThemeSettings()
INFO: Theme set to Theme{type=EMBEDDED, name='Dark.css'} with base css StyleSheet{file:/C:/Prog/Development/JabRef_JabRef-JabRef/JabRef-JabRef/jabref/build/resources/main/org/jabref/gui/Base.css}
2024-09-02 20:44:08 [JavaFX Application Thread] org.jabref.gui.theme.ThemeManager.addStylesheetToWatchlist()
INFO: Watching css C:\Prog\Development\JabRef_JabRef-JabRef\JabRef-JabRef\jabref\build\resources\main\org\jabref\gui\Dark.css for live updates
2024-09-02 20:44:09 [pool-2-thread-1] org.jabref.logic.ai.models.UpdateEmbeddingModelTask.call()
INFO: Downloading embedding model...
2024-09-02 20:44:10 [JavaFX Application Thread] org.jabref.gui.StateManager.setActiveDatabase()
INFO: No open database detected
2024-09-02 20:44:12 [pool-2-thread-2] org.jabref.gui.JabRefDialogService.notify()
INFO: ´┐¢ffne: 'C:\server-t-150\Jabref & LatexTests\OneMillionEntries.bib'
2024-09-02 20:44:13 [pool-2-thread-2] org.jabref.gui.JabRefDialogService.notify()
INFO: ´┐¢ffne: 'C:\server-t-150\Jabref & LatexTests\test-Files.bib'
2024-09-02 20:44:13 [pool-2-thread-3] org.jabref.gui.JabRefDialogService.notify()
INFO: ´┐¢ffne: 'C:\server-t-150\Jabref & LatexTests\Chocolate.bib'
2024-09-02 20:44:14 [pool-2-thread-1] ai.djl.pytorch.engine.PtEngine.newInstance()
INFO: PyTorch graph executor optimizer is enabled, this may impact your inference latency and throughput. See: https://docs.djl.ai/docs/development/inference_performance_optimization.html#graph-executor-optimization
2024-09-02 20:44:14 [pool-2-thread-1] ai.djl.pytorch.engine.PtEngine.newInstance()
INFO: Number of inter-op threads is 12
2024-09-02 20:44:14 [pool-2-thread-1] ai.djl.pytorch.engine.PtEngine.newInstance()
INFO: Number of intra-op threads is 6
2024-09-02 20:44:15 [pool-2-thread-1] ai.djl.util.Platform.detectPlatform()
INFO: Found matching platform from: jar:file:///C:/Users/Thilo/.gradle/caches/modules-2/files-2.1/ai.djl.huggingface/tokenizers/0.29.0/edd38326d16523f2b8c2b1e360b896e7ae48a1fa/tokenizers-0.29.0.jar!/native/lib/tokenizers.properties
2024-09-02 20:44:17 [JavaFX Application Thread] org.jabref.logic.ai.models.JabRefEmbeddingModel.lambda$startRebuildingTask$0()
INFO: Embedding model was successfully updated
2024-09-02 20:45:14 [JavaFX Application Thread] org.jabref.logic.l10n.Localization.lookup()
WARN: Warning: could not get translation for "RAG min score must be greater than 0 and less than 1" for locale de
2024-09-02 20:45:15 [JavaFX Application Thread] org.jabref.logic.push.CitationCommandString.from()
INFO: Wrong indexes -1 -1 for completeCiteCommand \parencite. Using default delimiter and suffix.

Below are the missing defaults:
image

Even when filling the settings, it does not accept what I enter, namely 0,3 and 0,7. In German language, the comma is used as building blocks to real numbers (with the exception of integers, which are a subset of real numbers). In English language, one uses a dot (.).

image

Temporarily disabling the AI functionality, then setting JabRef's language to English, then re-enabling the AI functionality and reset expert settings to default, is the workaround for now.

@ThiloteE ThiloteE added [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs component: internationalization i18n component: ai Related to AI Chat/Summarization component: preferences labels Sep 2, 2024
@ThiloteE ThiloteE added this to the 6.0-alpha milestone Sep 2, 2024
@InAnYan
Copy link
Collaborator

InAnYan commented Sep 3, 2024

@ThiloteE thank you very much for finding the bug!

OMG, you even probably found the root cause - commas and dots!

That's probably is the problem with DoubleTextField component (it's thirdparty), it might not support the numbers...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: ai Related to AI Chat/Summarization component: internationalization i18n component: preferences [outdated] type: bug Confirmed bugs or reports that are very likely to be bugs
Projects
Status: Done
4 participants