-
Notifications
You must be signed in to change notification settings - Fork 2
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
VR180 file format & VR toggle support? #1
Comments
Actually the version 4 of the libary is still in beta state. So we won't manage to integrate it at the moment. |
Any news on this? Looks like v4 is stable now: https://photo-sphere-viewer.js.org/guide/migration-v3.html |
Sounds interesting. Migrating towards the new viewer version shouldn't be too hard. But i don't have any VR180 data. If i upgrade the component would you test the new app version? |
Yes I have some VR180 images from the DaydreamVR camera that I can share as test material. |
Ok sounds good to me 👍 I added this to my ToDo list and will inform you on any news |
@poVoq i implemented the update. If you like you can test it by manually installing the app from https://github.com/nextcloud/files_photospheres/suites/2770953847/artifacts/61382672 (first unzip and then untar the Would be happy to get some feedback 👍 |
@poVoq i have some news: seems like your sample can be correctly shown inside the app. The problem in your sample is, that it's missing the mandatory Is this file directly copied from your camera? If yes regarding to the specs the metadata is incomplete. You can inspect the metadata on your own if you just open the file in a regular text-editor like You can download the current testing version for this branch via https://github.com/nextcloud/files_photospheres/suites/2875375155/artifacts/64279786 Would be glad to get some feedback :-) |
Yes strait from the camera, and I now double checked if Github might have stripped some metadata, which doesn't seem to be the case. This is quite odd, as the Lenovo Daydream camera is pretty much the reference implementation for the VR180 format. I don't think any other such cameras were ever produced. |
Unfortunately i cannot find any specs for this format. Nevertheless we'd need a clear decision if the |
This seems to have some details on the format: https://github.com/google/spatial-media/blob/master/docs/vr180.md |
I did some research and regarding to the specs i found here, the The current version can be downloaded here: https://github.com/nextcloud/files_photospheres/suites/2880576389/artifacts/64428192 . For me it works with your sample and some other regular photosphere images i have in my personal store. Let me know you results :-) |
Is there anything new related to the VR switch? I would be happy if I could view 360-degree images directly in the NextCloud with my smartphone and Cardboard. That should be possible with the new version of photo-sphere. |
Well i prepared the update to version 4 of photosphere viewer js. Could you imagine testing the new version? |
Many thanks for the quick response. I would like to test the new version. But unfortunately I can't manage to install it in my test system.
The directory is copied and visible.
As soon as I call up the frontend again, I get the error that the apps need to be updated. After that I have the "old" version of the viewer again. I tried to set the dependency to max-version="23" under /appinfo/info.xml. That without success, too. What to do, to test the new version? |
I created a quick package rebased on the current https://github.com/nextcloud/files_photospheres/suites/5178736840/artifacts/157252866 Hope this works, didn't have the time to test it unfortunately :-D |
Many thanks for the quick response. Now I was able to install the app and test it with many browsers. It works without any problems. |
The viewer is created at Line 28 in 911b1da
Currently there are no plugins involved but i could imagine to include some plugins hard-wired as "default" or to make them configurable, maybe? |
Unfortunately, I still haven't found the place or the code to activate the plugins. |
* Update to latest photosphere viewer * Add JS for gyroscope and stereo * Support for gyroscope and stereo (see #1 (comment))
I updated to the latest photosphere viewer version (4.6) and added the gyroscope and stereo plugins. You can download the app version at https://github.com/nextcloud/files_photospheres/suites/6058044292/artifacts/209044322 (compatible to NC23) Please note, that you'll have to serve your NC via Feedback would be really appreciated. |
@xausy did you already have the possibility to check this out? |
* Update to latest photosphere viewer * Add JS for gyroscope and stereo * Support for gyroscope and stereo (see #1 (comment))
* Update to latest photosphere viewer * Add JS for gyroscope and stereo * Support for gyroscope and stereo (see #1 (comment))
* Update to latest photosphere viewer * Add JS for gyroscope and stereo * Support for gyroscope and stereo (see #1 (comment))
* Update photo-sphere-viewer.js to v4 related to #1 * Add support for gyroscope and stereo * Update to latest photosphere viewer * Add JS for gyroscope and stereo * Support for gyroscope and stereo (see #1 (comment)) * Fix JSON testdata formatting * Fix TOC generator closing our PR
… (#89) * Update photo-sphere-viewer.js to v4 related to #1 * Add support for gyroscope and stereo * Update to latest photosphere viewer * Add JS for gyroscope and stereo * Support for gyroscope and stereo (see #1 (comment)) * Fix JSON testdata formatting * Fix TOC generator closing our PR * Fix php version compatibility
Great app, thanks for creating this.
Would be nice if the new VR180 format would be supported (as used by the Lenovo Daydream camera), and ideally also to have a VR toggle to look at VR and 360° photos that way.
It seems like the new v4 version of the library used is supporting some of this:
https://photo-sphere-viewer.js.org/v4/
Thanks for considering!
(NOTE:: this issue was moved from https://github.com/R0Wi/Nextcloud-PhotoSphereViewer/issues/1)
The text was updated successfully, but these errors were encountered: