Reduce memory footprint#6298
Conversation
- Cache latex free field values for display in the main table - Only access preferences once for how to display names (instead of for every entry again) - Update search async only after 400ms without typing
by removing some caching of ids and keys in databases
|
You seem to have committed on top of an old branch. I had a déjà vu seeing some changes |
|
Yep, now it should be fine. Sorry |
koppor
left a comment
There was a problem hiding this comment.
I like intelligent models. No need for "artificial" separation in model and logic. However, we wanted to keep the model package very slim ("dumb"?).
I'd like to hear @stefan-kolb's take on this.
|
Could you please check the performance of import? The duplication checker is heavily used on import |
|
I didn't experience any performance problems with it. Adding a new entry is now slightly slower because we have to traverse a ArrayList instead of looking up a key in a hashmap. But I guess the difference should be negligible to average size databases of <= 10k entries. @koppor not sure what you mean with model vs logic. All my changes concern only the model package. |
…ionCaseInsensitive * upstream/master: (25 commits) ActionHelper to test for present file (#6151) Reduce memory footprint (#6298) Add missing abbreviated journal names (#6292) fix l10n again fix checkstyle fix l10n Try to minimize CodeCov "wrong metrics" Showing correct icon on main table linked files column (#6264) Fix labels for outdated dependency issue Change one more line Squashed 'src/main/resources/csl-styles/' changes from c31d9ca..c1793d2 Resolve unit test from failing Add one more change Fix errors RIS import takes the wrong date and duplicates abstract (#6272) Update EntryTypeView.java Change to the old school format Fix XmpExporterTest (#6289) Add checkstyle screenshot (and lint guidelines-...md) Squashed 'src/main/resources/csl-styles/' changes from db54e56..c31d9ca ...
Here a screenshot of the diff. I marked the package in green. I read
|
43566f2 Update molecular-oncology.csl (#6354) 38f2b5f Update san-francisco-estuary-and-watershed-science.csl (#6350) 724cb12 Update australasian-journal-of-philosophy.csl (#6344) df6af86 Fix webpage in-text citation for council-of-science-editors-author-date.csl (#6318) 6900b58 Add month to magazine for Bluebook e0a8148 Update cambridge-university-press-author-date-cambridge-a.csl (#6345) 0823448 Add space & comma before pp (#6343) ff38cd2 Update american-journal-of-respiratory-and-critical-care-medicine.csl (#6341) 8727dfb Update early-music-history.csl (#6323) 1154354 Update boletin-de-pediatria.csl (#6310) f25438e Update pravnik.csl (#6309) db7a4ae Update zoological-journal-of-the-linnean-society.csl (#6304) 6c043a7 Create polygraphia.csl (#6307) 255e00c Create intellect-newgen-books.csl (#6308) 323629f Update historical-materialism.csl (#6300) f62b70d Create european-review-of-international-studies.csl (#6301) dff2698 Update ucl-university-college-harvard.csl (#6298) 0ce09c9 Update sciences-po-ecole-doctorale-note-french.csl (#6290) bdd53ec Update sciences-po-ecole-doctorale-author-date.csl (#6291) efde4d4 Create journal-of-law-medicine-ethics.csl (#6296) 7539b2c Create theses-de-sorbonne-universite.csl (#6295) 905f25a Update biochemical-society-transactions.csl (#6292) a76a3f5 Update smithsonian-institution-scholarly-press author-date and note (#6294) e6b6c6c Create exploration-of-targeted-anti-tumor-therapy.csl (#6276) 024c9c8 Create nys-nydanske-studier.csl (#6331) d9ac8e1 Update vox-sanguinis.csl (#6327) 9a98e92 Remove DOI from Genetics & Molecular Biology git-subtree-dir: buildres/csl/csl-styles git-subtree-split: 43566f2
by removing some caching of ids and keys in databases.
Realized to late that the numbers are actually quite small. So this doesn't give much of a boost, but anyway it also doesn't hurt. Moreover, in my opinion, the code is more readable now.