Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EXIF information #5106

Closed
emresaracoglu opened this issue May 25, 2017 · 27 comments
Closed

EXIF information #5106

emresaracoglu opened this issue May 25, 2017 · 27 comments
Assignees
Labels
1. to develop Accepted and waiting to be taken care of enhancement

Comments

@emresaracoglu
Copy link

emresaracoglu commented May 25, 2017

EXIF support possible for milestone 13?
I think with this feature, many improvements can be made.

nextcloud/ios#170 by me
nextcloud/maps#4 by @jancborchardt
nextcloud/android#4501 by @Gatak

@emresaracoglu emresaracoglu changed the title EXIF information for 12 EXIF information for 13 May 25, 2017
@Spartachetto
Copy link

Not sure this is a feature of the server.
You can find some information about this topic here: MediaMetadata app to extract and store meta data from media files

@nickvergessen
Copy link
Member

I agree that this should better be done as an app.

@jancborchardt
Copy link
Member

@oparoz is there any app work-in-progress available online which was done by that student?

@nickvergessen
Copy link
Member

Follow the link to the forum posted above:
https://help.nextcloud.com/t/mediametadata-app-to-extract-and-store-meta-data-from-media-files/1601

there is a link in the first post to the wiki of the repo:
https://github.com/interfasys/mediametadata/wiki/Possible-Fields-that-can-be-extracted

Click on code:
https://github.com/interfasys/mediametadata/

@jancborchardt
Copy link
Member

We should discuss doing this in a central way again. There's several apps which could use it:

As pictures are a pretty core file type, this is not something we should simply brush off.

If showing photos in the Calendar for example doesn't need the full EXIF data though, we could say that specifically extracting the GPS data would be a task for the Maps app. But we should avoid some silly dependecy like a specific "EXIF extraction" app.

@jancborchardt jancborchardt reopened this Aug 29, 2017
@oparoz
Copy link
Member

oparoz commented Aug 30, 2017

I'm happy to move the app to Nextcloud if it's deemed as a good enough starting point. Lots of work was done to write specs to make sure it's useful, but I understand that the server is also collecting some EXIF data already.

@Spartachetto
Copy link

Spartachetto commented Feb 7, 2018

Some notes:

  • 13 was released today!
  • an app to view metadata does exist ( https://github.com/gino0631/nextcloud-metadata ). It allows to view the metadata in the sidebar
  • the existing app does not allow the advanced functionalities @jancborchardt was describing (e.g.: showing the photo on a map). Basically you cannot use the data
  • several metadata types are missing up to now from the app (e.g. bibliographic metadata)

@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@Tyler-2
Copy link

Tyler-2 commented Jan 15, 2019

This was brought up as a pre-requisite for sorting by EXIF 'taken' date, which someone has developed and submitted a PR for. And yet this portion has gone stale.

Is this anywhere on the roadmap? Viewing a collection of photos without having the ability to sort by when they were taken has both professional and family sharing impacts.

@nextcloud-bot nextcloud-bot removed the stale Ticket or PR with no recent activity label Jan 15, 2019
@jancborchardt
Copy link
Member

Here’s some notes I made for the last cycle on Gallery sidebar, EXIF data etc etc. Maybe it’s helpful:

cc @MorrisJobke @skjnldsv for this cycle.

@jancborchardt jancborchardt changed the title EXIF information for 13 EXIF information Apr 1, 2019
@MorrisJobke MorrisJobke self-assigned this Apr 2, 2019
@MorrisJobke MorrisJobke added this to the Nextcloud 17 milestone Apr 2, 2019
@stale

This comment has been minimized.

@stale stale bot added the stale Ticket or PR with no recent activity label Jun 5, 2019
@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of and removed stale Ticket or PR with no recent activity labels Jun 5, 2019
@hairlesshobo
Copy link

Was just wanting to see if there is any update on this request? I am very much looking forward to seeing the improvements that can be made in the mobile apps once this is implemented. Thanks!

@gino0631
Copy link

gino0631 commented Jan 8, 2020

If there is anything that can be done in the https://github.com/gino0631/nextcloud-metadata app regarding this, feel free to submit a feature request. Thanks!

@skjnldsv skjnldsv removed this from the Nextcloud 17.0.3 milestone Jan 9, 2020
@jancborchardt
Copy link
Member

It probably makes sense to integrate @gino0631’s Metadata app into the Photos app: https://github.com/nextcloud/photos

Right @skjnldsv @rullzer @karlitschek?

@karlitschek
Copy link
Member

I agree. This is a good suggestion. I'm currently unsure if it should be in the photos app or more standalone so that other apps like File or Search can also benefit from it.

@skjnldsv
Copy link
Member

skjnldsv commented Jan 14, 2020

I'm currently unsure if it should be in the photos app or more standalone so that other apps like File or Search can also benefit from it.

I'm guessing it would offer us more flexibility as a standalone app :)
What do you think about this @gino0631

@jancborchardt
Copy link
Member

It is already a stand-alone app in the store :) https://apps.nextcloud.com/apps/metadata

What we could do is integrate it better like via a "Details" tab in the sidebar rather than adding yet another specific tab for an app?

@szaimen
Copy link
Contributor

szaimen commented Jan 14, 2020

It is already a stand-alone app in the store :) https://apps.nextcloud.com/apps/metadata

What we could do is integrate it better like via a "Details" tab in the sidebar rather than adding yet another specific tab for an app?

You could make the app official so that it is installed by default.

@Spartachetto
Copy link

Spartachetto commented Jan 14, 2020

I understood that the real issue was to integrate more the EXIF information so that other apps could use the data...

The apps that could use the data are at least:

  • Files (display info and also order files by metadata)
  • Search
  • Photo
  • Maps
  • Calendar
  • ...

In addition the android client developers were interested in this issue to use the metadata in the client ( see nextcloud/android#4501 )

@gino0631
Copy link

Maybe the Metadata App should provide an API that could by used by other apps?

@skjnldsv
Copy link
Member

skjnldsv commented Jan 15, 2020

Maybe the Metadata App should provide an API that could by used by other apps?
@gino0631

Is the app making the properties available as dav attributes?

What do you think about only showing the tab in the photos app sidebar ? 😉

@gino0631
Copy link

Is the app making the properties available as dav attributes?

Currently, no, but the idea is quite interesting.

What do you think about only showing the tab in the photos app sidebar ? 😉

Not sure if I understand - no tab in Files you mean?

@skjnldsv
Copy link
Member

Not sure if I understand - no tab in Files you mean?

yes
Only in the new gallery

@gino0631
Copy link

The app is not just for pictures.

@jancborchardt
Copy link
Member

Just an adjustment of the last sentence @skjnldsv said: We would show the sidebar tab, but we need a general way for apps to not add the xth tab, but we should have one "Details" tab where the Metadata app can hook into. :)

As the Metadata app would be the first to hook into this, de facto it would be the same as now but properly done. A first step would just be if the metadata app renames the tab title to "Details"?

@gino0631
Copy link

General way for apps to provide additional details sounds good, but I can't see how renaming the tab helps to implement that.

@skjnldsv
Copy link
Member

Closing as this is not related to server in the end.
There is already a talk in photos here: nextcloud/photos#226

@skjnldsv
Copy link
Member

The issue is not closed because it's abandoned, but because now that photos is here, we have a wider discussion (like we had on gallery), we definitely want EXIF implementation on nextcloud, do not worry :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of enhancement
Projects
None yet
Development

No branches or pull requests