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

[media_server] Some minor issues #69

Open
cdondrup opened this issue Mar 9, 2015 · 18 comments
Open

[media_server] Some minor issues #69

cdondrup opened this issue Mar 9, 2015 · 18 comments
Assignees

Comments

@cdondrup
Copy link
Member

cdondrup commented Mar 9, 2015

Some quick feedback after trying the web interface, copied from here: strands-project/aaf_deployment#20 (comment)

  • Having to click upload after choosing the file seems redundant. Just automatic upload after choosing it would be nice.
  • I can only select single files, being able to select multiple and then automatically upload them would be great.
  • I uploaded music but it doesn't show in the options for creating a music set.
  • Images are autorotated to landscape
  • downloading creates a file with the internal id instead of the filename. 54fd79cecd82ea7718a53014.mp4 is not a really speaking name.
@nilsbore
Copy link
Member

nilsbore commented Mar 9, 2015

I'm also having some troubles. I uploaded a few music files to the webpage. When I query for all Music sets using the python interface I get (u'aaf_walking_group', 8, ObjectId('54fd8eed71ad041b160dce7e')), which looks correct. However, when I want to get the files in that set via mc.get_set("Music/aaf_walking_group"), I get this error: bson.errors.InvalidId: Music/aaf_walking_group is not a valid ObjectId. Any ideas?

@cburbridge
Copy link
Member

You need to do mc.get_set("54fd8eed71ad041b160dce7e"), which is the id you found for that set. There are a lot of improvements that could be made to this clumsy API :-)

@nilsbore
Copy link
Member

nilsbore commented Mar 9, 2015

But if you look at the guide: https://github.com/strands-project/strands_ui/tree/hydro-devel/mongodb_media_server and the API it looks like you should be able to get it both ways?

@cburbridge
Copy link
Member

Sorry, I wrote that imagining that I would go back and add that call later but have not done yet. I'll add it to the list of improvments, but can't do it today...

@nilsbore
Copy link
Member

nilsbore commented Mar 9, 2015

Ok, thanks. I'm able to get what I want now.

@cdondrup
Copy link
Member Author

Are there in news on the issues I mentioned, @cburbridge? Still cannot create a music collection because it does not show me the music files I uploaded.

@nilsbore
Copy link
Member

@cdondrup It works for me. Does the files not show at all? I think you need to create a music set and when you click it you can see the files which are available to add.

@cdondrup
Copy link
Member Author

file_manager
music_set

@nilsbore
Copy link
Member

Hmm, weird... I get them there in the second screen. Maybe it's some weirdness with the filenames.

@cdondrup
Copy link
Member Author

Tried renaming and re-encoding as wave files... Nothing works. Problem is that I cannot test the system if there are no music files in the media collections. I'll try adding some manually for now.

@marc-hanheide
Copy link
Member

Just tried myself... I don't see music files when creating a playlist. @cburbridge

@cburbridge
Copy link
Member

Sorry missed this. How are you running/hosting the media_server ?

@marc-hanheide
Copy link
Member

Ah. Could that be it: http://lcas.lincoln.ac.uk/strands_media

@cburbridge
Copy link
Member

The problem is that the audio files are uploaded with content type "audio/mp3" but i only looked for "audio/mpeg". I think it depends on the browser uploading it.
Regardless, the media server should accept more types. I have patched it here: https://github.com/strands-project/strands_ui/tree/settype-patch-1 but can't test it now as am on a different machine.

@cburbridge
Copy link
Member

The way it is hosted causes a problem: when I do

raise web.seeother(web.ctx.env.get('HTTP_REFERER'))

to redirect back to the referring page when adding or removing an item from a set, the webserver does not respond with the page as requested. For example, the referer is /strands_media/something so I redeirect back to that page, but end up with /strands_media/strands_media/something. Somehow the webserver is changing the requested page or something @marc-hanheide?

@marc-hanheide
Copy link
Member

Oh. OK. Then forget this. Sorry.

@cdondrup
Copy link
Member Author

I created a PR for this as now works on my machine as well. Thank you!
There are still some other minor issues in the first post. So I will reopen until @cburbridge says that this will be changed or won't/

@cdondrup cdondrup reopened this Mar 23, 2015
@marc-hanheide
Copy link
Member

right... sorry

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

No branches or pull requests

4 participants