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

What happened to image metadata? #791

Open
ljchang opened this issue May 8, 2023 · 6 comments
Open

What happened to image metadata? #791

ljchang opened this issue May 8, 2023 · 6 comments

Comments

@ljchang
Copy link

ljchang commented May 8, 2023

Many years back we had helped add image metadata functionality to neurovault to support a number of other projects in the ecosystem including neurolearn and nltools. This included the ability to upload and download image metadata and edit via the GUI. We have discovered that this functionality appears to now be broken.

The edit image metadata option on a collection is returning a 500 error.
Screenshot 2023-05-08 at 1 14 25 PM

Our ability to download image metadata via pynv in nltools no longer returns any of the image metadata, only the collection meta data.

Just wanted to start a discussion about this to see if this functionality has recently been refactored and if so, where we can find documentation about the new API. Alternatively, perhaps this was just an oversight in some new refactor that we could help contribute to fixing.

There are a number of datasets available on neurovault that people have been relying that contain this image metadata, so hopefully it is still available and not completely disappeared.

All of our tests for nltools (currently maintained by @ejolly and myself) rely on downloading images from neurovault to test the functionality of various image analyses, so it would be great if we could help get this working again quickly.

Thanks!

@ejolly
Copy link

ejolly commented May 8, 2023

As an example:

All images in collection 504 previously used to have the following fields but are now missing: 'Age', 'PainLevel', 'SubjectID', and 'Sex'

@62442katieb
Copy link

Related, in trying to edit images' meta-data today, I was greeted with this (same on Firefox v113.0.1, Chrome v113.0.5..., & Safari v15.2):
Screen Shot 2023-05-26 at 12 40 59 PM

@ejolly
Copy link

ejolly commented Aug 14, 2023

@rwblair Just wanted to bring this to your attention as it might be related to #803 i.e. missing data? Do you have any sense of whether/when editing + getting image meta-data functionality might be restored to Neurovault? It was pretty critical functionality to our toolbox which also supports general neurovault operations. Apologies for being pushy, but we're held up cutting a new release of our toolbox and need to make a call about abandoning neurovault if it doesn't seem like this issue will get addressed any time soon!

@rwblair
Copy link
Member

rwblair commented Aug 15, 2023

@ejolly Taking a look into it now, will let you know what I find out.

@rwblair
Copy link
Member

rwblair commented Aug 15, 2023

Found three issues. The css for the table library was missing. The metadata field in the database not having migrated from the hstore type to jsonb type. The javascript that triggered the csv download had a missing function being called.

Manually editing and downloading image data works for me now. @ejolly Let me know if it works for you.

@rwblair
Copy link
Member

rwblair commented Aug 15, 2023

pynv may need to cut a new release to handle changes to api authentication.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants