search

kmenschig / Weather

Weather Data from Cologne, Germany
1 stars 0 forks source link

cronjob doesn't consider timezones #1

Closed mmenschig closed 6 years ago

mmenschig commented 7 years ago

Currently, the script is set to run via cronjob every 1pm GMT+1 (can that be confirmed?)

Problem I see is when executing the script, other locations such as PWS:IMANIZAL5 will have their data retrieved at night, possibly returning unforeseen / not considered weather data such as lower temperatures etc.

The question therefore is:

  1. Can we confirm that the cronjob does not need to be adjusted?
  2. Does this require fixing?
kmenschig commented 7 years ago

(env) klaus@klaus-Precision-M6800:~/Documents/Projects/Weather$ git push warning: push.default is unset; its implicit value has changed in Git 2.0 from 'matching' to 'simple'. To squelch this message and maintain the traditional behavior, use:

git config --global push.default matching

To squelch this message and adopt the new behavior now, use:

git config --global push.default simple

When push.default is set to 'matching', git will push local branches to the remote branches that already exist with the same name.

Since Git 2.0, Git defaults to the more conservative 'simple' behavior, which only pushes the current branch to the corresponding remote branch that 'git pull' uses to update the current branch.

See 'git help config' and search for 'push.default' for further information. (the 'simple' mode was introduced in Git 1.7.11. Use the similar mode 'current' instead of 'simple' if you sometimes use older versions of Git)

Username for 'https://github.com': kmenschig Password for 'https://kmenschig@github.com': To https://github.com/kmenschig/cologneWeather.git ! [rejected] master -> master (fetch first) error: failed to push some refs to ' https://github.com/kmenschig/cologneWeather.git' hint: Updates were rejected because the remote contains work that you do hint: not have locally. This is usually caused by another repository pushing hint: to the same ref. You may want to first integrate the remote changes hint: (e.g., 'git pull ...') before pushing again. hint: See the 'Note about fast-forwards' in 'git push --help' for details.

On Wed, Oct 4, 2017 at 10:37 PM, Marian Menschig notifications@github.com wrote:

Assigned #1 https://github.com/kmenschig/Weather/issues/1 to @kmenschig https://github.com/kmenschig.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/kmenschig/Weather/issues/1#event-1279171449, or mute the thread https://github.com/notifications/unsubscribe-auth/AcqCrICJi7ptRYkIf9wdTBLzZl2gMDiYks5spE8BgaJpZM4PuhWl .

mmenschig commented 6 years ago

Closing this. Seems like this is resolved.