python n00b vraag, object met variable hoe dan?

dinsdag 23 juli 2019 15:56

Acties:

0 Henk 'm!

Topicstarter

Beste mede Tweakers.

Ik zit vreselijk te tobben met een python script wat ik probeer te maken. Waarschijnlijk is het super simpel maar mijn gebrek aan kennis van python nekt me;)

Het script bestaat uit een config-file en het script zelf. De config file is zo opgebouwd:

Python:

database1 = {
    #Enable this database 
    'enabled': 'true',
    #location of the database we might add more than one
    'server' : 'localhost',
    #database port
    'port' : '8086',
    #databse username
    'username' : 'statistics',
    #database password 
    'password' : 'verwijderd',
    #database name 
    'dbname' : 'powerstatistics',
}
#can be used for remote storage
database2 = {
    #Enable this database 
    'enabled': 'false',
    #location of the database we migt add more than one
    'server' : '10.10.30.3',
    #database port
    'port' : '8086',
    #databse username
    'username' : 'remotepowerstatistics',
    #database password 
    'password' : 'verwijderd',
    #database name 
    'dbname' : 'powerstatistics',
}
# I thought about making tis in a array, but my pyhthon sikkls lacked;)
inverter = { 
    #IP sould be changed to match your inveter. Make shore it supports sunspec.
    'ipaddress': '"10.10.30.31"', 
    #Port should be left defauld.
    'port': '502', 
    #Only change if you know what you are doing
    'timeout': '2.0', 
    #Logical discription of the solar panels
    'logicalname': 'Barn',
    #Phase output (1 or 3) this will determine the report template
    'phase': '1'
}

probleem script

code:

#!/usr/bin/python2.7


# to do:
# extra off standard IP toevoegen en naar de PI rapporteren (dubbbel dus) als multi setup
# Unfortunatly pysunspec is build with python 2.7 so we are limitied in upgrades
# 
# 
# 

#importing modules

#Please change this line acording to your config!!
import configexample as config

#if modules are missing, please install via 'pip install $modulename'
from termcolor import colored
from influxdb import InfluxDBClient
import sunspec.core.client as client


# key component is pysunspec. This is an python bases program to communicatei wit the sunspec standard.
# more info about the pysunspec program: https://github.com/sunspec/pysunspec
# Solar Edge SunSpec: https://www.solaredge.com/sites/default/files/sunspec-implementation-technical-note.pdf


#we shoudl fix this
#setting up connection to Solar Edge inverter
#configtimeout = config.inverter['timeout']


#d = client.SunSpecClientDevice(client.TCP, 1, ipaddr="10.10.30.31", ipport=502, timeout=2.0)

d = client.SunSpecClientDevice(client.TCP, 1, ipaddr=config.inverter['ipaddress'], ipport=config.inverter['port'], timeout=config.inverter['timeout'])

#reading data from Solar Edge Inverter
d.common.read()
d.inverter.read()
d.close()

#date used in JSON report for Grafana
date = datetime.datetime.now().isoformat()

#Show data sets we retreived
print colored ('Common info about the inverter', 'green')
print d.common

print colored ('Detailed info about inverter','green')
print d.inverter


#this message is realy imported.
#if the wrong schama is selected info will be missed.
print colored ('----------------------------------', 'green')
print colored ('-        Data for Grafana        -', 'green')
print colored ('-   Please verify info above     -', 'green')
print colored ('-   with the selected 1 or 3     -', 'green')
print colored ('-   -phase inverter schema!      -', 'green')
print colored ('----------------------------------', 'green')

print 'Date (UTC):',date


# we should add a for loop to have multiple inverters supported in one site

#if we use a 1 phase inverter
if config.inverter['phase'] == '1':

    print colored ('----------------------------------', 'green')
    print colored ('- using 1 phase inverter schema  -', 'green')
    print colored ('----------------------------------', 'green')


    json_body = [
        {
            "measurement": "Ampere",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.A,
            }
        },
        {
            "measurement": "Ampere phase A",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.AphA,
            }
        },
        {
            "measurement": "Voltage phase A",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.PPVphAB,
            }
        },
        {
            "measurement": "Watt",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.W,
            }
        },
        {
            "measurement": "Hertz",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.Hz,
            }
        },
        {
            "measurement": "Volt Ampere",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.VA,
            }
        },
        {
            "measurement": "Volt Ampere Reactive",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.VAr
            }
        },
        {
            "measurement": "Power Factur",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.PF,
            }
        },
        {
            "measurement": "Watt Hour (Total)",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.WH,
            }
        },
        {
            "measurement": "DC Ampere",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.DCA,
            }
        },
        {
            "measurement": "DC Volt",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.DCV,
            }
        },
        {
            "measurement": "DC Watt",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.DCW,
            }
        },
        {
            "measurement": "Temperature Headsink",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.TmpSnk,
            }
        },
        {
            "measurement": "Status Vandor Code",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.StVnd,
            }
        },
    ]

#if we use a 3 phase inverter -> needs a check, never tested!
if config.inverter['phase'] == '3':

    print colored ('----------------------------------', 'green')
    print colored ('- using 3 phase inverter schema  -', 'green')
    print colored ('----------------------------------', 'green')


    json_body = [
        {
            "measurement": "Ampere",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.A,
            }
        },
        {
            "measurement": "Ampere phase A",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.AphA,
            }
        },
        {
            "measurement": "Ampere phase B",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.AphB,
            }
        },
        {
            "measurement": "Ampere phase C",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.AphC,
            }
        },
        {
            "measurement": "Volt phase A",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.PhVphA,
            }
        },
           {
            "measurement": "Volt phase B",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.PhVphB,
            }
        },
           {
            "measurement": "Volt phase C",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.PhVphC,
            }
        },
        {
            "measurement": "Watt",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.W,
            }
        },
        {
            "measurement": "Hertz",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.Hz,
            }
        },
        {
            "measurement": "Volt Ampere",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.VA,
            }
        },
        {
            "measurement": "Volt Ampere Reactive",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.VAr
            }
        },
        {
            "measurement": "Power Factur",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.PF,
            }
        },
        {
            "measurement": "Watt Hour (Total)",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.WH,
            }
        },
        {
            "measurement": "DC Ampere",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.DCA,
            }
        },
        {
            "measurement": "DC Volt",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.DCV,
            }
        },
        {
            "measurement": "DC Watt",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.DCW,
            }
        },
        {
            "measurement": "Temperature Headsink",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.TmpSnk,
            }
        },
        {
            "measurement": "Status Vandor Code",
            "tags": {
                "Host": d.common.SN,
                "logicalname": config.inverter['logicalname'],
            },
            "time": date,
            "fields": {
                "value": d.inverter.StVnd,
            }
        },
    ]

#Write to databse 1 if enabled
if config.database1['enabled'] == 'true':
    print colored ('----------------------------------', 'green')
    print colored ('-      send data to databse 1    -', 'green')
    print colored ('----------------------------------', 'green')

    client1 = InfluxDBClient(host=config.database1['server'], port=config.database1['port'], username=config.database1['username'], password=config.database1['password'])
    client1.switch_database(config.database1['dbname'])
    client1.write_points(json_body)
else: 
    print colored ('----------------------------------', 'green')
    print colored ('-      skipping  databse 1       -', 'green')
    print colored ('----------------------------------', 'green')

#Write to databse 2 if enabled
if config.database2['enabled'] == 'true':

    print colored ('----------------------------------', 'green')
    print colored ('-      send data to databse 2    -', 'green')
    print colored ('----------------------------------', 'green')

    client2 = InfluxDBClient(host=config.database2['server'], port=config.database2['port'], username=config.database2['username'], password=config.database2['password'])
    client2.switch_database(config.database2['dbname'])
    client2.write_points(json_body)

else: 
    print colored ('----------------------------------', 'green')
    print colored ('-      skipping  databse 2       -', 'green')
    print colored ('----------------------------------', 'green')


print colored ('----------------------------------', 'green')
print colored ('-            Done                -', 'green')
print colored ('----------------------------------', 'green')

error output

code:

poweradmin@powerstatistics:~/powerstatistics$ ./sunspecread.py
Traceback (most recent call last):
  File "./sunspecread.py", line 34, in <module>
    d = client.SunSpecClientDevice(client.TCP, 1, ipaddr=config.inverter['ipaddress'], ipport=config.inverter['port'], timeout=config.inverter['timeout'])
  File "/usr/local/lib/python2.7/dist-packages/sunspec/core/client.py", line 775, in __init__
    self.device.scan(progress=scan_progress, delay=scan_delay)
  File "/usr/local/lib/python2.7/dist-packages/sunspec/core/client.py", line 218, in scan
    self.modbus_device.connect()
  File "/usr/local/lib/python2.7/dist-packages/sunspec/core/modbus/client.py", line 674, in connect
    raise ModbusClientError('Connection error: %s' % str(e))
sunspec.core.modbus.client.ModbusClientError: Connection error: a float is required

in regel 40 ziet u wat wel werkt
in regel 42 ziet u wat niet werkt

volgnes mij komt het er op neer dat de variable van het 'ipaddress' & 'port' &
'timeout' niet worden door gegeven.

In plaatst daarvan struikelt de client.SunSpecClientDevice er over dat 'ipaddres' een string is (text, namelijk het woord ipadress) en niet een ip addres (10.10.30.31) in dit geval. zeflde geld voor de port en de time out.

Mijn vraag, hoe krijg ik python zover dat hij de variable doorgeeft aan de client.SunSpecClientDevice?

Alvast bedankt voor het mee denken!!!

[ Voor 98% gewijzigd door jawmoret op 23-07-2019 16:44 ]

Do the math: sin adds to your troubles, subtracts from your peace, multiplies your sorrows and divides you from God.

dinsdag 23 juli 2019 16:06

Acties:

+1 Henk 'm!

DataGhost

iPL dev

Zet je script eerst eens in [code] tags, en copy/paste de relevante errors letterlijk hier. Je post nu een eigen interpretatie van de error die volgens mij niet overeen komt met wat er bedoeld wordt, kijkend naar het beetje code wat je gepost hebt. Je hebt in ieder geval een fout in het ip-adres in je config-file zitten, dus dat kan je waarschijnlijk zelf al vinden en fixen, maar het posten van de error en de uitleg ervan door iemand anders zal je waarschijnlijk wat extra inzicht geven.

dinsdag 23 juli 2019 16:41

Acties:

0 Henk 'm!

Mink Z.

Komt het niet domweg doordat je een object aan maakt met de naam 'client1' in regel 5 en in regel 8 verder gaat met een object met de naam 'client'?

dinsdag 23 juli 2019 16:49

Acties:

0 Henk 'm!

DataGhost

iPL dev

Mink Z. schreef op dinsdag 23 juli 2019 @ 16:41:
Komt het niet domweg doordat je een object aan maakt met de naam 'client1' in regel 5 en in regel 8 verder gaat met een object met de naam 'client'?

Nee, daar wees de interpretatie van de foutmelding al helemaal niet op, en uit de context kan je ook wel bedenken dat het niet logisch is dat je een SunSpecClientDevice zou kunnen maken vanuit een InfluxDBClient. Sowieso zit die laatste niet in de base library van Python dus daar had je al kunnen vermoeden dat TS slechts snippets uit z'n code had gepost.

@jawmoret: top dat je dat had gedaan (alleen het relevante pasten), maar helaas heb je er nu met je edit een gigantisch script van gemaakt waarvan het grootste gedeelte niet relevant is. Wat er daarvoor stond was veel relevanter, hoewel er wat kleine (triviale) stukjes misten. Je hebt echter nog steeds de foutmelding niet gepastet, en nog niet gevonden wat er in je config fout is.

dinsdag 23 juli 2019 16:50

Acties:

0 Henk 'm!

jawmoret

Topicstarter

@DataGhost bedankt voor de tip.
dit is inderdaad overzichtelijker en het was inderdaad mijn interpetatie.. niet handig natuurlijk;)

Do the math: sin adds to your troubles, subtracts from your peace, multiplies your sorrows and divides you from God.

dinsdag 23 juli 2019 16:54

Acties:

0 Henk 'm!

DataGhost

iPL dev

jawmoret schreef op dinsdag 23 juli 2019 @ 16:50:
@DataGhost bedankt voor de tip.
dit is inderdaad overzichtelijker en het was inderdaad mijn interpetatie.. niet handig natuurlijk;)

De foutmelding die je krijgt betekent inderdaad niet wat je denkt, that is, het betekent in ieder geval niet dat het ip-adres de string "ipaddress" is, zoals je zei. Het betekent wel dat een ander "type" verwacht werd (en duidt op een extreem lelijke implementatie

maar als het werkt, werkt het he?). Kijk nog eens in je config en type (niet copy/pasten) de regel met het IP-adres anders opnieuw en kijk wat het verschil is.

dinsdag 23 juli 2019 16:55

Acties:

0 Henk 'm!

Room42

@jawmoret Ik denk dat je Timeout een float moet zijn (in plaats van string):

Python:

1	d = client.SunSpecClientDevice(client.TCP, 1, ipaddr=config.inverter['ipaddress'], ipport=config.inverter['port'], timeout=float(config.inverter['timeout']))

[ Voor 4% gewijzigd door Room42 op 23-07-2019 16:57 ]

"Technological advancements don't feel fun anymore because of the motivations behind so many of them." Bron

dinsdag 23 juli 2019 16:58

Acties:

+1 Henk 'm!

Z-Dragon

Room42 schreef op dinsdag 23 juli 2019 @ 16:55:
@jawmoret Ik denk dat je Timeout een float moet zijn:
Python:
1
d = client.SunSpecClientDevice(client.TCP, 1, ipaddr=config.inverter['ipaddress'], ipport=config.inverter['port'], timeout=float(config.inverter['timeout']))

Dat, en port een int. In plaats van te converteren bij de aanroep, kun je ook de config aanpassen (als dat niks anders sloopt) door simpelweg de quotes om de waarden heen weg te halen.

[ Voor 7% gewijzigd door Z-Dragon op 24-07-2019 04:37 ]

^ Wat hij zegt.

dinsdag 23 juli 2019 17:02

Acties:

+1 Henk 'm!

Reinier

\o/

Inveter? Make shore? Defauld?

Maak zulke spelfouten niet in je code, dat is vragen om problemen.

dinsdag 23 juli 2019 17:03

Acties:

0 Henk 'm!

jawmoret

Topicstarter

DataGhost schreef op dinsdag 23 juli 2019 @ 16:54:
[...]

De foutmelding die je krijgt betekent inderdaad niet wat je denkt, that is, het betekent in ieder geval niet dat het ip-adres de string "ipaddress" is, zoals je zei. Het betekent wel dat een ander "type" verwacht werd (en duidt op een extreem lelijke implementatie maar als het werkt, werkt het he?). Kijk nog eens in je config en type (niet copy/pasten) de regel met het IP-adres anders opnieuw en kijk wat het verschil is.

Ik denk dat je bedoeld het feit dat ik er quotes omheen had gezet dus '"10.10.30.31"' ip.v. '10.10.30.31' ?

je zei dat het een extreem lelijke implementatie is, bedoel je dat ik de data type moet definieren bij het laden van die variable? zoals voorgesteld door Evanescent met een float?

Do the math: sin adds to your troubles, subtracts from your peace, multiplies your sorrows and divides you from God.

dinsdag 23 juli 2019 17:06

Acties:

+1 Henk 'm!

DataGhost

iPL dev

jawmoret schreef op dinsdag 23 juli 2019 @ 17:03:
[...]

Ik denk dat je bedoeld het feit dat ik er quotes omheen had gezet dus '"10.10.30.31"' ip.v. '10.10.30.31' ?

je zei dat het een extreem lelijke implementatie is, bedoel je dat ik de data type moet definieren bij het laden van die variable? zoals voorgesteld door Evanescent met een float?

Ik was biased doordat ik jouw interpretatie van de foutmelding al had gelezen en daar dus begon met zoeken, en inderdaad die extra quotes in je IP-adres zag, terwijl de melding zelf eigenlijk helemaal niks over een IP zei. Ik ging ervan uit dat die klasse op de een of andere manier het IP-adres als float wilde hebben (hoe dat er dan in de praktijk uit zou zien kon ik even niet op komen). Dat het de timeout is zoals Evanescent zegt is veel logischer, en dat zoals Z-Dragon zegt de port een int moet zijn ook. Waarschijnlijk zitten er dus 3 fouten in je config

dinsdag 23 juli 2019 17:21

Acties:

+2 Henk 'm!

Xesxen

Je mist een ) op het einde van regel twee (van je laatste code paste), daarmee "vloeit" het commando door naar regel 5 (en resulteert het in een syntax error)

Rare vogel in spe

dinsdag 23 juli 2019 17:59

Acties:

+1 Henk 'm!

jawmoret

Topicstarter

heren mijn dank is groot, het is gelukt!

definitieve code:

code:

1	d = client.SunSpecClientDevice(client.TCP, 1, ipaddr=(config.inverter['ipaddress']), ipport=int(config.inverter['port']), timeout=float(config.inverter['timeout']))

@DataGhost heb je nog meer goede ideeen die ik zou kunnen implementeren om data integriteit te waarborgen?
Misschien een beetje een brede vraag maar alles waar ik van leren kan is welkom! $_/-\o_$

Do the math: sin adds to your troubles, subtracts from your peace, multiplies your sorrows and divides you from God.

donderdag 1 augustus 2019 01:14

Acties:

0 Henk 'm!

MueR

Admin Tweakers Discord

is niet lief

Doe mij (en mijn collegas) een lol.. De volgende keer: doe zelf wat debuggen, dump niet 500 regels code en plaats het in het juiste forum.

Anyone who gets in between me and my morning coffee should be insecure.

Vraag

Alle reacties