Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[Impeller] Map UVs correctly for TiledTextureContents - #38894

Merged
bdero merged 3 commits into
flutter-team-archive:mainfrom
bdero:bdero/tiled-texture-uvs
Jan 16, 2023
Merged

bdero merged 3 commits into
flutter-team-archive:mainfrom
bdero:bdero/tiled-texture-uvs

Conversation

@bdero

@bdero bdero commented Jan 15, 2023

Copy link
Copy Markdown
Contributor

Makes the UVs relative to the geometry origin. Discovered this issue while implementing Impeller Scene widgets.

@bdero
bdero force-pushed the bdero/tiled-texture-uvs branch from df20aa3 to e24aeab Compare January 15, 2023 13:43

@zanderso zanderso left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm w/ nit

auto geometry_result =
GetGeometry()->GetPositionBuffer(renderer, entity, pass);

// TODO(bdero): The geometry should be fetched from GetPositionUVBuffer and

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a link to an issue here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@bdero bdero added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 16, 2023
@auto-submit

auto-submit Bot commented Jan 16, 2023

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/engine, pr: 38894, due to - The status or check suite Mac Unopt has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jan 16, 2023
@bdero
bdero merged commit afa093d into flutter-team-archive:main Jan 16, 2023
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Jan 18, 2023
* 5aee791 Roll Dart SDK from 645fd748e79e to 0097dcbada06 (3 revisions) (flutter-team-archive/engine#38883)

* 78a68b9 Roll Skia from c64a10d525d1 to aa99fa0bdd31 (10 revisions) (flutter-team-archive/engine#38886)

* 398af8c Roll Fuchsia Mac SDK from w0hr1ZMvYGJnWInwK... to nPbEqgIfE8TLOokQH... (flutter-team-archive/engine#38889)

* 7035027 Roll Skia from aa99fa0bdd31 to 5a51583d09dd (2 revisions) (flutter-team-archive/engine#38892)

* 2075cf3 Roll Fuchsia Mac SDK from nPbEqgIfE8TLOokQH... to SmEVvAFBC8oriPBou... (flutter-team-archive/engine#38897)

* ef06ef6 Roll Skia from 5a51583d09dd to 34a14247bd4b (1 revision) (flutter-team-archive/engine#38901)

* 0d374a9 Roll Skia from 34a14247bd4b to c727016cc09e (1 revision) (flutter-team-archive/engine#38903)

* 31f2020 Roll Skia from c727016cc09e to 214411b09e92 (2 revisions) (flutter-team-archive/engine#38907)

* a3006de Bump actions/setup-python from 4.4.0 to 4.5.0 (flutter-team-archive/engine#38909)

* eacc09c Bump github/codeql-action from 2.1.37 to 2.1.38 (flutter-team-archive/engine#38910)

* c33c11c Roll Skia from 214411b09e92 to 3704033dd106 (1 revision) (flutter-team-archive/engine#38908)

* afa093d [Impeller] Map UVs correctly for TiledTextureContents (flutter-team-archive/engine#38894)

* 46ea2b3 Roll Fuchsia Mac SDK from SmEVvAFBC8oriPBou... to bZfX8KCoI3CfxEzsG... (flutter-team-archive/engine#38912)

* aadb971 Roll Skia from 3704033dd106 to 58e745419425 (4 revisions) (flutter-team-archive/engine#38936)

* 24b21a5 Roll Fuchsia Mac SDK from bZfX8KCoI3CfxEzsG... to iwCuzAuMgsk75hlT0... (flutter-team-archive/engine#38938)

* e655580 Roll Skia from 58e745419425 to 6afb97022fa7 (1 revision) (flutter-team-archive/engine#38943)

* 5932bad Revert "Revert "[web] Don't overwrite editing state with semantic updates (#38271)" (#38562)" (flutter-team-archive/engine#38854)

* 928772e [web] Reduce code size impact of fallback font data (flutter-team-archive/engine#38787)

* 88a8d9a Roll Skia from 6afb97022fa7 to 8ea9b39f7213 (18 revisions) (flutter-team-archive/engine#38952)

* 3a47943 Roll Skia from 8ea9b39f7213 to 69d8378097df (6 revisions) (flutter-team-archive/engine#38953)

* 641623a wrap focus() in a post update callback (flutter-team-archive/engine#38958)

* 7d71ee3 Roll Fuchsia Mac SDK from iwCuzAuMgsk75hlT0... to -1N2YUML8GqNhK8YP... (flutter-team-archive/engine#38960)

* d762142 reland fix canvas drawLine bugs (flutter-team-archive/engine#38949)

* 0d79533 [Impeller] Fix back-to-back clip restoration (flutter-team-archive/engine#38964)

* 19fe86c Update scenario screenshots to iOS 16 (flutter-team-archive/engine#38962)

* eb18ac0 Fix iOS safari keyboard issue when semantics is enabled (flutter-team-archive/engine#38822)

* 84a73d9 [Impeller Scene] Add SceneNodeValue for synchronously fetching loaded ipscenes (flutter-team-archive/engine#38913)

* 7bff5dd Revert "Roll Dart SDK from 645fd748e79e to 0097dcbada06 (3 revisions) (#38883)" (flutter-team-archive/engine#38973)

* c52b290 Roll Fuchsia Mac SDK from -1N2YUML8GqNhK8YP... to P5QcCJU8I71xVXuMT... (flutter-team-archive/engine#38976)
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…18720)

* 5aee791 Roll Dart SDK from 645fd748e79e to 0097dcbada06 (3 revisions) (flutter-team-archive/engine#38883)

* 78a68b9 Roll Skia from c64a10d525d1 to aa99fa0bdd31 (10 revisions) (flutter-team-archive/engine#38886)

* 398af8c Roll Fuchsia Mac SDK from w0hr1ZMvYGJnWInwK... to nPbEqgIfE8TLOokQH... (flutter-team-archive/engine#38889)

* 7035027 Roll Skia from aa99fa0bdd31 to 5a51583d09dd (2 revisions) (flutter-team-archive/engine#38892)

* 2075cf3 Roll Fuchsia Mac SDK from nPbEqgIfE8TLOokQH... to SmEVvAFBC8oriPBou... (flutter-team-archive/engine#38897)

* ef06ef6 Roll Skia from 5a51583d09dd to 34a14247bd4b (1 revision) (flutter-team-archive/engine#38901)

* 0d374a9 Roll Skia from 34a14247bd4b to c727016cc09e (1 revision) (flutter-team-archive/engine#38903)

* 31f2020 Roll Skia from c727016cc09e to 214411b09e92 (2 revisions) (flutter-team-archive/engine#38907)

* a3006de Bump actions/setup-python from 4.4.0 to 4.5.0 (flutter-team-archive/engine#38909)

* eacc09c Bump github/codeql-action from 2.1.37 to 2.1.38 (flutter-team-archive/engine#38910)

* c33c11c Roll Skia from 214411b09e92 to 3704033dd106 (1 revision) (flutter-team-archive/engine#38908)

* afa093d [Impeller] Map UVs correctly for TiledTextureContents (flutter-team-archive/engine#38894)

* 46ea2b3 Roll Fuchsia Mac SDK from SmEVvAFBC8oriPBou... to bZfX8KCoI3CfxEzsG... (flutter-team-archive/engine#38912)

* aadb971 Roll Skia from 3704033dd106 to 58e745419425 (4 revisions) (flutter-team-archive/engine#38936)

* 24b21a5 Roll Fuchsia Mac SDK from bZfX8KCoI3CfxEzsG... to iwCuzAuMgsk75hlT0... (flutter-team-archive/engine#38938)

* e655580 Roll Skia from 58e745419425 to 6afb97022fa7 (1 revision) (flutter-team-archive/engine#38943)

* 5932bad Revert "Revert "[web] Don't overwrite editing state with semantic updates (flutter#38271)" (flutter#38562)" (flutter-team-archive/engine#38854)

* 928772e [web] Reduce code size impact of fallback font data (flutter-team-archive/engine#38787)

* 88a8d9a Roll Skia from 6afb97022fa7 to 8ea9b39f7213 (18 revisions) (flutter-team-archive/engine#38952)

* 3a47943 Roll Skia from 8ea9b39f7213 to 69d8378097df (6 revisions) (flutter-team-archive/engine#38953)

* 641623a wrap focus() in a post update callback (flutter-team-archive/engine#38958)

* 7d71ee3 Roll Fuchsia Mac SDK from iwCuzAuMgsk75hlT0... to -1N2YUML8GqNhK8YP... (flutter-team-archive/engine#38960)

* d762142 reland fix canvas drawLine bugs (flutter-team-archive/engine#38949)

* 0d79533 [Impeller] Fix back-to-back clip restoration (flutter-team-archive/engine#38964)

* 19fe86c Update scenario screenshots to iOS 16 (flutter-team-archive/engine#38962)

* eb18ac0 Fix iOS safari keyboard issue when semantics is enabled (flutter-team-archive/engine#38822)

* 84a73d9 [Impeller Scene] Add SceneNodeValue for synchronously fetching loaded ipscenes (flutter-team-archive/engine#38913)

* 7bff5dd Revert "Roll Dart SDK from 645fd748e79e to 0097dcbada06 (3 revisions) (flutter#38883)" (flutter-team-archive/engine#38973)

* c52b290 Roll Fuchsia Mac SDK from -1N2YUML8GqNhK8YP... to P5QcCJU8I71xVXuMT... (flutter-team-archive/engine#38976)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

2 participants