[RESPDAM-16] Digital artifacts with ResponsiveDamTemplatingFunctions Created: 22/Jul/19  Updated: 14/Feb/22  Resolved: 25/Jul/19

Status: Closed
Project: DAM Focal
Component/s: None
Affects Version/s: None
Fix Version/s: 1.1.4

Type: Bug Priority: High
Reporter: Stefan Vikoler Assignee: Adrien Manzoni
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: 0d
Time Spent: 5.5h
Original Estimate: Not Specified

Attachments: PNG File antialiasing.png     File config.modules.dam-images-responsive.config.yaml     PNG File image-2019-07-23-10-48-48-937.png     PNG File image-2019-07-23-17-04-05-804.png     PNG File respDamFn.png    
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

Displaying images with the Magnolia ResponsiveDamTemplatingFunctions (left picture) creates unwanted digital artifacts:

 

Original: https://cdn.ecomstage.bruendl.at/dam/jcr:3f8c52e6-e11c-46e6-8cab-c32035071c96/SpieljochbahnTalstation-Winter1718-Innen01-(c)Woeckinger.jpg
ResponsiveDamTemplatingFunctions: https://cdn.ecomstage.bruendl.at/.imaging/responsive/dam/cubic/460x460/dam/jcr:3f8c52e6-e11c-46e6-8cab-c32035071c96/SpieljochbahnTalstation-Winter1718-Innen01-(c)Woeckinger.jpg



 Comments   
Comment by Thomas Peintner [ 23/Jul/19 ]

For comparison, this is how it comes out with Magnolia 5.5.12 Image Rendition Generation, wich is much better than it is now since 6.0

Comment by Adrien Manzoni [ 23/Jul/19 ]

Hi svikoler, Hi tpeintner,

May I ask you your orignal picture and your list of configured variationsets (an xml export would be perfect) ? 

Thanks !

Comment by Julius Rabe [ 23/Jul/19 ]

Hi,
original file is the file from description: https://cdn.ecomstage.bruendl.at/dam/jcr:3f8c52e6-e11c-46e6-8cab-c32035071c96/SpieljochbahnTalstation-Winter1718-Innen01-(c)Woeckinger.jpg

config is: config.modules.dam-images-responsive.config.yaml [

https://cdn.ecomstage.bruendl.at/.imaging/responsive/dam/cubic/460x460/dam/jcr:3f8c52e6-e11c-46e6-8cab-c32035071c96/SpieljochbahnTalstation-Winter1718-Innen01-(c)Woeckinger.jpg]

Rendition is generated with ResponsiveDamTemplationFunctions
info.magnolia.dam.imaging.responsive.templating.functions.ResponsiveDamTemplatingFunctions#buildLink

Cheers

Comment by Adrien Manzoni [ 23/Jul/19 ]

Thanks j.rabe

We have just release a new version. Could you please try that version ?

<!-- DAM focal -->
<dependency>
  <groupId>info.magnolia.dam</groupId>
  <artifactId>magnolia-dam-images-responsive</artifactId>
  <version>1.1.3</version>
</dependency>
Comment by Julius Rabe [ 23/Jul/19 ]

Hi,
just installed this module version 1.1.3 but I do not see an improvement in image quality, should I?

Comment by Adrien Manzoni [ 23/Jul/19 ]

Hi j.rabe

Could you please try that version ?

<!-- DAM focal -->
<dependency>
  <groupId>info.magnolia.dam</groupId>
  <artifactId>magnolia-dam-images-responsive</artifactId>
  <version>1.1.4</version>
</dependency>

We brought more antialiasing control on the imaging operations. Please check the following readme to check how to enhance it even more (section Enhance the interpolation algorithm)

https://git.magnolia-cms.com/projects/SERVICES/repos/dam-focal/browse

I have attached a screenshot of the new rendering.

Comment by Julius Rabe [ 24/Jul/19 ]

Hi Adrien,

thanks a lot for your effort!
Just installed 1.1.4 and I see an huge improvement.

Will have it checked by some other peaple once we get it into the next Release of the Project but I pretty sure it will be pleasant.

Cheers,
Julius

 

Generated at Mon Feb 12 00:41:42 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.