ESC meeting minutes: 2023-04-20

Miklos Vajna vmiklos at collabora.com
Thu Apr 20 14:54:16 UTC 2023


* Present:
     + Heiko, Thorsten, Caolan, Cloph, Ilmari, Michael S, Olivier, Stephan, Stephane, Xisco, Miklos, Eike, Hossein, Michael M, Tomaz

* Completed Action Items:

* Pending Action Items:

* Release Engineering update (Cloph)
     + 7.6 status: alpha 1 in 3 weeks, from master
       + feature freeze in June
     + 7.5 status: 7.5.3 rc2 next week
     + 7.4 status: 7.4.7 rc1 this week (tagged yesterday)
       + last planned release, be careful with backports

* Documentation (Olivier)
     + Documenting the API (?)
        + Rationale (see wiki)
        + Can be set as 10's of easy-hacks
        + example: https://gerrit.libreoffice.org/c/core/+/149590
        + PVT msg : "Then again, the perfect is the enemy of the good,
        and if we can Easy-Hackify & crowd-source this for Basic,
        why not start doing it."
        + Summary https://wiki.documentfoundation.org/Documentation/GSoDOC/2023#Project_2:_Illustrate_API_Documentation_with_examples
        + already commented on the gerrit change (Stephan)
          + do we want the basic code samples to the IDL files?
          + thought would be use the DevGuide to describe the bigger picture
        + do we have links from the HTML (from IDL) to the DevGuide? (Michael S)
          + yes, generated from odk/docs/idl/idl_chapter_refs.txt
        + any drawback to this approach? (Stephane)
          + is used to seeing lots of examples
        + sitting on the fence (Thorsten)
          + the API is hard to grok with its generic interfaces
          + is it useful to have in-code examples?
          + OTOH we should not stop because we don't have something perfect
          + this setup is already there, just need to write the examples
          + e.g. PHP: formal doc + users can comment on the website
          + good idea to start, but let's not forget the long-term goals
        + like the idea, but is this the correct place to host such documentation? (Hossein)
          + vs hosting it in the wiki, what is the benefit?
          + currently have to use xray to understand what's ongoing (Olivier)
            + and only then read the API documentation
            + simple use-cases are very complicated
            + the programming language is just a detail
            + also like the PHP approach
            + and integrate API docs into an IDE
            + but for the short-term, start producing results early
        + it would be possible to add examples to the wiki, then refer to it from offapi/, similar to the DevGuide, but is it better? (Michael S)
          + depends on the size of the example, large samples are better not inline (Stephan)
        + see a bit of danger: sample example in 5 different languages (Stephan)
     * Helpcontent2
        + Updates and fixes (ohallot)
     * Guides
        + Updates in progress
     + Bugzilla Documentation statistics
         265(265) bugs open
     + Updates:
         BZ changes   1 week    1 month    3 months   12 months
            created     12(0)      40(-4)    123(3)      307(5)
          commented      7(-39)   102(-35)   347(-7)    1121(-16)
           resolved      3(-1)     16(-3)     58(0)      176(3)
     + top 10 contributors:
           Seth Chaiklin made 53 changes in 1 month, and 492 changes in 1 year
           Olivier Hallot made 47 changes in 1 month, and 458 changes in 1 year
           Bogdan B made 39 changes in 1 month, and 218 changes in 1 year
           Stéphane Guillou made 36 changes in 1 month, and 190 changes in 1 year
           Heiko Tietze made 15 changes in 1 month, and 114 changes in 1 year
           Ilmari Lauhakangas made 11 changes in 1 month, and 97 changes in 1 year
           Rafael Lima made 7 changes in 1 month, and 119 changes in 1 year
           Eyal Rozenberg made 6 changes in 1 month, and 11 changes in 1 year
           Xisco Fauli made 4 changes in 1 month, and 30 changes in 1 year
           Rathke, Eike made 2 changes in 1 month, and 15 changes in 1 year

* UX Update (Heiko)
     + Bugzilla (topicUI) statistics
         281(281) (topicUI) bugs open, 96(96) (needsUXEval) needs to be evaluated by the UXteam
     + Updates:
         BZ changes   1 week    1 month   3 months   12 months
              added      10(2)     27(4)     51(3)       85(3)
          commented     122(67)   328(38)   748(44)    2318(52)
            removed       0(-4)     7(-2)    10(0)       31(-2)
           resolved       6(3)     24(-3)    84(-4)     279(-1)
     + top 10 contributors:
           Heiko Tietze made 191 changes in 1 month, and 1414 changes in 1 year
           Vernon, Stuart Foote made 86 changes in 1 month, and 326 changes in 1 year
           Stéphane Guillou made 52 changes in 1 month, and 213 changes in 1 year
           Ilmari Lauhakangas made 49 changes in 1 month, and 226 changes in 1 year
           Dieter made 39 changes in 1 month, and 227 changes in 1 year
           Eyal Rozenberg made 33 changes in 1 month, and 287 changes in 1 year
           Bogdan B made 15 changes in 1 month, and 44 changes in 1 year
           Rafael Lima made 14 changes in 1 month, and 277 changes in 1 year
           Baole Fang made 11 changes in 1 month, and 11 changes in 1 year
           Don Matschull made 11 changes in 1 month, and 11 changes in 1 year
     + [Bug 154781] Pasting into a cell should make it change to edit mode
     + [Bug 154890] Enhancement:Change labels in the Curves and Polygons toolbar
     + [Bug 154903] Impress: Master Slide deck in Sibebar should have Title,
            Footers and more options
     + [Bug 154794] Add -None- to choices in the Arrow Styles tab of Line dialog
     + [Bug 154831] Replace string is empty
     + [Bug 154766] management of ellipsis variants
     + [Bug 154693] Proposal to Modernize Appearance of Active Slide
  -> + [Bug 154788] The default width of Calc columns should be a bit narrower
     + [Bug 154714] Auto capital should not be applicable to a single letter
            inside quotations
     + [Bug 154787] Ambiguous File menu entry for creating a new document
            from a template

  -> + two more word ignore at a time
       + https://bugs.documentfoundation.org/show_bug.cgi?id=154499


* Crash Testing (Caolan)
     + 91(+25) import failure, 20(-3) export failures
       + new split fly frame
       - but older existing export crash fixed
     + 0 coverity issues
     + 4 ossfuzz issues, no crashes

* Crash Reporting (Xisco)
    + https://crashreport.libreoffice.org/stats/version/7.4.6.2
          + (+170) 683 513 392 371 241 0
    + https://crashreport.libreoffice.org/stats/version/7.5.1.2
          + (-180) 980 1160 1400 1241 854 546 0
    + https://crashreport.libreoffice.org/stats/version/7.5.2.2
          + (+423) 950 527 0

    + 5-6 crashes identified and fixed by Caolán and Xisco this week

* Mentoring (Hossein)
           committer...   1 week    1 month     3 months    12 months
                   open      96(35)    152(21)     177(23)      183(23)
                reviews     348(16)   1138(-24)   3560(2)     11682(72)
                 merged     300(94)   1110(-12)   3860(-85)   14632(37)
              abandoned      19(6)      79(11)     199(-1)      714(11)
            own commits     220(75)    815(16)    2702(-61)   10649(27)
         review commits      57(14)    214(-52)    816(8)      2891(19)
         contributor...   1 week     1 month     3 months     12 months
                   open      25(-1)      56(-5)     233(2)        240(1)
                reviews     830(200)   2694(16)    8520(-160)   31128(102)
                 merged       6(-9)      51(-12)    178(-9)       726(-11)
              abandoned       8(2)       25(0)       92(5)        501(2)
            own commits      20(0)       82(-36)    321(-2)       946(10)
         review commits       0(0)        0(0)        0(0)          0(0)
     + easyHack statistics:
        needsDevEval 9(9)   needsUXEval 1(1)   cleanup_comments 323(323)
        total 391(391)   assigned 32(32)   open 333(333)
     + top 10 contributors:
           Armin Le Grand (allotropia) made 10 patches in 1 month, and 57 patches in 1 year
           Stéphane Guillou made 7 patches in 1 month, and 13 patches in 1 year
             made 4 patches in 1 month, and 38 patches in 1 year
           Bogdan B made 4 patches in 1 month, and 91 patches in 1 year
           Bayram Çiçek made 3 patches in 1 month, and 4 patches in 1 year
           Baole Fang made 3 patches in 1 month, and 10 patches in 1 year
           Damjan, Jovanovic made 2 patches in 1 month, and 4 patches in 1 year
           Vojtěch Doležal made 2 patches in 1 month, and 8 patches in 1 year
           Czeber László made 2 patches in 1 month, and 3 patches in 1 year
           Arvind K made 2 patches in 1 month, and 3 patches in 1 year
     + top 10 reviewers:
           McNamara, Caolán made 120 review comments in 1 month, and 1148 in 1 year
           Adolfo Jayme Barrientos made 100 review comments in 1 month, and 1048 in 1 year
           Michael Stahl made 86 review comments in 1 month, and 1052 in 1 year
           Nabet, Julien made 84 review comments in 1 month, and 862 in 1 year
           Xisco Fauli made 78 review comments in 1 month, and 792 in 1 year
           Ilmari Lauhakangas made 62 review comments in 1 month, and 310 in 1 year
           Kłos, Szymon made 56 review comments in 1 month, and 190 in 1 year
           Thorsten Behrens made 54 review comments in 1 month, and 536 in 1 year
           Timar, Andras made 52 review comments in 1 month, and 318 in 1 year
           Vajna, Miklos made 50 review comments in 1 month, and 698 in 1 year
     + Patches automatically abandoned:
         tdf#98226 sw_undo:UnitTest ( Poonam Shokeen )
             - https://gerrit.libreoffice.org/#/c/142497
         wip: don't offer line join of "none" in the UI anymore ( Caolán McNamara )
             - https://gerrit.libreoffice.org/#/c/145359
         wip: ditch B2DLineJoin::NONE ( Caolán McNamara )
             - https://gerrit.libreoffice.org/#/c/145360
         Translate example from German to English ( Andrea Gelmini )
             - https://gerrit.libreoffice.org/#/c/145709
         tdf#153319 fix footnote related crash ( Caolán McNamara )
             - https://gerrit.libreoffice.org/#/c/146534
         tdf#141908: replace usage of sal_Int32 with Color ( Devansh Jain )
             - https://gerrit.libreoffice.org/#/c/146685
         Remove break; ( Andrea Gelmini )
             - https://gerrit.libreoffice.org/#/c/146751
         Writer UITest: Outline numbering tooltips when scrolling are missing space ( Radhey Parekh )
             - https://gerrit.libreoffice.org/#/c/147091
‌
* Worked on Python port of the DevGuide
   * DevGuide, Chapter 1: First Steps
   * Done: 2, remaining: 1 (Python)
   * Small code snippets remaining
   https://wiki.documentfoundation.org/Documentation/DevGuide/First_Steps
   * the more languages, the better (Stephan)
     * better than having many languages in idl files

* Reviewed old EasyHacks to make sure they are still relevant
     https://wiki.documentfoundation.org/Development/EasyHacks/Review

* Some of the mentee problems:
   * Poblems with ccache -> wrong shell instead of bash
   * Problem with giving input in console -> losing .bashrc/bash_profile

* ESC tender project proposal process (Thorsten)
   + as previously announced (in email), changes coming down from the board
   + how the tenderable projects are selected by the ESC
   + early draft is circulated - feedback appreciated
   + would be good to do this transparently (Michael M)
   + need more time to provide feedback (Eike)
   + it goes at great length specifying how to exclude people (Michael M)
     + but everyone can rank
     + result may be that nearly only the TDF staff turn up to vote, because others may want to bid
     + concerned to make sure staff can vote according to their personal opinion, not management hierarchy up to & including the Board.
     + the idea that big tasks can be precisely estimated is interesting
       + depends on who does the work, what other tasks those people have, risks etc.
       + there is no uniform 'person day'
       + excluding more people because they helped estimate seems un-helpful
     + otherwise looks reasonable
   + also worried about precise estimates (Stephan)
     + making the pool even smaller
     + already a small set of people understand that area, then silly to exclude them
   + proposal would be: (Thorsten)
     + share the draft in public: see https://nextcloud.documentfoundation.org/s/YprpsFP45z7a7p3
     + what's a useful timeline for feedback ?
        + its a blocker for budgeting.
        + depends on if the "precise estimate" rule can be e.g. removed (Michael M)
     + timeline for the first round of feedback? (Thorsten)
       + next Thur? 27th Apr?
       + support the idea (Eike, Michael M)
     + what is the alternative? (Heiko)
       + the alternative is to not tender anything, nobody wants that
       + alternative could be that all TDF members can rank ideas (Michael M)
          + need to present things to others (Heiko)
             + can non-devs vote on core things ?
             + paying technical debt hard to market
       + worries about noise from board of trustees (Olivier H)
          + pet bug not fixed while other things are done.
       + not wrong that the BoD needs to decide one way or another (Thorsten)
          + but - having the people who have to implement the process - should
            read & ask Q's and be able to provide feedback - if something doesn't
            make sense - is important to provide that opportunity
     + overall pleased it preserves inclusion for ranking & concept of ESC
       consensus helpfully (Michael M)
     => feedback in a week.

* GSoC (Ilmari)
   + timeline: 27th of April ranking needs to be done
   + mentors being invited, joining the org
   + reviews appreciated https://gerrit.libreoffice.org/q/status:open+-ownerin:Committers+repo:core+branch:master

* Commit Access

* Developer Certification (Stephan/Kendy/Thorsten/László)
   + resting since: 5 weeks (limit: 20 weeks)

* Jenkins / CI update (Cloph)
gerrit_android_aarch64     jobs: 217 ok: 191 ko:  11 fail ratio: 5.07% mean_ok:  14 ( 23) median_ok:  10 ( 16)
gerrit_android_arm         jobs: 218 ok: 192 ko:  15 fail ratio: 6.88% mean_ok:  14 ( 23) median_ok:  10 ( 15)
gerrit_android_x86         jobs: 220 ok: 195 ko:  12 fail ratio: 5.45% mean_ok:  14 ( 24) median_ok:   9 ( 16)
gerrit_android_x86_64      jobs: 219 ok: 200 ko:   9 fail ratio: 4.11% mean_ok:  14 ( 23) median_ok:  10 ( 16)
gerrit_linux_clang_dbgutil jobs: 465 ok: 290 ko: 146 fail ratio: 31.40% mean_ok:  52 ( 63) median_ok:  48 ( 57)
gerrit_linux_gcc_release   jobs: 437 ok: 374 ko:  41 fail ratio: 9.38% mean_ok:  26 ( 33) median_ok:  22 ( 26)
gerrit_mac                 jobs: 449 ok: 338 ko:  75 fail ratio: 16.70% mean_ok:  43 ( 68) median_ok:  39 ( 53)
gerrit_windows             jobs: 474 ok: 328 ko:  97 fail ratio: 20.46% mean_ok:  61 (101) median_ok:  60 ( 74)
gerrit_master              jobs: 501 ok: 245 ko: 208 fail ratio: 41.52% mean_ok:  97 (106) median_ok:  73 ( 76)
    + https://dev-www.libreoffice.org/tmp/gerrit_230420.html
    + tests that failed more than twice in last seven days
       3 CppunitTest_sw_mailmerge        gerrit_windows
       3 CppunitTest_sw_uiwriter4        gerrit_linux_gcc_release
       4 Killed by the kill-wrapper        gerrit_linux_clang_dbgutil
       5 CppunitTest_framework_dispatch        gerrit_windows
       5 CppunitTest_sw_ooxmlexport7        gerrit_windows
       5 CppunitTest_sw_uiwriter4        gerrit_windows
       6 UITest_sw_findBar        gerrit_linux_clang_dbgutil
       8 Killed by the kill-wrapper        gerrit_mac
       8 UITest_calc_tests8        gerrit_linux_clang_dbgutil
      14 CppunitTest_sw_uiwriter4        gerrit_linux_clang_dbgutil
      32 UITest_impress_tests        gerrit_linux_clang_dbgutil
    + UITest_impress_test failure limited to two fastest builders (tb88-linux / tb89-linux)
      those also happen to be qemu/kvm VMs
    + had a high number of failures in the UITest_impress_test, see above
      + can add auto-retry rules...
    + are auto-retry rules removed after some time? (Xisco)
      + not many rules currently (Cloph)
      + so far didn't remove any old retry rules
      + till now hadn't have rules specific to tests
      + Windows one had one test case name to detect leftover processes, though

* Swift for macOS-specific code? (Julien)
   + see https://bugs.documentfoundation.org/show_bug.cgi?id=154849 for context
   + it doesn't make much difference either way (Cloph)
   + also think: if there is no technical reason to switch, then why switch from Objective-C (Stephan)
     + not looking forward to invest the necessary configure / gbuild work
     + unless there is a compelling technical reason
   + Apple deprecates technology fast (Michael M)
     + if the person wants to also do the gbuild work, why not?
   + idea was: convert, then fix things (Caolan)
     + so these are separated cleanly.
   => first do the fixing, then the conversion ideally (Miklos)

* What’s cooking (Miklos)
   + Multi-page floating tables in Writer (Miklos)
     + next is to look at DOCX-related crashtest finds
     + does it work for RTL ? (Hossein)
        + interested in bug reports (Miklos)
   + multi-color gradients (Armin)
     + if you want to play with that, opt in with MCGR_TEST=1
     + expect: one more set of fixes, then enable for OOXML
     + Regina is working on the ODF side
   + theming work (Tomaz)
     + merging patches for filter bits for Writer
     + then some more UI work

* LibreOffice conference Bucharest 2023 (Gabriel)
   + Italo visiting the venue (Olivier)

* QA update (Xisco)
     + UNCONFIRMED: 1132 (-9)
         + enhancements: 307  (+2)
         + needsUXEval: 11 (+3)
         + haveBackTrace: 6 (-1)
         + needsDevAdvice: 32 (+0)
         + documentation:  2 (-2)
         + android:  3 (+0)
         + iOS:  0 (+0)
         + Online:  12 (+0)

     + Most pressing bugs:
      New:

     None


     + New high severity bugs of the week:
         + Long export to ods from xls / xlsx since 7.4.0beta1
             + https://bugs.documentfoundation.org/show_bug.cgi?id=152104
             + regression from jumbo sheets work

* QA stats (Stéphane)
     + Small dashboard changes: https://stragu.shinyapps.io/lo_qa_dashboard/
        + Make categorised bars proportional (show fraction rather than absolute value)
        + Merged renamed major releases
        + Download current table view
     + New meta bug for Thunderbird interop: https://bugs.documentfoundation.org/show_bug.cgi?id=154755
     + New meta bug for Accessibility Sidebar / Automatic Accessibility Checking: https://bugs.documentfoundation.org/show_bug.cgi?id=154929
     + https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
       +132  +21 (-143) overall)
       many thanks to the top bug squashers:
        Stéphane Guillou (stragu) 25
        QA Administrators       16
        Buovjaga                10
        Caolán McNamara         8
        Mike Kaganski           8
        Justin L                7
        Julien Nabet            5
        Eike Rathke             4
        Andreas Heinisch        3
        BogdanB                 3

     + top 10 bugs reporters:
        Rafael Lima             12
        Gabor Kelemen (allotropia) 8
        Eyal Rozenberg          7
        Telesto                 4
        Alexander Kurakin       3
        Don Matschull           3
        Joysn71                 3
        Michael FA              2
        Regina Henschel         2
        Robert Großkopf         2

     + top 10 bugs fixers:
        McNamara, Caolán        7
        Andreas Heinisch        3
        Grandin, Noel           3
        Rathke, Eike            3
        Justin Luth             2
        Kaganski, Mike          2
        Németh, László          2
        Patrick Luby            2
        Robert Großkopf         2
        Tünde Tóth              2

     + top 10 bugs confirmers:
        Ilmari Lauhakangas      32
        Stéphane Guillou        28
        Dieter                  8
        Heiko Tietze            8
        Nabet, Julien           7
        Raal                    4
        m.a.riosv               4
        Kelemen, Gabor          3
        Rathke, Eike            3
        Robert Großkopf         3

* Bisected bugs open: keyword 'bisected'
    + more accurate - down to a single commit.
    + http://bit.ly/2dyIfDy
    +

      done by:
         Ilmari Lauhakangas     6
         Stéphane Guillou       4
         Raal                   3
         Kelemen, Gabor         2
         Weghorn, Michael       1

* Bibisected bugs open: keyword 'bibisected'
    + http://bit.ly/2cSCXlS
    +

      done by:
         Ilmari Lauhakangas     6
         Stéphane Guillou       4
         Raal                   3
         Kelemen, Gabor         2
         Weghorn, Michael       1
         sockseight             1

* all bugs tagged with 'regression'
    + 1247(-4) bugs open of 12493(+13) total 15(-2) high prio.

      done by:
         Stéphane Guillou       4
         Ilmari Lauhakangas     3
         Kelemen, Gabor         1
         Weghorn, Michael       1
         Raal                   1
         Telesto                1

* ~Component   count net * high severity regressions
    + http://bit.ly/1HWHb3E
      Writer        - 8(+0)
      Calc          - 3(+0)
      LibreOffice   - 2(+0)
      Draw          - 1(+0)
      Impress       - 1(+0)
      BASIC         - 0(-1)
      Formula Editor - 0(-1)

    by OS:
      All           - 10(-1)
      Linux         - 2(+0)
      Mac OS X      - 0(+0)
      Windows       - 3(-1)

* ~Component   count net * all regressions
    + http://bit.ly/1BUdI8i
      Writer: other            - 443(+3)
      Calc                     - 226(+1)
      Impress                  - 133(-1)
      LibreOffice              - 54(-1)
      Writer: docx filter      - 52(+0)
      Crashes                  - 50(-3)
      Draw                     - 47(+0)
      Base                     - 44(-1)
      UI                       - 37(-1)
      Writer: perf             - 34(-1)
      Borders                  - 32(+0)
      Writer: other filter     - 30(+0)
      Chart                    - 26(+0)
      Printing and PDF export  - 23(+1)
      RTL                      - 23(+0)
      Writer: doc filter       - 21(+0)
      BASIC                    - 17(-1)
      filters and storage      - 15(+0)
      graphics stack           - 12(+0)
      Formula Editor           - 3(-1)
      Extensions               - 2(+0)
      sdk                      - 2(+0)
      Linguistic               - 1(+0)
      framework                - 1(+0)


More information about the LibreOffice mailing list