Skip to content

Data Fixes#291

Merged
freezy merged 16 commits into
masterfrom
fix/data
Feb 15, 2021
Merged

Data Fixes#291
freezy merged 16 commits into
masterfrom
fix/data

Conversation

@freezy
Copy link
Copy Markdown
Owner

@freezy freezy commented Feb 15, 2021

This PR fixes a few bugs when writing data. Additionally, we now are aware which tags were added and removed in VP10.7, and which ones were added and removed in VPE, so we can write near-identical files by setting the WRITE_VP106 and WRITE_VP107 compiler flags (documentation).

Changes

  • Alpha channel of color values is now correctly written.
  • Layer names are correctly computed when importing a 10.6 file.
  • Add support for Extended ASCII strings.
  • Clear texture and material references that don't exist before writing (VP 10.7 behavior)
  • Fix bug in writing animation vertices which caused VP to hang when re-reading the file
  • Add support for Elasticity Falloff in walls (added in VP 10.7)
  • Add support for table notes (added in VP 10.7)

@freezy freezy self-assigned this Feb 15, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 15, 2021

Codecov Report

Merging #291 (9d1e074) into master (9a81ccd) will increase coverage by 0.46%.
The diff coverage is 77.38%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #291      +/-   ##
==========================================
+ Coverage   65.59%   66.05%   +0.46%     
==========================================
  Files         136      136              
  Lines        8370     8417      +47     
  Branches      859      866       +7     
==========================================
+ Hits         5490     5560      +70     
+ Misses       2645     2620      -25     
- Partials      235      237       +2     
Impacted Files Coverage Δ
VisualPinball.Engine/IO/BiffIgnoreAttribute.cs 100.00% <ø> (ø)
VisualPinball.Engine/Math/DragPointData.cs 37.14% <0.00%> (-2.26%) ⬇️
VisualPinball.Engine/VPT/BinaryData.cs 90.32% <ø> (+19.35%) ⬆️
VisualPinball.Engine/VPT/Flasher/FlasherData.cs 97.05% <ø> (ø)
VisualPinball.Engine/VPT/Flipper/FlipperData.cs 16.34% <ø> (ø)
VisualPinball.Engine/VPT/Kicker/KickerData.cs 41.17% <ø> (ø)
VisualPinball.Engine/VPT/Surface/SurfaceData.cs 27.45% <0.00%> (ø)
VisualPinball.Engine/VPT/Table/TableData.cs 86.95% <ø> (ø)
VisualPinball.Engine/VPT/TextureData.cs 85.50% <ø> (+10.14%) ⬆️
...isualPinball.Engine/VPT/Primitive/PrimitiveData.cs 53.84% <42.85%> (-0.58%) ⬇️
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4da4f0e...9d1e074. Read the comment docs.

@freezy freezy merged commit 5ababb9 into master Feb 15, 2021
@freezy freezy deleted the fix/data branch February 15, 2021 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant