You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is probably worth adding to the API Docs as it's a useful feature. It's not currently because in my mind, this isn't a core osxphotos feature. With the exception of timewarp and batch-edit commands (which aren't exposed in the public API), osxphotos provides read only access to the Photos library. It is not a goal of mine to enable the API to modify the library. I've built some tools like PhotosAlbum as internal helpers to do modification where necessary but there are actually implemented using another library of mine, PhotoScript.
for p in edited_photos:
p.albums.append('GR3 edited')
This syntax won't be supported in osxphotos because it would require a complete rewrite of the API which is predicated on the PhotoInfo object being read only.
It does however make sense to add append and extend to the PhotosAlbum code and to expose this as a public API of osxphotos. So few people use the API that focus most of my effort on the CLI which is widely used.
This is probably worth adding to the API Docs as it's a useful feature. It's not currently because in my mind, this isn't a core osxphotos feature. With the exception of
timewarp
andbatch-edit
commands (which aren't exposed in the public API), osxphotos provides read only access to the Photos library. It is not a goal of mine to enable the API to modify the library. I've built some tools likePhotosAlbum
as internal helpers to do modification where necessary but there are actually implemented using another library of mine, PhotoScript.This syntax won't be supported in osxphotos because it would require a complete rewrite of the API which is predicated on the
PhotoInfo
object being read only.It does however make sense to add
append
andextend
to the PhotosAlbum code and to expose this as a public API of osxphotos. So few people use the API that focus most of my effort on the CLI which is widely used.Originally posted by @RhetTbull in #1523 (comment)
The text was updated successfully, but these errors were encountered: