Major changes in PhotoFlow "stable" branch

[Carmelo_DrRaw]

@RobA

So far the only difficulty I have is backward compatibility for those images I processed in the stable branch where I went straight to lab colour space in the initial raw development.

The packages from today should completely solve the issue with opening old .pfi files with straight conversions to Lab colorspace… please let me know if this is still not fixed for you, or if you still experience other backward compatibility issues…

[RobA]

Thanks for doing this, I’ll check it out when the OSX version is available.

[Carmelo_DrRaw]

@Elle

Can PhotoFlow open CIELab images?

It seems that the underlying VIPS library has some troubles reading such type of image… I will investigate.

[RobA]

Just checked again and whilst there are appimage and windows versions dated 29th there is no OS X version. The one dated 28th does not include the fix I presume as it does not open the direct to Lab files correctly.

[Carmelo_DrRaw]

Correct. I have some issues with the automated OSX building, hopefully something that will be sorted out today…

[Carmelo_DrRaw]

The OSX package is finally available for download! It is tagged with the date of today 30th of November.

[RobA]

The ‘direct to Lab’ files open fine with this version. Thanks again.

[afre (Alan)]

@Carmelo_DrRaw If I embed perspective cor + crop within a Group Layer,

the edges of the image get distorted; e.g.,

This remains in exported images. Outside of the Group Layer, they work as expected.

[Carmelo_DrRaw]

Set the group layer blend mode to passthrough to disable any blending, otherwise it tries to combine two images with different sizes.

[afre (Alan)]

Thanks. Are there other uses for passthrough?

[agriggio (Alberto)]

I don’t know if you prefer issues reported via github or here, but… I just tried the latest appimage of PhotoFlow (./PhotoFlow-20171129_0741-git-stable-070c6d3e7be6e654b13c83e980bb5c2213742b7f.glibc2.17-dbg.glibc2.17-x86_64.AppImage) and I get a segfault if I try to set a custom DCP profile in the RAW developer module. The crash happens as soon as I select “DCP from disk”…

[Carmelo_DrRaw]

Thanks! I will have a look. Generally I prefer issues reported on github 'cause they are easier to follow, but here is fine as well…

[Carmelo_DrRaw]

@agriggio

I get a segfault if I try to set a custom DCP profile in the RAW developer module. The crash happens as soon as I select “DCP from disk”…

Confirmed, I am looking into it…

[Carmelo_DrRaw]

@agriggio

I get a segfault if I try to set a custom DCP profile in the RAW developer module. The crash happens as soon as I select “DCP from disk”…

It was just a missing initialisation of a pointer… the latest AppImage should not crash anymore when switching to DCP profiles.

Thanks for discovering this!

[afre (Alan)]

@Carmelo_DrRaw When I export the image to a TIF (Rec.2020) and import it back into PF, why is the input profile embedded (sRGB)? When I import into GIMP, it recognizes the ICC. Also, GIMP asks me to select a page (of 3) in the import dialog.

[Carmelo_DrRaw]

embedded (sRGB) means “read the embedded profile, and fall back to sRGB is no profile is found”. If the colors of the image imported back into PhF are OK, it means that the Rec2020 profile is loaded and used correctly.

Concerning the 3-pages issue with some images, it looks like there is some EXIF data which is not removed correctly. I have plans to look into this as well…

[Elle (Elle Stone)]

@afre

Are there other uses for passthrough?

Organizing lots of layers - makes it easier to figure out what each group of layers is supposed to accomplish, by allowing to group them without having the actual layer group do anything other than “contain” the layers.

@Carmelo_DrRaw - is my explanation correct? I think I asked Ell (one of the GIMP devs) this question about GIMP’s passthrough blend (new in GIMP-2.9), and this was the answer, I think . . .

@afre - try setting up a passthrough group, and if my understanding is correct, moving a layer in or out of the passthrough group shouldn’t make any difference.

[Carmelo_DrRaw]

@Elle

makes it easier to figure out what each group of layers is supposed to accomplish, by allowing to group them without having the actual layer group do anything other than “contain” the layers.

I think this is indeed the main use of the passthrough mode. I might even consider setting the blend mode to passthrough by default for group layers. At the moment I cannot think of some other typical usage…

@Elle

try setting up a passthrough group, and if my understanding is correct, moving a layer in or out of the passthrough group shouldn’t make any difference.

I have to think about it more carefully, but most likely you are right… at least in the case in which the group only contains one layer. Otherwise, moving a layer in or out could actually change the order in which the layers are applied.

[afre (Alan)]

@Carmelo_DrRaw

embedded (sRGB) means “read the embedded profile, and fall back to sRGB is no profile is found”. If the colors of the image imported back into PhF are OK, it means that the Rec2020 profile is loaded and used correctly.

The selection menu shouldn’t be showing embedded (sRGB) if the profile has been loaded successfully. To avoid confusion maybe indicate the name of the embedded profile.

[afre (Alan)]

@Carmelo_DrRaw PF stable is much snappier than linear_gamma was. Again, thanks for your time and effort in development. I have two feature requests.

1. I often export a TIF file from PF to make edits elsewhere and then import it to take advantage of PF’s color management, which isn’t available in G’MIC for instance. Currently, there is no way to refresh a currently open TIF other than to close the image tab and then reopen the file. It would be handy to have a button or shortcut key to refresh the image to reflect the altered file contents.

2. It would also be handy to have a reset tool parameters button per tab or section in a multi-tab or otherwise complex module. E.g., this feature would be nice to have in Raw developer; esp. when I have trouble with the WB tab.