Rhet Turnbull
Rhet Turnbull
Maintaining backwards compatibility comes at a significant cost both in terms of your time and complexity of code. Python setup tools allow version pinning to deal with this sort of...
> I wrote quite a bit about this in the issue. Do I really need to go into even more detail? Nope, wasn't trying to imply that. I guess as...
Yes, I think it's likely best to explicitly list the path of all external commands. Not doing so introduces a potential security vulnerability as an attacker could place malicious versions...
> OK. But if he could do that, you've already lost the battle. :P True...but I suppose it's best not to make it easier for him/her. 😄 Not sure exactly...
For place, rewrite PlaceInfo __str__ to just emit the actual details so can search a single string. What's currently the __str__ method should really be __repr__ https://github.com/RhetTbull/osxphotos/blob/d7a9ad1d0a6d1c4327e9d43b7719d860abd34836/osxphotos/placeinfo.py#L626-L635
I've not been able to reverse engineer the photo adjustments. I've not spent considerable time on this but have looked at the database and tried to understand how adjustments were...
@mccarverone Thanks to some amazing [sleuthing](https://github.com/neilpa/photohack/issues/4) by @neilpa , I'll now be able to extract the adjustments/edits from Photos. I'll add the ability to extract basic information about the adjustments....
Crop data looks like this: ``` {'adjustments': [{'enabled': True, 'formatVersion': 1, 'identifier': 'RKRawDecodeOperation', 'settings': {'DGOperationClassName': 'DGRawDecodeOperation', 'DGOperationDisplayName': 'RAW Fine Tuning', 'DGOperationIdentifier': 'RKRawDecodeOperation', 'enabled': True, 'inputKeys': {'inputMethodVersion': '8'}}}, {'enabled': True, 'formatVersion':...
I think I might be able to get the crop data into the XMP but I'm not hopeful on much of the edits beyond that. All the other edits would...
Can't fix this until I get access to a Monterey machine. Not sure if this happens on both Intel and M1 or just M1. Terminal.app should ask for authorization as...