Skip to content
Snippets Groups Projects

Sprite model manifest and sprite instance preparation improvements

Merged Imbris requested to merge veloren/dev/veloren:imbris/small-fixes-2489235 into master
All threads resolved!
  • By opening this merge request, you agree to release your code and all other changes under the GPL 3 license and to abide by the terms set by this license.
  • Migrations have been added if applicable (migrations are always necessary for removing/renaming item asset files)
  • Significant changes of this merge request have been added to the changelog.

This includes some performance improvements for sprite processing when preparing to render a chunk and some changes to the sprite manifest format. This also adds more checks/tests for a few errors that can happen such as missing sprites.

See commit message for a few more details.

Edited by Imbris

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • crab
  • Nice stuff :thumbsup:

  • crab approved this merge request

    approved this merge request

  • Imbris added 1 commit

    added 1 commit

    • 1a179e36 - Remove filter scoring and remove a comment that appears to be obselete

    Compare with previous version

  • Imbris resolved all threads

    resolved all threads

  • Imbris added 5 commits

    added 5 commits

    • 1a179e36...9641b487 - 2 commits from branch veloren:master
    • ab67495e - Remove unnecessary pub on a module
    • 12d6f06f - Sprite model manifest and sprite instance preparation improvements
    • 427fff99 - Remove filter scoring and remove a comment that appears to be obselete

    Compare with previous version

  • Imbris added this merge request to the merge train at position 2

    added this merge request to the merge train at position 2

  • Imbris changed the description

    changed the description

  • Imbris mentioned in commit 79cb3eda

    mentioned in commit 79cb3eda

  • merged

  • Please register or sign in to reply
    Loading