feat!: add LevelIndices type defining a level's location in a project and use it in LevelSelection::Indices
#221
Conversation
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
…nough, I suppose)
…, and naively update some methods
Trouv
added a commit
that referenced
this pull request
Feb 11, 2024
🤖 I have created a release *beep* *boop* --- ## [0.9.0](v0.8.0...v0.9.0) (2024-02-11) ### ⚠ BREAKING CHANGES * upgrade to bevy 0.12 ([#265](#265)) * upgrade to LDtk 1.5.3, dropping support for previous versions ([#295](#295)) * add `SpawnExclusions` to `LdtkSettings` for skipping layers by identifier ([#275](#275)) * add layer entity for Entity layers, changing the hierarchy ([#257](#257)) * upgrade to LDtk types and examples to 1.4.1 (drop support for <1.4.1) ([#256](#256)) * LdtkLevel renamed to LdtkExternalLevel and is no longer used as a component ([#244](#244)) * redesign LdtkProject with better level data accessors and correct modeling of internal/external levels ([#244](#244)) * use the bundle's `Default` implementation rather than the field's in `LdtkEntity` and `LdtkIntCell` derive macros ([#222](#222)) * add `RawLevelAccessor` trait for `LdtkJson` level borrowing/iteration, replacing existing methods ([#225](#225)) * add `LevelIndices` type defining a level's location in a project and use it in `LevelSelection::Indices` ([#221](#221)) * change `LevelEvent` inner types from `String` to `LevelIid` ([#219](#219)) * change `LevelSet` inner type from `String` to `LevelIid` ([#219](#219)) * change `LevelSelection::Iid` inner type from `String` to `LevelIid` ([#219](#219)) * replace `LevelSet::from_iid` with `LevelSet::from_iids`, which can convert from any collection of strings. ([#219](#219)) * use new LevelIid type in LevelEvent, LevelSet, and LevelSelection, plus other improvements ([#219](#219)) * `LdtkProject::project` and `LdtkLevel::level` fields have both been renamed to `data` ([#206](#206)) * All fields of `LdtkProject` and `LdtkLevel` are now privatized, and have immutable getter methods ([#206](#206)) * `LevelMap` and `TilesetMap` type aliases have been removed ([#206](#206)) * `LdtkAsset` and `LdtkProject` are now exported in new `assets` module instead of `lib.rs` ([#206](#206)) * asset `Loader` types are now private ([#206](#206)) * `LdtkAsset` renamed to `LdtkProject` ([#206](#206)) ### Features * add `LevelIndices` type defining a level's location in a project and use it in `LevelSelection::Indices` ([#221](#221)) ([59618fe](59618fe)) * add `RawLevelAccessor` trait for `LdtkJson` level borrowing/iteration, replacing existing methods ([#225](#225)) ([d3de2d9](d3de2d9)) * add `SpawnExclusions` to `LdtkSettings` for skipping layers by identifier ([#275](#275)) ([282404d](282404d)), closes [#272](#272) * add layer entity for Entity layers, changing the hierarchy ([#257](#257)) ([ee20a53](ee20a53)) * add LdtkJsonWithMetadata type for representing internal- and external-level project data with generics ([#242](#242)) ([630434a](630434a)) * add LdtkProjectData for representing either internal- or external-level project data concretely ([#243](#243)) ([c530bc9](c530bc9)) * add level locale types and begin splitting internal_levels and external_levels features ([#237](#237)) ([8129e55](8129e55)) * add LevelIid component and spawn it on every level ([#215](#215)) ([ad83455](ad83455)) * add LoadedLevel type that wraps around levels with complete data ([#214](#214)) ([3d40c15](3d40c15)) * add types and traits around LevelMetadata ([#229](#229)) ([382dea2](382dea2)) * change `LevelEvent` inner types from `String` to `LevelIid` ([#219](#219)) ([0039ed7](0039ed7)) * change `LevelSelection::Iid` inner type from `String` to `LevelIid` ([#219](#219)) ([0039ed7](0039ed7)) * change `LevelSet` inner type from `String` to `LevelIid` ([#219](#219)) ([0039ed7](0039ed7)) * LdtkLevel renamed to LdtkExternalLevel and is no longer used as a component ([#244](#244)) ([670cd4e](670cd4e)) * redesign LdtkProject with better level data accessors and correct modeling of internal/external levels ([#244](#244)) ([670cd4e](670cd4e)) * replace `LevelSet::from_iid` with `LevelSet::from_iids`, which can convert from any collection of strings. ([#219](#219)) ([0039ed7](0039ed7)) * upgrade to bevy 0.12 ([#265](#265)) ([194731e](194731e)) * upgrade to LDtk 1.5.3, dropping support for previous versions ([#295](#295)) ([4926a50](4926a50)) * upgrade to LDtk types and examples to 1.4.1 (drop support for <1.4.1) ([#256](#256)) ([ab21e2c](ab21e2c)) * use new LevelIid type in LevelEvent, LevelSet, and LevelSelection, plus other improvements ([#219](#219)) ([0039ed7](0039ed7)) * use the bundle's `Default` implementation rather than the field's in `LdtkEntity` and `LdtkIntCell` derive macros ([#222](#222)) ([f003127](f003127)) ### Bug Fixes * don't apply level set until project and dependencies are completely loaded ([#296](#296)) ([dbfe1c6](dbfe1c6)) * normalize resolved asset paths using `path_clean` ([#255](#255)) ([33a8998](33a8998)), closes [#240](#240) * only spawn invisible tiles on first sub-layer of AutoTile+IntGrid layers ([#231](#231)) ([d2873e3](d2873e3)) * recalculate layer offset to adjust for tileset sizes ([#254](#254)) ([c00085d](c00085d)) * use entity definition tile size instead of entity instance tile size as basis when calculating ldtk entity scale ([#271](#271)) ([833af01](833af01)) ### Documentation Changes * add 0.8 to 0.9 migration guide ([#266](#266)) ([bb91660](bb91660)) * add collectathon cargo example ([#288](#288)) ([32dfb85](32dfb85)) * add mdbook with outline and introduction ([#261](#261)) ([810b25a](810b25a)) * add tile-based game example w/ a tutorial in the book, replacing getting-started guide ([#269](#269)) ([2d43efa](2d43efa)) * document all-features in docs.rs ([#252](#252)) ([321bb07](321bb07)) * reference book in API ref and README.md, replacing redundant sections ([#282](#282)) ([e7afdad](e7afdad)) * remove README.md caveat for hot reloading external levels ([#253](#253)) ([59eb6b3](59eb6b3)) * write *Anatomy of the World* chapter of book ([#285](#285)) ([29d5e33](29d5e33)) * write *Create bevy relations from ldtk entity references* chapter of book ([#287](#287)) ([8080f24](8080f24)) * write *Game Logic Integration* chapter of the book ([#279](#279)) ([a62a556](a62a556)) * write *Level Selection* chapter of book ([#284](#284)) ([226c60c](226c60c)) * write *Make level selection follow player* chapter of book ([#293](#293)) ([201d908](201d908)) * write *Respawn levels and worlds* chapter of book ([#289](#289)) ([55ed30f](55ed30f)) ### Code Refactors * `LdtkAsset` and `LdtkProject` are now exported in new `assets` module instead of `lib.rs` ([#206](#206)) ([fe44774](fe44774)) * `LdtkAsset` renamed to `LdtkProject` ([#206](#206)) ([fe44774](fe44774)) * `LdtkProject::project` and `LdtkLevel::level` fields have both been renamed to `data` ([#206](#206)) ([fe44774](fe44774)) * `LevelMap` and `TilesetMap` type aliases have been removed ([#206](#206)) ([fe44774](fe44774)) * All fields of `LdtkProject` and `LdtkLevel` are now privatized, and have immutable getter methods ([#206](#206)) ([fe44774](fe44774)) * asset `Loader` types are now private ([#206](#206)) ([fe44774](fe44774)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: Trevor Lovell <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To provide constant lookup of raw/loaded levels in an asset, we need to use its world index/level index separately. Simply iterating through all the levels with the existing
LdtkProject::iter_levelsand getting the.nth()element isn't constant, even though it does chain together root levels and world levels. This PR provides a simple typeLevelIndicesthat stores a world index and a level index, which can be used for this kind of lookup. Methods for performing this lookup and more have been added toLdtkJson/LdtkProject. It is also further integrated with the API by being used in a newLevelSelection::Indicesvariant, which replacesLevelSelection::Index.Getting a level by
LevelSelectioncan now be made more performant if users are using theIndicesvariant, but this optimization will be implemented in a future PR. Some new convenience constructor methods have been added toLevelSelectionto ease the migration.Changes
LevelIndicestypeiter_levels_with_indicesmethods toLdtkProjectandLdtkJsonthat iterate through levels while enumerating them using the new typeget_level_at_indicesmethods toLdtkProjectandLdtkJsonfor constant lookup of levels withLevelIndicesLevelSelection::indexconstructor method for easing migration to new variant for most usersLevelSelection::indicesconstructor method for more ergonomically using the new variant in multi-world projectsBreaking
feat!: replace
LevelSelection::IndexwithLevelSelection::Indicesvariant that usesLevelIndicesinternallyBREAKING-CHANGE: Since it has been replaced by
LevelSelection::Indices, any usage ofLevelSelection::Indexwill break. Construction of the new variant can easily be migrated for non-multi-world projects with theLevelSelection::indexconstructor.