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

location based switching #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

Feo-d-oR
Copy link

Wallpaper changes based on geographical location.

If set and enabled, wallpapers will change in respect of geoposition.
Looks more natural and closer to reality, especially in Northern places
like Saint-Petersburg or Helsinki...

Also, main script was changed to be more flexible in xfce wallpaper
channels querying - now its queried from system rather than hardcoded.

The only added dependency - pyethem that exists in every modern python

Wallpaper changes based on geographical location.

If set and enabled, wallpapers will change in respect of geoposition.
Looks more natural and closer to reality, especially in Northern places
like Saint-Petersburg or Helsinki...

Also, main script was changed to be more flexible in xfce wallpaper
channels querying - now its queried from system rather than hardcoded.

The only added dependency - pyethem that exists in every modern python
@Feo-d-oR
Copy link
Author

Little errors in day part calculation detected. I'll fix them in next commit plus some comments will be added.

Wallpaper changes based on geographical location.

If set and enabled, wallpapers will change in respect of geoposition.
Looks more natural and closer to reality, especially in Northern places
like Saint-Petersburg or Helsinki...

Also, main script was changed to be more flexible in xfce wallpaper
channels querying - now its queried from system rather than hardcoded.

The only added dependency - pyethem that exists in every modern python

Added necessary comments in change_theme and removed some stdout output
in wallpaper.py in thins commit
@Feo-d-oR
Copy link
Author

Corrected day part decision part, everything seems to work as expected for now...

Found a side-effect of moving to subprocess rather than os.system
resulting in set_enable failed to xfconf-query did not recreated
wallpaper paths for founs channels. This issue is fixed as per now.
If default switch time is selected, then pictures will change at
predefined times: 06:00, 08:00, 10:00, 16:00, 18:00, 19:00, 20:00, 21:00

Also, the form has been changed a bit: label for Latitude and
Longtitude, 'Default location' was renamed to 'Default switch'.
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

Successfully merging this pull request may close these issues.

1 participant