ESC meeting minutes: 2023-01-05

[Miklos Vajna <vmiklos@xxxxxxxxxxxxx>]

* Present:
    + Stephan, Michael S, Ilmari, Heiko, Michael W, Olivier, Thorsten, Miklos, Kendy, Eike, Stephane

* Completed Action Items:

* Pending Action Items:
  + backport the touchpad changes (Xisco)

* Release Engineering update (Cloph)
    + Missing Cloph.
    + 7.4 status: 7.4.4 to be released next week?
    + 7.5 status: rc2 to be tagged next week?
      * Late features
        + reverting a patch https://gerrit.libreoffice.org/c/core/+/145007 (Heiko)
        + ran into an unexpected problems, much more complicated than originally expected, best to revert
        + have no concerns (Thorsten, Miklos)
        + (Stéphane) this string change needs a post-string freeze follow-up or revert for 7.5, as it makes the problem worse: https://gerrit.libreoffice.org/c/core/+/136851 (see my comment in https://bugs.documentfoundation.org/show_bug.cgi?id=101731#c22 )

          + goal was to improve things, but after the change it seems the current state is worse

          + propose to go back to the original state for 7.5


* Documentation (Olivier)
    + New Help
       + Changes in makefiles, speed-up build (cloph)
    + Helpcontents
       + Improvements in Content Control (ohallot)
         + also seeing some menu item renames (Heiko)
         + help is for master only, no conflict here (Olivier)
       + Same on cell addressing (ohallot)
       + Updates and fixes (stragu, S. Horacek, B. Buzea, erack, buovjaga)
    + Guides
       + Stalled
    + Bugzilla Documentation statistics
        249(249) bugs open
    + Updates:
        BZ changes   1 week   1 month   3 months   12 months
           created     10(5)    29(-3)     92(-1)     316(-5)
         commented     10(7)    65(-8)    286(2)     1165(-47)
          resolved      4(4)    13(-2)     48(3)      195(-2)
    + top 10 contributors:
          Bogdan B made 27 changes in 1 month, and 174 changes in 1 year
          Olivier Hallot made 26 changes in 1 month, and 469 changes in 1 year
          Stéphane Guillou made 17 changes in 1 month, and 66 changes in 1 year
          Ilmari Lauhakangas made 14 changes in 1 month, and 99 changes in 1 year
          Kaganski, Mike made 12 changes in 1 month, and 118 changes in 1 year
          Adolfo Jayme Barrientos made 7 changes in 1 month, and 38 changes in 1 year
          Vernon, Stuart Foote made 7 changes in 1 month, and 16 changes in 1 year
          Dieter made 4 changes in 1 month, and 86 changes in 1 year
          Faure, Jean-Baptiste made 4 changes in 1 month, and 4 changes in 1 year
          Alain Romedenne made 2 changes in 1 month, and 33 changes in 1 year

* UX Update (Heiko)
    + Bugzilla (topicUI) statistics
        282(282) (topicUI) bugs open, 82(82) (needsUXEval) needs to be evaluated by the UXteam
    + Updates:
        BZ changes   1 week    1 month    3 months   12 months
             added      7(-4)     28(6)      40(6)       73(7)
         commented     18(-58)   224(-23)   532(-35)   2247(9)
           removed      0(0)       4(-1)     10(0)       37(0)
          resolved      2(-2)     19(-1)     67(-5)     300(-2)
    + top 10 contributors:
          Eyal Rozenberg made 92 changes in 1 month, and 282 changes in 1 year
          Heiko Tietze made 81 changes in 1 month, and 1508 changes in 1 year
          Stéphane Guillou made 80 changes in 1 month, and 100 changes in 1 year
          Vernon, Stuart Foote made 66 changes in 1 month, and 170 changes in 1 year
          Kaganski, Mike made 44 changes in 1 month, and 151 changes in 1 year
          Ilmari Lauhakangas made 39 changes in 1 month, and 182 changes in 1 year
          Dieter made 38 changes in 1 month, and 221 changes in 1 year
          maicol figueroa made 31 changes in 1 month, and 31 changes in 1 year
          m.a.riosv made 22 changes in 1 month, and 42 changes in 1 year
          lostbits made 8 changes in 1 month, and 8 changes in 1 year
    + 32 new tickets with needsUXEval

    + Cannot undo typing in Hyperlink dialog
      + https://bugs.documentfoundation.org/show_bug.cgi?id=147783
      + undo/redo within RichTextBox
      => needsDevAdvice
      + probably a bug / not implemented, not intentional (Miklos)

    + Add option to export slides from Impress as SVG without JavaScript
      + https://bugs.documentfoundation.org/show_bug.cgi?id=147776
      + none in ooO.Common.Filter.Graphic.Export.SVG
      => needsDevAdvice
      + think there are 2 export filters (Kendy)
        + old memories, but one is probably meant to be a JS presentation, the other would be SVG export (vector image); save-as vs exprot

    + AutoCorrect DOI (ISO 26324) recognition
      + https://bugs.documentfoundation.org/show_bug.cgi?id=145925
      + autocorrect doi: into http://doi...
      => interesting easyhack
      + arguing this is similar to auto-URL-correct

* Crash Testing (Caolan)
    + Missing Caolan.
    + 105(+4) import failure, 3(+0) export failures
    + ??? coverity issues
    + Google / ossfuzz: ?? fuzzers active now

* Crash Reporting (Xisco)
    + Missing Xisco.
    + 7.4.0.3    21538(+510)
    + 7.4.1.2    25483(+704)
    + 7.4.2.3    26842(+1097)
    + 7.4.3.2    10358(+2457)

* Easyhack update / mentoring (Hossein)
  + Missing Hossein.
          committer...   1 week     1 month      3 months    12 months
                  open      78(24)     153(26)      156(19)      164(20)
               reviews     246(38)    1274(-44)    2928(-54)   11682(-124)
                merged     132(-55)   1169(-105)   3341(-65)   14496(-88)
             abandoned       6(3)       42(-1)      155(-2)      820(-13)
           own commits      93(-17)    705(-78)    2377(-43)   10911(-125)
        review commits      48(13)     279(2)       841(15)     2920(-60)
        contributor...   1 week    1 month      3 months    12 months
                  open      44(23)     67(9)        69(-2)       84(-3)
               reviews     538(16)   2936(-136)   7520(-74)   31792(-278)
                merged      11(4)      74(-7)      248(6)       948(-60)
             abandoned       9(8)     142(-22)     257(-1)      568(-1)
           own commits      21(6)     116(0)       367(13)      949(-53)
        review commits       0(0)       0(0)         0(0)         0(0)
    + easyHack statistics:
       needsDevEval 9(9)   needsUXEval 1(1)   cleanup_comments 320(320)
       total 382(382)   assigned 17(17)   open 334(334)
    + top 10 contributors:
          Bogdan B made 16 patches in 1 month, and 85 patches in 1 year
          Patrick Luby made 13 patches in 1 month, and 13 patches in 1 year
          Povilas Kanapickas made 9 patches in 1 month, and 18 patches in 1 year
          Armin Le Grand (allotropia) made 7 patches in 1 month, and 37 patches in 1 year
          Paris Oplopoios made 6 patches in 1 month, and 88 patches in 1 year
            made 5 patches in 1 month, and 25 patches in 1 year
          Galdam Jitsu made 3 patches in 1 month, and 5 patches in 1 year
          Fišeras, Aurimas made 2 patches in 1 month, and 3 patches in 1 year
          Srebotnjak, Martin made 2 patches in 1 month, and 24 patches in 1 year
          Stéphane Guillou made 2 patches in 1 month, and 6 patches in 1 year
    + top 10 reviewers:
          McNamara, Caolán made 144 review comments in 1 month, and 922 in 1 year
          Vajna, Miklos made 114 review comments in 1 month, and 628 in 1 year
          Xisco Fauli made 94 review comments in 1 month, and 846 in 1 year
          Adolfo Jayme Barrientos made 88 review comments in 1 month, and 1036 in 1 year
          Grandin, Noel made 86 review comments in 1 month, and 422 in 1 year
          Michael Stahl made 80 review comments in 1 month, and 1126 in 1 year
          Christian Lohmaier made 64 review comments in 1 month, and 370 in 1 year
          Vajngerl, Tomaž made 60 review comments in 1 month, and 438 in 1 year
          Nabet, Julien made 60 review comments in 1 month, and 792 in 1 year
          Németh, László made 54 review comments in 1 month, and 292 in 1 year
    + Patches automatically abandoned:
        tdf#146429 - Fallback to other character encodings detected by ICU ( Daniel Thomas )
            - https://gerrit.libreoffice.org/#/c/127539
        tdf#114441 sal_uLong to sal_uInt32 ( Hemant Kumar Singh )
            - https://gerrit.libreoffice.org/#/c/141607
        tdf#38890 Remove hard-coded URLs in source Files ( Adoche Onaji )
            - https://gerrit.libreoffice.org/#/c/142005
        Notify entry selection has changed on KEY_LEFT ( Jim Raykowski )
            - https://gerrit.libreoffice.org/#/c/142057
        vcl: extract function Animation::ClearAnimationRenderers() ( Chris Sherlock )
            - https://gerrit.libreoffice.org/#/c/76423
        vcl: extract Animation function CreateDefaultRenderer() ( Chris Sherlock )
            - https://gerrit.libreoffice.org/#/c/76425
    + big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
          Brett T. Warden (intel.com)
          Douglas Guptill (accesswave.ca)

* GSOC (Ilmari)
  + We need more ideas and mentors
  + https://wiki.documentfoundation.org/Development/GSoC/Ideas
  + https://wiki.documentfoundation.org/Development/GSoC/Ideas_without_a_mentor
  + enough ideas are needed for qualification, please add more! (Ilmari)
  + student sent a mail to the dev list, interested in a DLP filter
  + Heiko tagging bugs which might be a good project
    + have some 55 candidates so far
    + can add them to the "ideas without a mentor" page (Ilmari)

* Commit Access

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

* Jenkins / CI update (Cloph)
  + frequent Jenkins restarts, but Cloph is aware (Ilmari)

* FOSDEM 2023 (Xisco)
  + Italo is putting together a schedule
    + If you didn't get a mail from Italo, something went wrong (Cloph)

* LibreOffice hackfest in Cambridge (Miklos)
  + as part of COOL days: 28 & 29th March 2023
  + Collabora can fund accommodation for people
  + https://www.collaboraoffice.com/cooldays/ has more info

* ESC budget & projects 2023 (Thorsten, Ilmari)
  + review & update https://wiki.documentfoundation.org/Development/Budget2023
  + after that, rank & vote 2023 ESC project proposals
  + need descriptions + estimates
  + Need somebody to run this process
  + Deadline for proposing budget items? (Miklos)
    + in 2-3 weeks, want to have a rough idea by FOSDEM
  + Think perhaps enough ideas are already collected (Ilmari)
    + what was useful last year may not be useful this year, good to re-evaluate (Thorsten)
    + think there is an indifinite amount of useful things to do
  + happy to own this (Ilmari)
    + can help (Stephane)

* What’s cooking (Miklos)
  + Writer number portion formatting improvements (Miklos)
    + now surviving ODT roundtrip, building on top of earlier work from Michael S for DOCX
  + unocast changes (Stephan)
    + at the end fixed differently by implementing dynamic_cast<> in the bridges code
    + so no need to change code outside bridges
    + which means it's now safe to dynamic_cast on UNO objects which might be proxy objects

* QA update (Xisco)
    + UNCONFIRMED: 1616 (+35)
        + enhancements: 300  (+6)
        + needsUXEval: 18 (+7)
        + haveBackTrace: 7 (+0)
        + needsDevAdvice: 33 (+0)
        + documentation:  14 (-1)
        + android:  3 (+0)
        + iOS:  0 (+0)
        + Online:  11 (+0)

    + Most pressing bugs:
     New:
     Old:
     Fixed:

    + New high severity bugs of the week:
        + CRASH: importing ooo84576-1.odt, crashtest; corrupt document structure
            + https://bugs.documentfoundation.org/show_bug.cgi?id=152710
            + Michael S?
            + Dave has some extensive notes in the bug, FWIW

* QA stats (Stephane)
    + Many bug reports by Bogdan about console warnings this week. Majority of triaged ones have been added to the "Console Noise" meta: https://bugs.documentfoundation.org/show_bug.cgi?id=140879. See Bogdan's recent reports: https://bugs.documentfoundation.org/buglist.cgi?email1=buzea.bogdan%40libreoffice.org&emailreporter1=1&emailtype1=substring&list_id=1545101&query_format=advanced&resolution=---. Opinion?
    + focus is on a dev build: those warnings, mostly Writer
      + would not bother with backporting such fixes (Miklos)
      + an other view (Thorsten)
        + there is value in fixing warnings
        + but if it generates work for the QA team, that's not optimal
          + let's not misuse triage resources, that's limited
        + not concerned about these bugs (Stephane)
        + have a neutral view: if somebody wants to fix these, no problem (Thorsten)

    + https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
      +170  -100 (-119) overall)
      many thanks to the top bug squashers:
       QA Administrators       13
       Stéphane Guillou (stragu) 11
       Buovjaga                9
       Julien Nabet            9
       himajin100000           9
       خالد حسني               8
       V Stuart Foote          7
       Dieter                  5
       Alex Thurgood           4
       Caolán McNamara         4

    + top 10 bugs reporters:
       BogdanB                 52
       Gabor Kelemen (allotropia) 6
       Eyal Rozenberg          4
       lifeline                3
       Alex Thurgood           2
       New Life                2
       New Life Foundations    2
       Philippe                2
       Regina Henschel         2
       Stéphane Guillou (stragu) 2

    + top 10 bugs fixers:
       McNamara, Caolán        4
       Adolfo Jayme Barrientos 3
       Aron Budea              3
       Olivier Hallot          3
       خالد حسني               3
       Grandin, Noel           2
       Lars Jødal              2
                               1
       Eyal Rozenberg          1
       Henschel, Regina        1

    + top 10 bugs confirmers:
       Stéphane Guillou        40
       Ilmari Lauhakangas      11
       Dieter                  9
       m.a.riosv               4
       *UNKNOWN*               3
       McNamara, Caolán        3
       Raal                    3
       Xisco Fauli             3
       Telesto                 2
       Vernon, Stuart Foote    2

* Highest-Priority bugs (aka "MABs"):
   + http://bit.ly/2lJAAJI
     6.0 : 1/53 -     1 %  (+0)
     5.4 : 0/41 -     0 %  (+0)
     5.3 : 0/54 -     0 %  (+0)
     5.2 : 0/43 -     0 %  (+0)
     5.1 : 0/35 -     0 %  (+0)
     5.0 : 0/64 -     0 %  (+0)
     4.4 : 0/76 -     0 %  (+0)
     4.3 : 0/73 -     0 %  (+0)
     4.2 : 2/134 -    1 %  (+0)
     4.1 : 2/85 -     2 %  (+0)
     4.0 : 1/82 -     1 %  (+0)
     old : 1/246 -    0 %  (+0)

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

     done by:
        Stéphane Guillou       3
        Xisco Fauli            3
        Ilmari Lauhakangas     2
        Raal                   1

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

     done by:
        Stéphane Guillou       3
        Xisco Fauli            3
        Ilmari Lauhakangas     2
        خالد حسني              1
        Raal                   1

* all bugs tagged with 'regression'
   + 1186(+1) bugs open of 12223(+12) total 8(+0) high prio.

     done by:
        Stéphane Guillou       4
        Xisco Fauli            3
        Ilmari Lauhakangas     2
        Dieter                 1
        Eyal Rozenberg         1
        Alex Thurgood          1
        خالد حسني              1

* ~Component   count net * high severity regressions
   + http://bit.ly/1HWHb3E
     Writer        - 5(+0)
     Calc          - 1(+0)
     Draw          - 1(+0)
     LibreOffice   - 1(+0)

   by OS:
     All           - 7(+0)
     Linux         - 0(+0)
     Mac OS X      - 0(+0)
     Windows       - 1(+0)

* ~Component   count net * all regressions
   + http://bit.ly/1BUdI8i
     Writer: other            - 416(-1)
     Calc                     - 208(+1)
     Impress                  - 119(+1)
     Writer: docx filter      - 57(+0)
     Crashes                  - 51(+0)
     LibreOffice              - 45(-1)
     Draw                     - 42(+1)
     Base                     - 40(+0)
     UI                       - 35(+0)
     Writer: perf             - 35(+0)
     Borders                  - 32(+0)
     Writer: other filter     - 29(+0)
     Chart                    - 24(+0)
     RTL                      - 23(+0)
     Writer: doc filter       - 19(+0)
     BASIC                    - 18(+0)
     Printing and PDF export  - 17(+0)
     filters and storage      - 15(+0)
     graphics stack           - 15(+0)
     Formula Editor           - 3(+0)
     sdk                      - 2(+0)
     framework                - 1(+0)