Cronjob doesn't work

[loes7815]

Hi!

I got two sensors connected to my pi and I like to log their status to a csv file.
I made the following code:

Code: Select all

#!/usr/bin/python
import csv, os, glob, time, datetime


tfile1 = open("/sys/bus/w1/devices/28-00000742260e/w1_slave")
text1 =tfile1.read()
tfile1.close()
secondline1 = text1.split("\n")[1]
temperaturedata1 = secondline1.split(" ")[9]
temperature1 = float(temperaturedata1[2:])
temperature1 = temperature1 / 1000

tfile2 = open("/sys/bus/w1/devices/28-0000074270de/w1_slave")
text2 =tfile2.read()
tfile2.close()
secondline2 = text2.split("\n")[1]
temperaturedata2 = secondline2.split(" ")[9]
temperature2 = float(temperaturedata2[2:])
temperature2 = temperature2 / 1000

with open("/home/pi/temp.csv", "a") as csvfile:
    out = csv.writer(csvfile, delimiter=",",
                    quotechar='|',quoting=csv.QUOTE_MINIMAL)
    date = datetime.datetime.now()
    out.writerow([date.strftime("%Y"),date.strftime("%m"),date.strftime("%d"),date.strftime("%H"),date.strftime("%M"), temperature1, temperature2])
    time.sleep(5) #wait 5seconds

I made a cronjob.
using:

Code: Select all

sudo crontab -e

I choose the second option (sudo)

In the file I wanted to add

Code: Select all

***** /usr/bin/python /home/pi/temperature_csv.py

but I get an error saying bad command.

Code: Select all

***** sudo /usr/bin/python /home/pi/temperature.csv.py

doesnt work either

in addition, how can i stop the loop to view the data using the terminal?


thank you!

[B.Goode]

how can i stop the loop to view the data using the terminal?
CTRL + Z only pauses the process.

CTRL + C

[loes7815]

Thx!
Do you also have a solution for my other problem?

[RaTTuS]

Thx!
Do you also have a solution for my other problem?

yes
don't edit your original post to make it completely different to what it was

sudo crontab -e
will edit the root crontab

Code: Select all

***** /usr/bin/python /home/pi/temperature_csv.py

will need to be

Code: Select all

* * * * * /usr/bin/python /home/pi/temperature_csv.py

but this is a good way of trashing your SDcard
if you are running permanently loop inside your program and collate data more sensibly

[loes7815]

I googled some more, and found this solution to my problem.
But it didn't work either, that's why I changed my original post.

But do you suggest to loop my program?
Because

Code: Select all

tfile1 = open("/sys/bus/w1/devices/28-00000742260e/w1_slave")
text1 =tfile1.read()
tfile1.close()
secondline1 = text1.split("\n")[1]
temperaturedata1 = secondline1.split(" ")[9]
temperature1 = float(temperaturedata1[2:])
temperature1 = temperature1 / 1000

Won't work if I make it a 'normal' loop, with 'While True' or 'For a in 1:'

[DirkS]

But do you suggest to loop my program?
Because

Code: Select all

tfile1 = open("/sys/bus/w1/devices/28-00000742260e/w1_slave")
text1 =tfile1.read()
tfile1.close()
secondline1 = text1.split("\n")[1]
temperaturedata1 = secondline1.split(" ")[9]
temperature1 = float(temperaturedata1[2:])
temperature1 = temperature1 / 1000

Won't work if I make it a 'normal' loop, with 'While True' or 'For a in 1:'

What exactly do you mean by that? Error messages?

[rpdom]

It works when I try it

Code: Select all

import time

while True:
    tfile1 = open("/sys/bus/w1/devices/28-00000742260e/w1_slave")
    text1 =tfile1.read()
    tfile1.close()
    secondline1 = text1.split("\n")[1]
    temperaturedata1 = secondline1.split(" ")[9]
    temperature1 = float(temperaturedata1[2:])
    temperature1 = temperature1 / 1000
    print temperature1
    sleep( 60 )

[loes7815]

thank you!
I think I made a typo, it's working now