Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgraded Magick.NET. #1383

Merged
merged 4 commits into from
Oct 14, 2020
Merged

Upgraded Magick.NET. #1383

merged 4 commits into from
Oct 14, 2020

Conversation

dlemstra
Copy link
Member

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following matches the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

This updates Magick.NET and sets an option for the BMP coder that fixes #1380. This option was added because the BMP coder in ImageMagick is now stricter and you need to set it to be able to read all files.

@JimBobSquarePants
Copy link
Member

JimBobSquarePants commented Oct 14, 2020

@dlemstra Doesn't this mean that according to Magick.NET our encoder is setting the length incorrectly? We have new exceptions thrown in our BmpEncoderTests where we have encoded the image.

@dlemstra
Copy link
Member Author

That could be a possibility then. Will investigate that this weekend. Thought the input was coming from a random test file.

@codecov
Copy link

codecov bot commented Oct 14, 2020

Codecov Report

Merging #1383 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1383   +/-   ##
=======================================
  Coverage   82.85%   82.85%           
=======================================
  Files         690      690           
  Lines       31074    31074           
  Branches     3512     3512           
=======================================
  Hits        25747    25747           
  Misses       4605     4605           
  Partials      722      722           
Flag Coverage Δ
#unittests 82.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 d67cb43...0f144c6. Read the comment docs.

@JimBobSquarePants
Copy link
Member

I'm gonna merge this despite the flaky unrelated test. It allows me to cleanup #1378 and fix the coverage issues there.

I'll keep #1380 open for tracing purposes.

@JimBobSquarePants JimBobSquarePants merged commit 581efcf into master Oct 14, 2020
@JimBobSquarePants JimBobSquarePants deleted the dl/upgrade-magick-net branch October 14, 2020 15:50
JimBobSquarePants added a commit that referenced this pull request Mar 13, 2021
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.

Magick.NET 7.22.0 Fails to decode some BMP images on Ubuntu
2 participants