Project

General

Profile

Bug #11124

Something causes multi_name to go to wrong history items.

Added by Roman Lebedev over 3 years ago. Updated over 3 years ago.

Status:
Fixed
Priority:
Critical
Assignee:
-
Category:
General
Target version:
Start date:
08/22/2016
Due date:
% Done:

100%

Estimated time:
Affected Version:
git master branch
System:
Debian
bitness:
64-bit
hardware architecture:
amd64/x86

Description

Steps:
1. clean library, clean image (no sidecar)
2. load image into darktable
3. add some spot correction
4. add new instance of spot correction, add some more correction
5. exit darktable
6. load image into darktable
There will be two "spot correction", both without any MI name, and flip and sharpen will both have MI name of "1"

If this is yet another regression of mine, i guess i will mostly step away from dt, for some time.

Associated revisions

Revision 829fb7d7 (diff)
Added by Tobias Ellinghaus over 3 years ago

Use a more consistent XMP schema for history

This breaks backwards compatibility of sidecar files. It has the big
benefit that things like empty multi_name (or even empty params) would
be supported. Next step is to add support for human readable params in
XMP using module introspection.

Fixes #11124 and probably others.

History

#1 Updated by Roman Lebedev over 3 years ago

Happens in 2.0.x, and even release-2.0.0
Exiv2 change?

#2 Updated by Tobias Ellinghaus over 3 years ago

  • % Done changed from 0 to 20
  • Status changed from New to Triaged

It seems some code path set the mi name to be empty, which, as said in quite a few bug reports, must not happen.

#3 Updated by Tobias Ellinghaus over 3 years ago

I am currently working on a new and improved XMP layout that will avoid problems like this in the future. Until that is used there isn't much that can be done I guess.

#4 Updated by Tobias Ellinghaus over 3 years ago

  • % Done changed from 20 to 100
  • Status changed from Triaged to Fixed

#5 Updated by Roman Lebedev over 3 years ago

  • Target version set to 2.2.0

Also available in: Atom PDF

Go to top