-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Retrospective on the May-July 2024 roadmap (#59)
[RENDERED](https://github.com/linebender/linebender.github.io/blob/roadmap-retro/content/blog/2024-08-23-roadmap-may-2024-retro.md) Current status: This is complete besides the harfruzz section, and is awaiting review --------- Co-authored-by: Raph Levien <[email protected]> Co-authored-by: Olivier FAURE <[email protected]>
- Loading branch information
3 people
authored
Aug 24, 2024
1 parent
d9e678c
commit cd8436b
Showing
1 changed file
with
246 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,246 @@ | ||
| +++ | ||
| title = "Retrospective on the May-July roadmap" | ||
| authors = ["Daniel McNab"] | ||
| +++ | ||
|
|
||
| At the start of May, we created a [roadmap](@/blog/2024-06-17-roadmap-may-2024.md) for the following three months. | ||
| We recently went through a process of analysing how well our progress was guided by the roadmap; this report summarises those results. | ||
|
|
||
| ## Vello [(original)](@/blog/2024-06-17-roadmap-may-2024.md#vello) | ||
|
|
||
| ### Robust dynamic memory [(original)](@/blog/2024-06-17-roadmap-may-2024.md#robust-dynamic-memory) | ||
|
|
||
| - **Priority:** Highest. | ||
| - **Status**: Experimental PR ([vello#606](https://github.com/linebender/vello/pull/606)), but hasn’t landed. | ||
| Work required on the CPU side API, but the GPU operations are well-understood. | ||
| This PR could not be released in its current form. | ||
|
|
||
| ### Reliable snapshot tests [(original)](@/blog/2024-06-17-roadmap-may-2024.md#reliable-snapshot-tests) | ||
|
|
||
| - **Priority:** High. | ||
| - **Status**: This was implemented in [vello#610](https://github.com/linebender/vello/pull/610). | ||
| This has already been extremely useful in our review process, as it allows changes to be validated in the files changed view. | ||
| We also implemented an experiment to use [Git LFS](https://git-lfs.com/), which has been working really well to avoid increasing repository size. | ||
|
|
||
| ### Sparse strips [(original)](@/blog/2024-06-17-roadmap-may-2024.md#sparse-strips) | ||
|
|
||
| - **Priority:** High. | ||
| - **Status**: We are continuing this work, but there is no code currently available. | ||
| See [#gpu > Sparse strip path rendering](https://xi.zulipchat.com/#narrow/stream/197075-gpu/topic/Sparse.20strip.20path.20rendering) for details. | ||
|
|
||
| ### Add tests [(original)](@/blog/2024-06-17-roadmap-may-2024.md#add-tests) | ||
|
|
||
| - **Priority:** Medium | ||
| - **Status**: This item had already been completed at the time the roadmap was written. | ||
| This was also expanded upon in. | ||
| During our retrospective meeting, we did determine some potential future expansions to testing, namely: | ||
| - Running individual stages in isolation, to enable property testing (TODO: Make Vello issue). | ||
| - Testing for additional platforms, such as DirectX on Windows and Android (TODO: Make Vello issue). | ||
|
|
||
| ### Progress beyond roadmap {#vello-extras} | ||
|
|
||
| We have also landed several important features which were not part of our planned roadmap. | ||
| These include: | ||
|
|
||
| - Blending which uses more than four deep layers ([vello#657](https://github.com/linebender/vello/pull/657)). | ||
| - Preliminary support for embedding pre-existing `wgpu` textures ([vello#636](https://github.com/linebender/vello/pull/636)). | ||
| This was driven by user needs. | ||
| Further work in this direction is tracked in [vello#664](https://github.com/linebender/vello/issues/664). | ||
| - Support for Emoji (COLRv1 in [vello#615](https://github.com/linebender/vello/pull/615), bitmap in [vello#641](https://github.com/linebender/vello/pull/641)). | ||
|
|
||
| The [stroke expansion paper](https://linebender.org/gpu-stroke-expansion-paper/) documents a key part of Vello's pipeline. | ||
| This was presented at ACM High Performance Graphics (HPG) 2024, placing 3rd Place in the Best Paper Award. | ||
|
|
||
| ## Masonry [(original)](@/blog/2024-06-17-roadmap-may-2024.md#masonry) | ||
|
|
||
| ### Complete text input [(original)](@/blog/2024-06-17-roadmap-may-2024.md#complete-text-input) | ||
|
|
||
| - **Priority**: Highest | ||
| - **Status**: Not that much progress. | ||
| This work is being continued, and indeed there has been significant progress as of the time of writing. | ||
| This is tracked in [parley#52](https://github.com/linebender/parley/issues/52). | ||
|
|
||
| ### Better tracing for debugging [(original)](@/blog/2024-06-17-roadmap-may-2024.md#better-tracing-for-debugging) | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: [xilem#384](https://github.com/linebender/xilem/pull/384) implements most of this, the rest is deferred. | ||
| There are known issues with the size of the log files created by default in very large apps. | ||
|
|
||
| ### Fix glaring documentation problems [(original)](@/blog/2024-06-17-roadmap-may-2024.md#fix-glaring-documentation-problems) | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: The issues mentioned in this item were addressed in [xilem#329](https://github.com/linebender/xilem/pull/329) and [xilem#503](https://github.com/linebender/xilem/pull/503). | ||
|
|
||
| ### Finish repository port [(original)](@/blog/2024-06-17-roadmap-may-2024.md#fix-glaring-documentation-problems) | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: This was completed. | ||
| We moved all Masonry issues into the Xilem repository, and updated all references in code to use full links. | ||
| This move was coordinated in [#linebender > Moving Masonry into Xilem](https://xi.zulipchat.com/#narrow/stream/419691-linebender/topic/Moving.20Masonry.20into.20Xilem). | ||
|
|
||
| ### Write “widgets in arena” RFC [(original)](@/blog/2024-06-17-roadmap-may-2024.md#write-widgets-in-arena-rfc) | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: The RFC was [rfcs#6](https://github.com/linebender/rfcs/pull/6), which was mostly implemented in [xilem#396](https://github.com/linebender/xilem/pull/396). | ||
|
|
||
| ### Write “pass order” RFC [(original)](@/blog/2024-06-17-roadmap-may-2024.md#write-pass-order-rfc) | ||
|
|
||
| - **Prioriy**: High | ||
| - **Status**: [rfcs#7](https://github.com/linebender/rfcs/pull/7) is in draft, and several important parts of the RFC have been completed. | ||
|
|
||
| ### Rewrite documentation [(original)](@/blog/2024-06-17-roadmap-may-2024.md#rewrite-documentation) | ||
|
|
||
| - **Priority**: Medium | ||
| - **Status**: This work has been deferred until the "pass order" refactor is complete, as we expect Masonry's core to be quite stable at that point. | ||
|
|
||
| ### Improve focus handling [(original)](@/blog/2024-06-17-roadmap-may-2024.md#improve-focus-handling) | ||
|
|
||
| - **Priority**: Medium | ||
| - **Status**: Not done. | ||
|
|
||
| ### Improve pointer status handling [(original)](@/blog/2024-06-17-roadmap-may-2024.md#improve-pointer-status-handling) | ||
|
|
||
| - **Priority**: Medium | ||
| - **Status**: Basic translation of touch events was added in ([xilem#313](https://github.com/linebender/xilem/pull/313)). | ||
| This unblocked using Masonry on devices where pointer input is not expected (primarily on Android). | ||
| We have split the remainder of this task into two tasks, with a priority placed on completing the parts which unblock other work (such as scrolling on Android). | ||
|
|
||
| ### Progress beyond roadmap {#masonry-extras} | ||
|
|
||
| Masonry is in a state of flux, however is starting to see a few exploratory users. | ||
| To unblock these users, some additional important features were added outside of the roadmap, such as: | ||
|
|
||
| - Support for embedding Masonry inside a pre-existing event loop ([xilem#417](https://github.com/linebender/xilem/commit/cf3530097baf0fc5c458593e215a19973572b991)). | ||
| - Support for a `gap` property in the flexbox ([xilem#437](https://github.com/linebender/xilem/pull/437)). | ||
| - Fixes on iOS, including [xilem#418](https://github.com/linebender/xilem/pull/418) and [xilem#421](https://github.com/linebender/xilem/pull/421). | ||
| - [xilem#233](https://github.com/linebender/xilem/pull/233) allowed our tests to be run on CI in a consistent manner. | ||
|
|
||
| ## Parley [(original)](@/blog/2024-06-17-roadmap-may-2024.md#parley) | ||
|
|
||
| ### Create abstractions for text selection and editing [(original)](@/blog/2024-06-17-roadmap-may-2024.md#write-pass-order-rfc#create-abstractions-for-text-selection-and-editing) ? | ||
|
|
||
| - **Priority**: Highest | ||
| - **Status**: There have been useful discussions in [parley#52](https://github.com/linebender/parley/issues/52). | ||
| Work has started on this in [parley#106](https://github.com/linebender/parley/pull/106) in response to this retrospective. | ||
|
|
||
| ### Inline block layout [(original)](@/blog/2024-06-17-roadmap-may-2024.md#inline-block-layout) | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: A first cut of this feature was added by Nico Burns in ([parley#67](https://github.com/linebender/parley/pull/67)). | ||
| [parley#76](https://github.com/linebender/parley/pull/76) expands upon this, and has been approved in principle, but is awaiting thorough review. | ||
|
|
||
| ### Write adequate documentation [(original)](@/blog/2024-06-17-roadmap-may-2024.md#write-adequate-documentation) | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: Some work was done towards this in [parley#26](https://github.com/linebender/parley/pull/26), which unfortunately could not be merged. | ||
| [parley#97](https://github.com/linebender/parley/pull/97) added some documentation to Fontique. | ||
| Further documentation work from Nico is currently blocked on the tree-like styles PR [parley#76](https://github.com/linebender/parley/pull/76) being merged (to avoid conflicts). | ||
|
|
||
| ### Software renderer [(original)](@/blog/2024-06-17-roadmap-may-2024.md#software-renderer) | ||
|
|
||
| - **Priority**: Low | ||
| - **Status**: This was not completed in the form envisioned. | ||
| However, Nico contributed two examples, showing how to use Parley with Swash ([parley#54](https://github.com/linebender/parley/pull/54)) and tiny-skia ([parley#55](https://github.com/linebender/parley/pull/55)). | ||
| We have decided not to carry this forward into the next roadmap. | ||
|
|
||
| ### Progress beyond roadmap {#parley-extras} | ||
|
|
||
| [parley#56](https://github.com/linebender/parley/pull/56) added support for Emoji, which is used with the [Vello PRs](#vello-extras). | ||
|
|
||
| Recently the [harfruzz](https://github.com/harfbuzz/harfruzz) project, a fork of rustybuzz to be integrated fully with fontations, has emerged as an extremely promising direction for text shaping for the Rust ecosystem. We plan to use it as the basis for text layout. | ||
|
|
||
| ## Xilem [(original)](@/blog/2024-06-17-roadmap-may-2024.md#xilem) {#xilem} | ||
|
|
||
| ### Android bringup [(original)](@/blog/2024-06-17-roadmap-may-2024.md#android-bringup) | ||
|
|
||
| - **Priority**: Highest | ||
| - **Status**: Android is supported in several of our examples, with the primary work required for support completed in [xilem#309](https://github.com/linebender/xilem/pull/309). | ||
| No non-boilerplate code changes are required to get a Xilem application running on Android. | ||
| Some compromises were made, in particular cargo gives an un-silencable warning for each example which supports Android and cross-platform backends. | ||
| We did reach out to the Cargo team about this use case, but did not get a positive response. | ||
|
|
||
| ### Update repository layout [(original)](@/blog/2024-06-17-roadmap-may-2024.md#update-repository-layout) | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: This was completed in [xilem#302](https://github.com/linebender/xilem/pull/302). | ||
| Kaur applied the same change to Vello in [vello#590](https://github.com/linebender/vello/pull/590). | ||
|
|
||
| ### Rework xilem_core to be compatible with Masonry backend [(original)](@/blog/2024-06-17-roadmap-may-2024.md#rework-xilem-core-to-be-compatible-with-masonry-backend) {#rework-core} | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: This was completed in [xilem#310](https://github.com/linebender/xilem/pull/310). | ||
| Philipp then applied this to Xilem Web in [xilem#403](https://github.com/linebender/xilem/pull/403). | ||
|
|
||
| ### Write documentation [(original)](@/blog/2024-06-17-roadmap-may-2024.md#write-documentation) | ||
|
|
||
| - **Priority**: High | ||
| - **Status**: This was not completed. | ||
| The work in [xilem#310](https://github.com/linebender/xilem/pull/310) did document some of the core, but there is very little user-facing documentation of the Xilem level. | ||
|
|
||
| ### Re-implement async integration [(original)](@/blog/2024-06-17-roadmap-may-2024.md#re-implement-async-integration) | ||
|
|
||
| - **Priority**: Medium | ||
| - **Status**: The core of this task was completed in [xilem#423](https://github.com/linebender/xilem/pull/423) was the main implementation. | ||
| Two-way communication with the async tasks is not yet implemented | ||
|
|
||
| ### Add unit tests [(original)](@/blog/2024-06-17-roadmap-may-2024.md#add-unit-tests) | ||
|
|
||
| - **Priority**: Low | ||
| - **Status**: This was not prioritised. | ||
| We have raised the priority of this item in our next roadmap. | ||
|
|
||
| ### Add basic benchmarks [(original)](@/blog/2024-06-17-roadmap-may-2024.md#add-basic-benchmarks) | ||
|
|
||
| - **Priority**: Low | ||
| - **Status**: There has been no progress on this item. | ||
| This will be a "highest" priority item in the next roadmap. | ||
|
|
||
| ### Progress beyond roadmap {#xilem-extras} | ||
|
|
||
| Xilem has also seen some very useful changes not tracked above, including many community contributions. | ||
| Some highlights of these contributions are: | ||
|
|
||
| - Philipp Mildenberger contributed heavily to Xilem's core design, such as by restoring the ability to return `ViewSequence`s by returning to a previous workaround for Rust's overly-cautious trait solver ([xilem#472](https://github.com/linebender/xilem/pull/394)). | ||
| He also extended the Xilem API by restoring `OneOf` ([xilem#394](https://github.com/linebender/xilem/pull/394)), and implementing various `Adapt` nodes ([xilem#401](https://github.com/linebender/xilem/pull/401)). | ||
| Additionally, he picked up the work to expand `Flex` to use the [new Xilem Core](#rework-core) in [xilem#428](https://github.com/linebender/xilem/pull/428). | ||
| - Jared O'Connell contributed two new examples, a calculator ([xilem#467](https://github.com/linebender/xilem/pull/467)), and a stopwatch ([xilem#492](https://github.com/linebender/xilem/pull/492)). | ||
|
|
||
| There has also been lots of really good progress on Xilem Web, but we do not roadmap Xilem Web and so we do not cover that progress here. | ||
|
|
||
| ## AccessKit [(original)](@/blog/2024-06-17-roadmap-may-2024.md#accesskit) | ||
|
|
||
| Matt Campbell's work on AccessKit has begun, and a proof-of-concept Android adapter is in progress. | ||
| <!-- TODO: Can we point to a PR instead? --> | ||
| At the time of writing there is not a permanent link to this work, but it can be seen in the [android-basics](https://github.com/AccessKit/accesskit/tree/android-basics) branch. | ||
|
|
||
| ## Android specific work [(original)](@/blog/2024-06-17-roadmap-may-2024.md#android-specific-work) | ||
|
|
||
| ### Subclassing View/Application/Activity [(original)](@/blog/2024-06-17-roadmap-may-2024.md#subclassing-view-application-activity) {#subclassing} | ||
|
|
||
| - **Priority**: Highest | ||
| - **Status**: This work is in progress, and unfortunately has required more research than expected. | ||
| We spent some time investigating DEX injection, which Matt has got implemented. | ||
| However, it’s now looking like a custom Activity and View is the way forward, which requires close collaboration with upstream tools. | ||
| We are continuing to prioritise this highly. | ||
|
|
||
| ### Handling IME [(original)](@/blog/2024-06-17-roadmap-may-2024.md#handling-ime) | ||
|
|
||
| - **Priority**: Highest | ||
| - **Status**: This work is ongoing. | ||
| [winit#3787](https://github.com/rust-windowing/winit/pull/3787) allows opening a virtual keyboard, however this does not integrate with full IME (so for example, it is incompatible with autocorrect). | ||
| Real IME requires the Activity to support it directly and therefore involves [content view subclassing](#subclassing). | ||
|
|
||
| ### Gesture recognition [(original)](@/blog/2024-06-17-roadmap-may-2024.md#gesture-recognition) | ||
|
|
||
| - **Priority**: Low | ||
| - **Status**: [xilem#313](https://github.com/linebender/xilem/pull/313) added emulated mouse press and move from touches. | ||
| This unblocked use of Xilem on Android, but is not the intended final state. | ||
| The priority of detecting a scrolling gesture has been increased slightly. | ||
|
|
||
| ## Conclusions | ||
|
|
||
| Overall, the roadmap provided a useful guide of what to work on, and we have implemented many of the most important features from it. | ||
| One clear recurring issue is documentation - even where we have chosen to roadmap this, we have made little progress. | ||
| Similar concerns are clear for testing in Xilem, although we have laid some important groundwork there. | ||
| We firmly recognise the role of documentation as an aid to onboarding, which is becoming more important as we work towards alpha releases. | ||
| I believe that we can make good progress in this area before the end of 2024. |