search

theothernt / AerialViews

A screensaver for Android TV devices including Nvidia Shield, Fire TV, and Chromecast with Google TV. Inspired by Apple TV's video screensaver.
GNU General Public License v3.0
469 stars 35 forks source link

Does the app support still images? #95

Closed jagjordi closed 3 months ago

curtisbarnard commented 7 months ago

I'm running TrueNAS and was able to get a 1080p video to play, but not all 1080p videos to play. I moved the video discussion to #128 since it doesn't seem related to the work being done for the images.

theothernt commented 7 months ago

maybe a function of the samba server? i'm pulling from a server running FreeNAS v. 11.1-U7.

That is possible but I think it's very unlikely, otherwise I would have seen more issues or crashes with different NAS devices.

(I use a Synology NAS btw)

EDIT: It looks like earlier me was wrong! Logs indicate a file info or file stream issue which could mean an issue with SMBJ or certain Samba configs.

theothernt commented 7 months ago

Version 1.7 has been released, I recommend updating if you're still running the beta...

https://github.com/theothernt/AerialViews/releases

Thanks to everyone who tested the feature and made suggestions.

brjhaverkamp commented 7 months ago

Ha, I tried to update, but my Android TV can't find the update. Will try every hour for the next 24 hours ;-) One small addition to your readme. For all linux adepts like me not be be feeling left out: I followed the windows instructions and it worked like a charm. Below my (anonymized) shell output:

bert@trout:~$ adb version Android Debug Bridge version 1.0.41 Version 33.0.3-debian Installed as /usr/lib/android-sdk/platform-tools/adb bert@trout:~$ adb connect

Kind regards,

Bert

theothernt commented 7 months ago

Ha, I tried to update, but my Android TV can't find the update. Will try every hour for the next 24 hours ;-)

If you're referring to the Play Store, then it won't be available for a few days. Typically, I release on a GitHub then wait about 5-7 days before posting it to the Play + Amazon Store.

One small addition to your readme. For all linux adepts like me not be be feeling left out: I followed the windows instructions and it worked like a charm. Below my (anonymized) shell output:

Thanks for that, I'll update the instructions to add a) Linux user support and b) add a bit about "authorising the connection" the first time you connect to a device with ADB.

brjhaverkamp commented 7 months ago

So you haven't posted it on the play store yet? That makes sense. I'll be patient. I thought I was looking at the delay between submitting and Googles verification. I read that can also take up to a few days.

theothernt commented 7 months ago

So you haven't posted it on the play store yet? That makes sense. I'll be patient. I thought I was looking at the delay between submitting and Googles verification. I read that can also take up to a few days.

Due to the limited amount of testing I can do with different devices, I stagger the roll out of updates.

Sometimes there is a beta for certain features. Then a stable release on GitHub. 5-7 days later on the Play Store - 20% rollout for 1 to 2 days then 100%.

Review times in the Play Store can vary. For the first year, it was 3-4 days I think. Now it's 1 hour or less.

EDIT: The v1.7 update is now live on the Play Store @brjhaverkamp

brjhaverkamp commented 6 months ago

Hello Neil,

I upgraded a few days ago, (thanks for the heads-up:-) ) And I am happy to report that the screensaver is working niceley! Huray and thank you soo much!

Few observations: First it took me a while to figure out that you have to select "Media type" under "Network". Images only/ video only/ videos &images. Wouldn't it be more logic to put that under "Appearance" ? It is not a source specific setting, right?

Secondly, I after I initially set aerialview as my screensaver, I couldn't figure out why it started almost immediately. Then I reread the instructions: To change the default timeout use this command with a value in milliseconds. So, 5 minutes is 30000, 10 minutes is 60000 and so on. I had read over it without thinking. But 30000ms is 30 seconds. And that was exactly what I was seeing :-)

Can you change this in the documentation? so: To change the default timeout use this command with a value in milliseconds. So, 5 minutes is 600000, 10 minutes is 1200000 and so on.

And then a feature request: I would like to see the folder name of the photo in the left bottom. As I mentioned earlier I have sorted my photos as in Photalbum/decade/year/topic/photoxxx.jpg " So for instance Photo album Bert/2020s/2023/2023-01-01 New years party" So the folder the photo or video is in is the topic name. If I could show that name as "title" that would be great. Or is this already possible somehow?

In all great work! Thanks for the great app.

Bert

theothernt commented 6 months ago

Few observations: First it took me a while to figure out that you have to select "Media type" under "Network". Images only/ video only/ videos &images. Wouldn't it be more logic to put that under "Appearance" ? It is not a source specific setting, right?

Currently, Aerial Views is marketed as a video screensaver, so that is the default. In future, on first run of the app, I will probably give the user the choice of videos, pictures or both.

Can you change this in the documentation? so: _To change the default timeout use this command with a value in milliseconds.

Thanks for catching that mistake, it's fixed now.

And then a feature request: I would like to see the folder name of the photo in the left bottom. As I mentioned earlier I have sorted my photos as in Photalbum/decade/year/topic/photoxxx.jpg " So for instance Photo album Bert/2020s/2023/2023-01-01 New years party" So the folder the photo or video is in is the topic name. If I could show that name as "title" that would be great. Or is this already possible somehow?

That might be possible but I would need a few more examples so see how that might be formatted.

brjhaverkamp commented 6 months ago

Hi Neil,

What examples do you mean? Hope this helps: As I mentioned the last level subfolder name is the topic of the set of photo in that folder. That foldername is a string of typically 10-30 characters. e.g. "Christmas 2024", "2023-12-05 St Nicolas" or "2021-02-18 Birthday" I normally put a date and a few words in the folder name. If it could be shown in the bottom of the image, that would be simple and effective. I've seen that in other screensavers. e.g. in Kodi.

Regards,

Bert

theothernt commented 6 months ago

That foldername is a string of typically 10-30 characters. e.g. "Christmas 2024", "2023-12-05 St Nicolas" or "2021-02-18 Birthday" I normally put a date and a few words in the folder name. If it could be shown in the bottom of the image, that would be simple and effective.

I had a quick look at the code for the Kodi screensaver you mentioned. If you have the following folders...

/photos/Christmas 2024/Photo.jpg /photos/2023-12-05 St Nicolas/Photo.jpg /photos/2021-02-18 Birthday/Photo.jpg

I think it will display...

Christmas 2024 / Photo 2023-12-05 St Nicolas / Photo 2021-02-18 Birthday / Photo

If that's the case, that is something I can add to Aerial Views.

brjhaverkamp commented 6 months ago

Hi Neil,

What screensaver in Kodi did you look at? I used the Picture Slideshow Screensaver add-on.
In this screensaver you can set (Under the "Addtional" tab) the Display image name. The options for this setting are:

From your example it looks you refer to Folder- and filename. I would set it to foldername. So I would only see "Christmas 2024" or "2023-12-05 St Nicolas"

Regards,

Bert

theothernt commented 6 months ago

What screensaver in Kodi did you look at? I used the Picture Slideshow Screensaver add-on. In this screensaver you can set (Under the "Addtional" tab) the Display image name.

I think that was it, I did only look at the code though.

  • Disabled
  • Use full path
  • Use folder- and filename
  • Use foldername
  • Use filename

I can implement all of those. I'm just not sure if it'll be in 1.7.1 or 1.7.2 yet.

brjhaverkamp commented 6 months ago

Hi Neil, That would be great!! I'm already enjoying the current version very much. So I'll wait (im)patiently :-) I just donated a small amount to your coffee fund for your efforts.

Regards,

Bert

brjhaverkamp commented 4 months ago

Hi Neil,

I just wanted to check if you are closer to implementing showing the the name or title, as we discussed a while back. Reason is that my kids mentioned it this evening when the screensaver was popping up. They remembered it from the previous osmc/kodi system. So now the whole family supports the request apparently :-)

Regards,

Bert

brjhaverkamp commented 4 months ago

Ah, forgot to mention: In the field of an "not so serious feature request" when I told the kids that we had already discussed it here, my youngest though it is a very good idea if you could delay the title by a few seconds , so everyone could first guess what the photo was about. As a quiz or game :-)

theothernt commented 4 months ago

I just wanted to check if you are closer to implementing showing the the name or title, as we discussed a while back.

I should have a beta version to try in a few days.

It's not likely I'll be able to add a 'delay' option to the photo/video descriptions any time soon - but I may be adding a feature to hide all overlays until a up/down button is pressed - that may help?

brjhaverkamp commented 4 months ago

Hi Neil,

A beta in a few days? That is great news! On the other comment. I was serious that it was not to be taken seriously :-). It was just a funny anecdote to to demonstrate how much the description would be appreciated! So please don't bother with that one (unless there is another request than the one my kids, I guess)

theothernt commented 4 months ago

A beta in a few days? That is great news!

Here is (link removed). A couple of features to be aware of...

  1. Settings > Appearance > Location / Description > New formatting options for photos, videos, etc
  2. Settings > Appearance > (scroll down) Gradients

EDIT: v1.7.1 is out.

brjhaverkamp commented 3 months ago

I just received my update via the play store. It works great. Thanks for creating the perfect screensaver for me :-)