There are some more, but they depend on raw_capture_info. Will add another series after merging that.
For testing, it's probably best to confirm each issue in unified, then check whether it was fixed in this branch.
TODO:
exact height for cameras with focus pixels (IIRC there are some issues with MLVFS)
port the mlv_lite changes to mlv_rec if desired
Comments (3)
DeafEyeJedi
Trying to wrap my head around the 'to do' list especially w cameras that have focus pixels, I feel like we're wasting your time by not generating valuable feedbacks in here and any hints would be appreciated.
Alexauthor
You are probably interested in this paragraph, not in the coding TODOs:
For testing, it's probably best to confirm each issue in unified, then check whether it was fixed in this branch.
Refer to each commit message for details.
Daniel Fort
Just a note on the todo list.
exact height for cameras with focus pixels (IIRC there are some issues with MLVFS)
Looking through my notes it seems that this issue came up around the time Raw fixes part 3 was merged. (pull request #778)
I didn't notice it at first because it wasn't consistent but it does seem that the vertical resolution can vary by 1 pixel after these changes. Not sure exactly what caused it. @Danne fixed my focus pixel script to allow for a little bit of slop but since MLVFS identifies the focus pixel map files by their file names which contains the exact size of the full raw buffer, it has a problem when the size varies.
It seems that this only affects the vertical resolution and the map files still work if renamed so the focus pixel coordinates do not move.
Trying to wrap my head around the 'to do' list especially w cameras that have focus pixels, I feel like we're wasting your time by not generating valuable feedbacks in here and any hints would be appreciated.
You are probably interested in this paragraph, not in the coding TODOs:
For testing, it's probably best to confirm each issue in unified, then check whether it was fixed in this branch.
Refer to each commit message for details.
Just a note on the todo list.
Looking through my notes it seems that this issue came up around the time Raw fixes part 3 was merged. (pull request #778) I didn't notice it at first because it wasn't consistent but it does seem that the vertical resolution can vary by 1 pixel after these changes. Not sure exactly what caused it. @Danne fixed my focus pixel script to allow for a little bit of slop but since MLVFS identifies the focus pixel map files by their file names which contains the exact size of the full raw buffer, it has a problem when the size varies.
It seems that this only affects the vertical resolution and the map files still work if renamed so the focus pixel coordinates do not move.