ESC meeting minutes: 2025-04-17
Miklos Vajna
vmiklos at collabora.com
Thu Apr 17 14:38:35 UTC 2025
* Present:
+ Cloph, Michael W, Olivier, Hossein, Caolan, Ilmari, Jonathan, Stephan, Thorsten, Miklos
* Completed Action Items:
* Pending Action Items:
* Release Engineering update (Cloph)
+ 24.8.7 rc1 this week: was tagged yesterday
+ 25.2.3 rc2 next week
+ upcoming 25.8: first alpha in early May (from master)
+ https://wiki.documentfoundation.org/ReleasePlan/25.8 is created now (may change)
* Documentation (Olivier)
+ Helpcontents:
+ Several help pages on BASIC number format (ohallot)
+ Updates and fixes (ohallot)
+ Scriptforge maintenance (A. Romedenne)
+ Guides
+ work in progress (D. Maddern, Ed Olson, ohallot)
+ Bugzilla Documentation statistics
220(220) bugs open
+ Updates:
BZ changes 1 week 1 month 3 months 12 months
created 7(3) 18(-3) 51(4) 300(3)
commented 12(7) 46(1) 112(6) 861(9)
resolved 2(1) 6(-3) 26(1) 173(1)
+ top 10 contributors:
Olivier Hallot made 17 changes in 1 month, and 367 changes in 1 year
Kaganski, Mike made 13 changes in 1 month, and 83 changes in 1 year
Vernon, Stuart Foote made 9 changes in 1 month, and 118 changes in 1 year
Ilmari Lauhakangas made 7 changes in 1 month, and 163 changes in 1 year
Eyal Rozenberg made 4 changes in 1 month, and 16 changes in 1 year
Raal made 4 changes in 1 month, and 16 changes in 1 year
Heiko Tietze made 3 changes in 1 month, and 74 changes in 1 year
Jean-Francois Nifenecker made 3 changes in 1 month, and 3 changes in 1 year
nobu made 3 changes in 1 month, and 17 changes in 1 year
Pierre F made 2 changes in 1 month, and 197 changes in 1 year
* UX Update (Heiko)
+ Missing Heiko
+ Bugzilla (topicUI) statistics
222(222) (topicUI) bugs open, 12(12) (needsUXEval) needs to be evaluated by the UXteam
+ Updates:
BZ changes 1 week 1 month 3 months 12 months
added 3(-1) 8(1) 8(1) 14(1)
commented 35(-9) 124(1) 255(-6) 1227(-27)
removed 0(0) 0(0) 0(0) 8(0)
resolved 9(8) 19(3) 45(4) 212(2)
+ top 10 contributors:
Heiko Tietze made 43 changes in 1 month, and 618 changes in 1 year
Vernon, Stuart Foote made 43 changes in 1 month, and 372 changes in 1 year
Eyal Rozenberg made 32 changes in 1 month, and 210 changes in 1 year
Ilmari Lauhakangas made 32 changes in 1 month, and 185 changes in 1 year
Bogdan B made 12 changes in 1 month, and 55 changes in 1 year
Németh, László made 12 changes in 1 month, and 14 changes in 1 year
Kaganski, Mike made 9 changes in 1 month, and 35 changes in 1 year
Olivier Hallot made 9 changes in 1 month, and 15 changes in 1 year
Mohamed Hani made 8 changes in 1 month, and 8 changes in 1 year
Devashish gupta made 6 changes in 1 month, and 6 changes in 1 year
* Crash testing (Caolan)
+ 143(-41) import failure, 36(-37) export failures
+ 2 coverity issues
+ 6 issues, 3 crashes, 2 timeouts, 1 OOM
+ oss-fuzz build needs meson/ninja installed on that side:
https://github.com/google/oss-fuzz/pull/13228
* Crash Reporting from crashreport.libreoffice.org (Xisco)
+ 25.2.0.2 206(+1)
+ 25.2.0.3 31695(+1718)
+ 25.2.1.2 23089(+2623)
+ 25.2.2.2 6975(+3739)
- https://crashreport.libreoffice.org/stats/signature/writerfilter::dmapper::DomainMapper::sprmWithProps(writerfilter::Sprm%20&,tools::SvRef%3Cwriterfilter::dmapper::PropertyMap%3E%20const%20&)
- Should be fixed with https://git.libreoffice.org/core/+/eedda19d84dde8f284b4e296bda5208503bf2a7c%5E%21
* Mentoring (Hossein)
+ Reviewed some difficultyInteresting EasyHack submissions
+ Added a new draft blog post:
Splash screen with VCL weld – difficultyInteresting EasyHack
https://dev.blog.documentfoundation.org/?p=1417&preview=1&_ppp=577f62555a
committer... 1 week 1 month 3 months 12 months
open 109(50) 176(36) 182(39) 182(39)
reviews 386(122) 1354(-30) 3594(-56) 13356(38)
merged 286(63) 982(30) 3217(31) 14144(78)
abandoned 19(8) 58(1) 200(10) 739(10)
own commits 217(51) 671(81) 2166(59) 9949(75)
review commits 36(6) 192(-23) 599(-20) 2900(-10)
contributor... 1 week 1 month 3 months 12 months
open 553(22) 1110(-1) 1111(-2) 1111(-2)
reviews 904(258) 2724(108) 7604(102) 31078(212)
merged 10(-6) 95(-13) 295(-21) 1328(-20)
abandoned 38(22) 102(25) 212(28) 541(26)
own commits 3(-7) 80(-21) 223(-28) 1214(-14)
review commits 0(0) 0(0) 0(0) 0(0)
+ easyHack statistics:
needsDevEval 9(9) needsUXEval 1(1) cleanup_comments 351(351)
total 418(418) assigned 55(55) open 327(327)
+ top 10 contributors:
Devashish gupta made 9 patches in 1 month, and 9 patches in 1 year
Simon Chenery made 8 patches in 1 month, and 42 patches in 1 year
Manish Mahapatra made 4 patches in 1 month, and 4 patches in 1 year
Shardul Vikram Singh made 3 patches in 1 month, and 7 patches in 1 year
Karthik Godha made 3 patches in 1 month, and 8 patches in 1 year
Marco Pinto made 2 patches in 1 month, and 6 patches in 1 year
Celia Palacios Gómez Tagle made 2 patches in 1 month, and 2 patches in 1 year
Theppitak Karoonboonyanan made 2 patches in 1 month, and 11 patches in 1 year
Pierre Vacher made 2 patches in 1 month, and 7 patches in 1 year
Mohamed Ali made 2 patches in 1 month, and 12 patches in 1 year
+ top 10 reviewers:
Ilmari Lauhakangas made 170 review comments in 1 month, and 814 in 1 year
Caolán McNamara made 112 review comments in 1 month, and 1248 in 1 year
Vajna, Miklos made 110 review comments in 1 month, and 1152 in 1 year
Hossein made 102 review comments in 1 month, and 352 in 1 year
allotropia jenkins made 92 review comments in 1 month, and 528 in 1 year
Thorsten Behrens made 86 review comments in 1 month, and 680 in 1 year
Christian Lohmaier made 70 review comments in 1 month, and 756 in 1 year
Kaganski, Mike made 66 review comments in 1 month, and 558 in 1 year
Xisco Fauli made 60 review comments in 1 month, and 796 in 1 year
Nabet, Julien made 54 review comments in 1 month, and 840 in 1 year
+ Patches automatically abandoned:
tdf#152943: Add NaN payload check at startup ( Xuan Chen )
- https://gerrit.libreoffice.org/#/c/165391
tdf#101546 add button to open edit templates paths directly from manager ( Amos Aidoo )
- https://gerrit.libreoffice.org/#/c/165753
tdf#161837 Use double in Fraction class internally ( Hossein )
- https://gerrit.libreoffice.org/#/c/171965
tdf#147021 Use std::size() or std::ssize() instead of SAL_N_ELEMENTS() macro ( Harris Popal )
- https://gerrit.libreoffice.org/#/c/177648
vcl: move createAlphaBlendFrame to BlendFrameCache source files ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/178098
vcl: make edit.cxx readable ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/178514
vcl: extract Edit::ImplSetTextColors() ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/178519
vcl: nLen -> nTextLen ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/178520
vcl: extract lcl_GetEndCharIndex() ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/178809
vcl: make ImplUpdateSeparatorString() a local function ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/178993
vcl: rename ScrollBar::ImplCalc() to ScrollBar::ImplUpdateControlRegions() ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/179282
vcl: move ImplDraw() to just below Draw() ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/179290
vcl: textdat2.hxx -> textedit.hxx ( Chris Sherlock )
- https://gerrit.libreoffice.org/#/c/179714
test BlendBitmapWithAlpha ( Noel Grandin )
- https://gerrit.libreoffice.org/#/c/180432
cid#1557827 Data race condition ( Caolán McNamara )
- https://gerrit.libreoffice.org/#/c/180492
tdf#145538 tdf#153294 Replace or simplify for loops in basctl ( Ivan Soria )
- https://gerrit.libreoffice.org/#/c/180615
sw: test shape insertion via drag and drop ( Xisco Fauli )
- https://gerrit.libreoffice.org/#/c/181219
tdf#38812 WiP: UI button good, PackageKit call for help install to be fixed ( Johann Lorber )
- https://gerrit.libreoffice.org/#/c/181323
+ big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
Parth Raiyani (Collabora)
* GSoC (Ilmari)
+ https://developers.google.com/open-source/gsoc/timeline
+ LLM AI cheaters caught
+ having the easy hack requirement is a good idea (Ilmari)
+ good to also mention this in the wiki
+ sounds like a good idea, the point of the assignment it to show off your skills (Miklos)
+ used to have a "no calculator" rule during exams (Oliver)
+ qt company did a model training for QML code, that is meaningful (Hossein)
+ Trained based on examples
https://www.qt.io/blog/qt-ai-assistant-v0.9-released-deploy-llms-locally-and-enjoy-the-upgraded-user-experience
+ think we can use AI for code completion (Hossein)
+ when discussing a GSoC candidate: expect to admit it's in use, to clarify whos work is the result
+ talking is easy: good-looking proposal can produced
+ good looking patch: harder
+ take this into account when comparing proposals
+ think the comparison to calculators isn't perfect (Ilmari)
+ LLMs won't solve problems perfectly, like a calculator
+ calculator does a perfect job (Jonathan)
+ unlike an LLM
+ sitting on the fence, between Olivier / Ilmari (Thorsten)
+ looking at what VS provides vs asking AI
+ very blurry situation
+ "was a prompt used while producing a patch"
+ good to be explicit about this
+ got obviously trash patches (Hossein)
+ outsourcing the task completely is not OK
+ the entire OSS community is discussing (Thorsten)
+ produced by AI as-is, submitted without review, not OK
+ but there is a large gray area where it makes sense
+ need to be honest what was used
+ but good to benefit from innovation
+ probably going to be part of the dev toolkit in the future
+ it's not useful to completely ban it
+ rather embrace the useful bits
+ still need to review anything manually, of course
+ against using LLMs for easy hacks (Ilmari)
+ the point is learning
+ different topic would be: use LLMs to discover some unknown area
+ difficulty interesting would be perhaps this area (Hossein)
+ not difficulty easy or difficulty medium
+ perhaps spell this out: if you submit a patch, you are responsible for the patch you submitted, you should be able to explain it (Miklos)
+ if we assume the students are fully dishonest, that would be tricky (Ilmari)
+ this would not scale: screensharing interviews, etc.
+ don't think LLMs can solve difficulty interesting easy hacks currently (Hossein)
+ there will be always people who try to cheat (Thorsten)
+ hard to solve this problem
+ tell the students: if we catch you that you cheat, then we'll react
+ perhaps tweak the application criteria, may be good enough
+ for one case, wasted hours on this (Ilmari)
+ AI: add guidelines wrt LLM usage to the GSoC page (Ilmari)
+ Next step: 29 April - GSoC contributor proposal rankings due from Org Admins
* Commit Access
* Developer Certification (Stephan/Miklos/László/Gabriel)
+ resting since: 20 weeks (limit: 20 weeks)
+ suggesting David Gilbert (Stephan)
+ was doing work in multiple areas, e.g. PDF import (poppler)
+ support the candidate (Caolan)
+ AI: Stephan to proceed with this
* Jenkins / CI update (Cloph)
gerrit_android_aarch64 jobs: 202 ok: 196 ko: 6 fail ratio: 2.97% mean_ok: 7 ( 12) median_ok: 5 ( 8)
gerrit_android_arm jobs: 202 ok: 195 ko: 5 fail ratio: 2.48% mean_ok: 7 ( 12) median_ok: 5 ( 8)
gerrit_android_x86 jobs: 202 ok: 194 ko: 8 fail ratio: 3.96% mean_ok: 6 ( 11) median_ok: 5 ( 8)
gerrit_android_x86_64 jobs: 202 ok: 195 ko: 4 fail ratio: 1.98% mean_ok: 6 ( 12) median_ok: 4 ( 8)
gerrit_linux_clang_dbgutil jobs: 421 ok: 306 ko: 109 fail ratio: 25.89% mean_ok: 36 ( 40) median_ok: 34 ( 37)
gerrit_linux_gcc_release jobs: 514 ok: 392 ko: 96 fail ratio: 18.68% mean_ok: 20 ( 23) median_ok: 19 ( 19)
gerrit_mac jobs: 450 ok: 369 ko: 67 fail ratio: 14.89% mean_ok: 50 ( 50) median_ok: 45 ( 45)
gerrit_windows jobs: 418 ok: 363 ko: 47 fail ratio: 11.24% mean_ok: 39 ( 55) median_ok: 35 ( 37)
gerrit_master_ml jobs: 547 ok: 290 ko: 218 fail ratio: 39.85% mean_ok: 84 ( 85) median_ok: 73 ( 74)
gerrit_master jobs: 34 ok: 20 ko: 13 fail ratio: 38.24% mean_ok: 57 ( 57) median_ok: 48 ( 48)
gerrit_master_seq jobs: 506 ok: 270 ko: 198 fail ratio: 39.13% mean_ok: 86 ( 86) median_ok: 74 ( 74)
+ tests that failed more than twice in last seven days
3 CppunitTest_sw_rtfexport5 gerrit_linux_gcc_release
3 CppunitTest_sw_txtencexport gerrit_linux_gcc_release
3 JunitTest_toolkit_unoapi_4 gerrit_linux_clang_dbgutil
4 CppunitTest_sw_txtexport gerrit_linux_gcc_release
4 JunitTest_chart2_unoapi gerrit_linux_clang_dbgutil
5 aborted by jenkins job timeout gerrit_windows
5 UITest_xmlsecurity_gpg gerrit_linux_clang_dbgutil
6 UITest_chart gerrit_linux_clang_dbgutil
6 UITest_impress_tests gerrit_linux_clang_dbgutil
8 Killed by the kill-wrapper gerrit_linux_clang_dbgutil
+ usual week (Cloph)
+ except: chart2 silent merge conflict
+ tinderboxes noticed
+ thanks to Stephan for fixing it
* What’s cooking (Miklos)
+ A11y update (Michael W)
+ a11y code refactoring/deduplication
+ planning to look into unreliable announcement of Writer paragraphs by JAWS
+ RTL/CTL/CJK update (Jonathan)
+ tdf#166152: CJK characters crowd together due to character compression
+ https://bugs.documentfoundation.org/show_bug.cgi?id=166152
+ tdf#163618: editeng inserting kashida in positions without enough room
+ https://bugs.documentfoundation.org/show_bug.cgi?id=163618
+ Making UINames and Programmatic Names typesafe in Writer (Noel)
* QA update (Xisco)
+ UNCONFIRMED: 1639 (+22)
+ enhancements: 436 (+4)
+ needsUXEval: 7 (+3)
+ haveBackTrace: 14 (+0)
+ needsDevAdvice: 47 (+0)
+ documentation: 6 (+1)
+ android: 0 (+0)
+ iOS: 0 (+0)
+ Online: 7 (+0)
+ Most pressing bugs:
New:
None
Fixed:
+ CRASH: Opening print dialog after print preview
+ https://bugs.documentfoundation.org/show_bug.cgi?id=166107
+ Thanks to Xisco and Tibor
+ New high severity bugs of the week:
* QA stats
+ https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
+107 -18 (-78) overall)
many thanks to the top bug squashers:
m_a_riosv 9
V Stuart Foote 6
Jonathan Clark 5
Khaled Hosny 5
Balázs Varga (allotropia) 4
Mike Kaganski 4
Sarper Akdemir (allotropia) 4
Xisco Faulí 4
Heiko Tietze 3
nobu 3
+ top 10 bugs reporters:
Justin L 8
Gabor Kelemen (allotropia) 4
Timur 3
Xisco Faulí 3
nobu 3
Aron Budea 2
Noel Grandin 2
Oliver Specht (CIB) 2
golemus 2
Eyal Rozenberg 1
+ top 10 bugs fixers:
Balazs Varga 4
Jonathan Clark 4
Sarper Akdemir 4
Khaled Hosny 3
Rathke, Eike 2
Vajna, Miklos 2
Xisco Fauli 2
*UNKNOWN* 1
Andreas Heinisch 1
Aron Budea 1
+ top 10 bugs confirmers:
m.a.riosv 11
Raal 6
Xisco Fauli 6
*UNKNOWN* 5
Ilmari Lauhakangas 5
Aron Budea 4
Henschel, Regina 4
Kaganski, Mike 3
Németh, László 3
opp 3
* Bisected bugs open: keyword 'bisected'
+ more accurate - down to a single commit.
+ http://bit.ly/2dyIfDy
+
done by:
Xisco Fauli 4
jquintanaalvarado 2
libre 2
Timur Gadzo 2
Bogdan B 1
Ilmari Lauhakangas 1
jessica.c 1
Jonathan Clark 1
mikhail.machine 1
* Bibisected bugs open: keyword 'bibisected'
+ http://bit.ly/2cSCXlS
+
done by:
Xisco Fauli 4
Ilmari Lauhakangas 2
jquintanaalvarado 2
libre 2
Timur Gadzo 2
Bogdan B 1
jessica.c 1
Jonathan Clark 1
mikhail.machine 1
Raal 1
* all bugs tagged with 'regression'
+ 1246(+5) bugs open of 14025(+12) total 23(+0) high prio.
done by:
Xisco Fauli 3
Ilmari Lauhakangas 2
Jonathan Clark 1
libre 1
m.a.riosv 1
mikhail.machine 1
Telesto 1
* ~Component count net * high severity regressions
+ http://bit.ly/1HWHb3E
Writer - 9(+0)
Calc - 6(+0)
LibreOffice - 4(+0)
Impress - 2(+0)
Installation - 1(+0)
UI - 1(+0)
by OS:
All - 17(+0)
Linux - 4(+0)
Mac OS X - 0(+0)
Windows - 2(+0)
* ~Component count net * all regressions
+ http://bit.ly/1BUdI8i
Writer: other - 413(+0)
Calc - 243(+3)
Impress - 126(+0)
LibreOffice - 54(-1)
Draw - 47(+1)
Writer: docx filter - 45(+0)
Crashes - 43(+0)
Base - 37(+0)
Borders - 35(+0)
UI - 34(+0)
Writer: perf - 33(+0)
Writer: other filter - 30(+0)
Chart - 26(-1)
Printing and PDF export - 26(+1)
filters and storage - 26(+0)
RTL - 23(+0)
BASIC - 19(+0)
Writer: doc filter - 18(+0)
graphics stack - 13(+0)
Formula Editor - 11(+0)
framework - 3(+0)
sdk - 3(+0)
Extensions - 2(+0)
Installation - 2(+0)
Linguistic - 2(+0)
More information about the Libreoffice-qa
mailing list