Einstieg in Addon-Entwicklung

Adromir

Hallo liebe Entwickler,

Ich trage mich mit einer Addon- Idee (naja eigentlich sind es 3 Addons) herum und wollte nun Anfangen mich in die Addon-Entwicklung einzuarbeiten. Ich hab das schon so weit verstanden, hab aber noch ein paar Fragen, die ich gerne im Vorfeld klären wollte, ob die Ideen überhaupt realisierbar sind und ob ich mir ggf. zu viel vornehme.

1) Für meine Addon- Ideen bräuchte ich verschiedene Pythonlibraries, die so in Kodi nicht vorhanden sind. Da es relativ Universell einsetzbare Libraries sind, war mein Gedanke, diese in separate Libraries zu packen, welche dann als Abhängigkeiten für die eigentlichen Addons fungieren. Dies scheint mir der bessere Weg zu sein, da ich diesen Teil des Addons dann leichter pflegen kann bzw. es dann auch Aufwand/Code spart falls mehre meiner Addons auf die gleichen libraries zurück greifen.

2) So ganz habe ich noch nicht verstanden, wie ich eine python library in ein Addon umwandeln kann. Gibt es vielleicht ein gutes Addon an dessen Quelltext man das dann verstehen kann? Also auch, wie diese Library dann in ein anderes Addon importiert werden kann.

3) Ein Addon würde eine Binary benötigen. Diese müsste ja dann für jede Prozessorarchitektur vorliegen. Ich habe gesehen, daß man Binary Addons mithilfe des Kodi build roots compilieren kann. Aber so ganz bin ich noch nicht dahinter gestiegen, wie das dann genau geht, diese z.B. für arm, aarch64, x86_64 und windows zu kompilieren. Oder macht es sinn, diese Binäry "klassisch" zu kompilieren und dann ins Plugin einzufügen, wo dann je nach vorhandener Prozessorarchitektur die entsprechende Binary ausgewählt wird? Da müsste ich mich dann noch ein bischen ins Thema Crosscompiling einlesen, da ich noch nicht ganz sicher bin, wie ich meinen Build- Server dazu einrichten muss.

4) Ist es prinzipiel Möglich, seine Eigenen "Medientypen" zu kreieren und diese im Addon zu laden und apzuspielen? Ich habe da was im Sinn, was über den regulären Player nicht möglich ist (zumindestens habe ich da nix in den entsprechenden Dokus gefunden). Diese "Medien" würden aus einer Zip-Datei bestehen in der mehrere Audiofiles gespeichert sind und eine Steuerungsdatei für diese. Ach ja, da stellt sich mir die Frage, ob es überhaupt möglich ist, dateien über Python am audio player "vorbei" abzuspielen?

ich bedanke mich herzlich
Adromir

don

Da fragen wir mal den @BJ1 und den @L0RE

Bei Punkt 4 bin ich mir ziemlich sicher, dass man Audiodaten auch anders abspielen kann.
Es gib ein Addon für die Squeezeboxen, welches einen eigenen Player mitbringt.
- XSqueeze

root2

Ich schmeiße mal das offizielle Wiki ins Rennen, das behandelt ebenfalls das Thema ziemlich ausführlich: http://kodi.wiki/view/Add-on_development

Inklusive Tutorials, Tools und Links auf viele Addons inklusive Quellcode.

Vor allem das Thema "Addon Rules" sollte beachtet werden, wenn es ein offiziell supportetes Addon sein soll.

PvD

Bei den Modulen kannst Du mal hier schauen, ob da nicht doch was dabei ist, was Du in deinen Addons verwenden kannst. Ansonsten sollte es schwierig sein, Deine Module ins offizielle Repo zu kriegen. Bleibt der Weg über

ein eigenes Repo und ins Addon einbinden per Dependency
Implementierung der/des Module in der Addonstruktur z.B. unter <meinAddon>/resources/lib/<meinModule>

Binary Bestandteile sind - soweit sie nicht aus dem eigenen Stall (kodi.tv) kommen - nicht erlaubt.

L0RE

Zu 1-2,
Ich würde beim Addon erst mal SImple beginngnen DIe Module in dein addon legen und laden.
Ansonsten beim Twitter Plugin habe ich es so gemacht,wenn du wo nachschauen willst.

3) kann warscheinich am besten @vdr.tuxnet was sagen

4) Teoretisch ja. Kodi17 Nutzt Inputstream als Abspielmethode. Die wiederum erweiterbar ist um neue Methoden

Du must aber dran denken wenn das jemand Nutzen will, relativ Complex ist, das zum Laufen zubringen, und hast viel Fehleranfälligkeit Wenn es nur darum geht ein Container format (zip zu nutzen) wäre es warscheinlich besser das Zipfile auszupacken und die Stuerinfos im Plugin auszuerten

Da hoerts aber dan bei mir auf vom Know hoff

Adromir

Erstmal herzlichen Dank für eure Antworten!

Um mal etwas konkreter zu werden.

Ich möchte ein Plugin entwickeln, was etwas ähnliches macht, wie diese Seite: https://www.ambient-mixer.com/

Kurz erklärt: Auf dieser Seite werden Ambiente Sounds gesammelt/gemixt, die bestimmte Szenen/Atmospären nachbilden.. Also z.B. ein Sturm oder in einem Wald.
Da der Player dort per Flash realisiert ist, gibt es warhscheinlich(?) keine Möglichkeit das in Kodi direkt einzubinden. Also war meine Idee, einen solchen "Mixer" selbst zu kreieren. Es wäre natürlich am einfachsten, einfach solche Sounds aufzunehmen und in eine Sounddatei abzumischen, aber da würde meiner Meinung nach einiges verloren gehen.

Ich habe mich da schon etwas amgesehen, was es für Möglichkeiten gibt, über Python Sound auszugeben und bin da auf den Mixer von pygame gestoßen, der anscheinend all das bietet, was ich mir für den Mixer wünsche:
1) Sounds müssen spezifisch an einen Audiokanal ausgegeben werden können
2) Eine Soundüberlagerung muss möglich sein (also Kanal Links spielt standardmäßig Sound A ab, zeitweise Kommt sound B dazu, wobei Sound A nicht gestoppt wird)

Deswegen war meine Idee, pygame in ein eigenes addon zu packen, dies benötigt aber SDL als binaries zum Funktionieren. Das sollte soweit aber möglich sein, da ich schon zwei Addons gesehen habe für den rpi, die das machen.

Die Zips würden dann gebraucht um ein Paket zu liefern, mit den entsprechenden Sounds und einer Steuerungsdatei, die dem Mixer sagt, welcher Sound wann, wie abgespielt werden soll.

L0RE

@Adromir
1. Seite Laden und die Verschiedenen Genres von der Seite Laden,unda nzeigen
2. Quell Code Des Seleckierten Gere Laden
3.Suche nach der ID
Im Code
soundTemplate : '2808', --> 2808 ist dann die ID
4. https://xml.ambient-mixer.com/audio-template…l5&id_template=[ID HIER EINFÜGEN]
So kommst du an die Lsite der MP3 die bei den Mixer Benutzt werden
6. Seite einblenden die man die Lautstärke regeln (Warscheinlcih muss mit Fenster gearbeitet werden damit die Schiebe Reglern gearbeitet werden kann)
https://github.com/jiaaro/pydub gibts nen Plugin mit dem (Wenn ffmpg instaliert ist es möglich ist N mp3 zu einem zusammen zu fügen
Beispiel
https://stackoverflow.com/questions/4039…her-with-python

Danach das Gemixte Mp3 speichern und dann Local abspielen

So ungefähr müsste sich "Relativ leicht machen lassen" Von der teorie gehen. Machbar wäre auch das zusammenführen mit ffmpg, und man muss in den Settings ffmpg Pfad angeben

Erweiterung mit greppen wie das speichern auf der Webseite der Werte geht, und das Ladne und Speichern mit von der Seite Nutzen.

Jatzt müßtest du es nur noch Coden

So wäre mein Vorschlag

Adromir

Danke @lore. Das hilft mir schon mal ne menge weiter.
ich muss mir das mal genau überlegen, welchen weg ich gehe. Das mit pydub scheint einfach zu sein, hat aber anscheinend den Nachteil, daß dabei die "dynamik" verloren geht. Das interessante an pygame.mixer ist für mich, daß man (nach meiner Einschätzung) eine größere Immersion erreichen kann gerade bei Abspielsystemen die mehr als 2 Kanäle bieten. Also zum Beispiel Sound A (Regen) als "Hintergrund" auf allen Kanälen dauerhaft loopen und dann ein Donnergeräusch per Zufall immer wieder abspielen und ggf nach dem Zufallsprinzip den Kanal wechseln lassen, so daß man den Eindruck eines sich Bewegenden Gewitters/Sturm bekommt.

Aber vielleicht versuche ich deinen Weg erstmal um ein gefühl für die Addonentwicklung zu bekommen, da es mir leichter scheint.

Adromir

Hallo liebe Addon-Entwickler,
ich hab jetzt schon ein wenig rumgestöbert und mich eingelesen, und hab so ein paar Ideen entwickelt, so daß ich noch mal Nachfragen muss, bevor ich mich gegebenfalls total verrene, weil ich einen Weg einschlage, der überhaupt nicht gangbar ist.

Basierend auf dem Addon (Kodisplay) gehe ich mal davon aus, daß es prinzipiel Möglich ist , SDL und pygame für mein Addon zu nutzen. Ich hab schon einen Ansatz angefangen, der auf diesem Addon basiert.

Nun aber meine Frage(n).

Ich würde SDL und Pygame gerne auch für andere Plugins verfügbar machen und mir sind diese Wege in den Sinn gekommen, von denen ich aber nicht sicher bin ob sie überhaupt möglich sind:

1. als library/addon modul
Geht das mit den Binaries? Können die dann von anderen Addons genutzt werden, gibt es ggf Pfadprobleme?
Ansatz wäre der wie bei Kodisplay, daß die binaries in einem Ordner des Addons liegen.

2. als binary addon

3. Mittels eines Installerscriptes wie bei libwidevinecdm.so

Sollten alle 3 mit nein beantwortet sein, dann werde ich das wohl nur in mein Addon und nicht als "library/modul" implementieren

L0RE

Also es geht
1. Must du die Library für jede Platform zu bauen. AM besten entsprechend Cross Compilern (da kann ich dir nicht helfen).
Dann solltest du ein Python Stump zu der Labray mitbauen und dies ale eigens Script addon erzeugen (Pro Platform jeweils das Selbe Paket )

Wie man ein Python Modul als eigenes Modul macht siehst du z.b.
https://github.com/kodinerds/repo…ipt.module.past

Da Binary Addons momentan im Offziellen Repo nicht erlaubt sind braust du pro Platform ein Eigenes Repository http://kodi.wiki/view/Add-on_repositories
Dann machst du ein Haupt Repository in dem du deine Unter repository(Repo WIndows,Rsberry, Linux32, Linux64bt,...) einbindest.....
Nun kann der User das Platform Abhängige Repo installieren.
Wenn du jetzt im Hauptattfon ein Addon hast das das Library Nutzt t z.b. (https://github.com/kodinerds/repo…rfeed/addon.xml) da nutze ich das mit zusazModul kannst du mit import ... die Python Funktion ansprechen

Da das dafür Prediestiniert ist das man nie Fertig wird, Würde ich es erst mal so
1. zum laufen bekommen, und alles Hardkodiiert
2. Die Library den Leute von Hand installieren lassen, Pfad in den Settings setzen
Und dann würde ich nach nach versuchen auf die Oben genannte Lösung zu kommen

Aber nicht alles auf einmal versuchen

Adromir

Danke, das hilft mir schon mal weiter, daß es prinzipiel hilft.

Muss ich die library wirklich selber kompilieren? Ich habe Debian Pakete für die von mir gewollten Plattformarchitekturen gefunden und hab die Library daraus extrahiert. Ist das so gangbar?

Zudem, da ich ein wenig Faul bin, und nicht "3 verschiedene" verschiedene Addons hatte ich einen Ansatz ähnlich in KoDisplay überlegt. In dem liegt die library als Zip-Datei vor und wird vom addon in die entsprechenden Addons entpackt. Meine Idee war für jede Architektur eine zip hineinzulegen und dieses dann dementsprechend entpacken zu lassen

harryberlin

hallo zusammen,

ich hoffe ich kann mein anliegen hier platzieren.
für mein addon habe ich ein zusatz modul erzeugt mit der source von pyserial.
lies sich auch installieren.
jedoch bekomme ich von mein addon trotzdem den fehler, dass serial nicht importiert werden kann.
muss da noch was zusätzlich eingestellt werden, vllt. in meiner addon.xml?

edit:
wie vermutet, ich muss in die addon.xml auch das modul importieren.

Code

<requires>
<import addon="xbmc.python" version="2.1.0" />
<import addon="script.module.pyserial" version="3.4" />
</requires>

Adromir

Ohne dein pyserial Addon zu kennen.. Abert kann es sein, daß die zu importierenden Dateien in einem Unterordner liegen, den du als libraryordner definiert hat? Bei BeautifulSoup 4 muss man z.B.

Code

import BeautifulSoup from bs4

benutzen, da die benötigten Dateien im Unterordner bs4 liegen

harryberlin

das hast du nicht ganz richtig verstanden.

da muss ich eine schritt zurück gehen. ich habe ein addon plugin.script.ibuscommunicator.
in dem habe ich das modul pyserial aktuell im ordner lib drin.
nun wollte ich dies aber gern extern haben.
also nun der weg über das modul script.module.pyserial.
da jetzt nun die abhängigkeiten fehlten, die mein addon benötigt, musste dies in der addon.xml zusätzlich importiert werden.
im script verwende ich ganz normal "import serial"

jetzt läuft alles wieder, wie es soll, nur dass ich das pyserial modul nun outgesourct habe.

ps. import und from musst du tasuchen --> from bs4 import BeautifulSoup

Adromir

So ich wurschtele mich gerade so durch, hab auch schon ein paar kleine Erfolge für mich erreicht. Ich merke aber, daß ich schon etwas überfordert bin an manchen Stellen und mich frage, ob ich da zu blöd bin (wahrscheinlich) oder ob die Doku auf der Webseite von Kodi auch nicht die beste ist.

Das fängt schon mal dabei an, daß ich mittlerweile unsicher bin, ob für mein Addon der Typ Plugin oder Script die bessere wahl ist.. In der Doku finde ich nichts so richtig, wann ich was nehmen sollte..

Dann der nächste Knackpunkt ist die GUI Entwicklung. Da sind die Informationen doch etwas Mager, so daß ich noch nicht mal weiß, ob ich das was ich möchte überhaupt umsetzen kann. Eigentlich bräuchte ich für das Addon mehrere Fenster mit unterschiedlichen Ansichten/Funktionen und mit einer Allgemeinen Navigation bei der der User zwischen den Fenstern navigieren kann. Soweit ich verstanden habe, geht das per xml datei Aaaber da gibt es iwie nichts, wo man das mal so beispielhaft sehen kann...

harryberlin

auch, wenn einiges nicht python konform ist, und einige verbessern wollen.
beim aufruf des skin werden alle control verknüpft. solang das nicht erledigt ist, zeige ich eine loader animation. welche danach deaktivert wird. dann können alle anderen sachen mit der condition Strin.IsEqual() angezeigt werden, oder vorangegangene animationen, pi pa po.
so kann das ganze aussehen: https://www.youtube.com/watch?v=5nZlD1gYje8
hier mal eine kleine zusammenstellung:

Python

#!/usr/bin/env python
# -*- coding: utf-8 -*-


__author__ = 'harryberlin'


import os
import xbmc, xbmcgui, xbmcaddon
from threading import Thread, Event




ADDON = xbmcaddon.Addon()
ADDON_NAME = ADDON.getAddonInfo('name')
ADDON_ID = ADDON.getAddonInfo('id')
ADDON_VERSION = ADDON.getAddonInfo('version')
ADDON_PATH = ADDON.getAddonInfo('path')
ADDON_USER_PATH = os.path.join(xbmc.translatePath('special://userdata'), 'addon_data', ADDON_ID)


ADDON_SKIN_MEDIA_PATH = os.path.join(ADDON_PATH, 'resources', 'skins', 'default', 'media')


ADDON_ICON = os.path.join(ADDON_PATH, 'icon.png')


ADDON_COLOR_YELLOW = 'FFE6C819'
ADDON_COLOR_GREEN = 'FF3BFC00'
ADDON_COLOR_BLUE = 'FF007CC3'
ADDON_COLOR_RED = 'FFDB214C'
ADDON_COLOR_AMBER = 'FFFF7E00'
ADDON_COLOR_WHITE = 'FFFFFFFF'
ADDON_COLOR_BLACK = 'FF000000'




class RepeatTimer(Thread):
    def __init__(self, interval, function, iterations=0, args=[], kwargs={}):
        Thread.__init__(self)
        self.daemon = True
        self.interval = interval
        self.function = function
        self.iterations = iterations
        self.args = args
        self.kwargs = kwargs
        self.finished = Event()


    def run(self):
        count = 0
        while not self.finished.isSet() and (self.iterations <= 0 or count < self.iterations):
            self.finished.wait(self.interval)
            if not self.finished.isSet():
                self.function(*self.args, **self.kwargs)
                count += 1
        [definition='1','0']log[/definition]('Timer Thread finished')


    def cancel(self):
        self.finished.set()




class ObcGuiClass(xbmcgui.WindowXML):
    ACTION_PARENT_DIR = 9
    ACTION_PREVIOUS_MENU = 10
    ACTION_CLOSE_DIALOG = 51
    ACTION_SELECT_ITEM = 7
    ACTION_MOVE_LEFT = 1
    ACTION_MOVE_RIGHT = 2
    ACTION_MOVE_UP = 3
    ACTION_MOVE_DOWN = 4
    ACTION_NAV_BACK = 92
    ACTION_BACKSPACE = 110
    ACTION_MOUSE_LEFT_CLICK = 100
    ACTION_SHOW_FULLSCREEN = 36


    def __init__(self, xmlFilename, scriptPath, defaultSkin, defaultRes):
        [definition='1','0']log[/definition]("Initializing OBC Gui...")


        set_property('IBUSCOMMUNICATOR_OBC_LOADED', '0')


        self.MAINLABEL = None
        self.MAINLEFT = None
        self.MAINRIGHT = None


        self.BUTTON101 = None
        self.BUTTON102 = None
        self.BUTTON103 = None
        self.BUTTON104 = None
        self.BUTTON105 = None
        self.BUTTON106 = None
        self.BUTTON107 = None
        self.BUTTON108 = None


        self.LABEL111 = None
        self.LABEL112 = None
        self.LABEL113 = None
        self.LABEL114 = None
        self.LABEL115 = None
        self.LABEL116 = None
        self.LABEL117 = None
        self.LABEL118 = None


        self.IMAGE121 = None
        self.IMAGE122 = None
        self.IMAGE123 = None
        self.IMAGE124 = None
        self.IMAGE125 = None
        self.IMAGE126 = None
        self.IMAGE127 = None
        self.IMAGE128 = None


        self.isActive = False


        self.updateTimer = None


        self.window_id = -10000


    def isVisible(self):
        [definition='1','0']log[/definition]('OBC: VISIBLE: WINDOW ID: %s' % xbmcgui.getCurrentWindowId(), 2)


        if self.isActive and bool(xbmc.getCondVisibility('Window.IsActive(%s)' % self.window_id)):
            return True
        else:
            return False


    def onInit(self):
        [definition='1','0']log[/definition]('OBC INIT: WINDOW ID: %s' % xbmcgui.getCurrentWindowId())
        self.window_id = xbmcgui.getCurrentWindowId()


        [definition='1','0']log[/definition]('OBC: GUI Opening')


        self.isActive = True


        self.MAINLABEL = self.getControl(200)
        self.MAINLEFT = self.getControl(201)
        self.MAINRIGHT = self.getControl(202)


        self.IMAGE101 = self.getControl(101)
        self.IMAGE102 = self.getControl(102)
        self.IMAGE103 = self.getControl(103)
        self.IMAGE104 = self.getControl(104)
        self.IMAGE105 = self.getControl(105)
        self.IMAGE106 = self.getControl(106)
        self.IMAGE107 = self.getControl(107)
        self.IMAGE108 = self.getControl(108)


        self.BUTTON111 = self.getControl(111)
        self.BUTTON112 = self.getControl(112)
        self.BUTTON113 = self.getControl(113)
        self.BUTTON114 = self.getControl(114)
        self.BUTTON115 = self.getControl(115)
        self.BUTTON116 = self.getControl(116)
        self.BUTTON117 = self.getControl(117)
        self.BUTTON118 = self.getControl(118)


        self.LABEL121 = self.getControl(121)
        self.LABEL122 = self.getControl(122)
        self.LABEL123 = self.getControl(123)
        self.LABEL124 = self.getControl(124)
        self.LABEL125 = self.getControl(125)
        self.LABEL126 = self.getControl(126)
        self.LABEL127 = self.getControl(127)
        self.LABEL128 = self.getControl(128)


        # Bottom Line
        self.LABEL211 = self.getControl(211)


        self.set_default_navigation()


        self.update()


        if self.updateTimer:
            self.updateTimer.cancel()
            self.updateTimer.join()
        self.updateTimer = RepeatTimer(1.000, self.update)
        self.updateTimer.start()


        set_property('IBUSCOMMUNICATOR_OBC_LOADED', '1')


    def onMouseLeftClick(self, controlId):
        [definition='1','0']log[/definition]('ONCLICK: %s' % controlId, 3)


        if controlId == 201:  # left
            [definition='1','0']log[/definition]('event left')
        elif controlId == 202:  # right
            [definition='1','0']log[/definition]('event right')


        elif 111 <= controlId <= 118:  # Buttons
            [definition='1','0']log[/definition]('event select button')


    def onAction(self, Action):
        if Action in [self.ACTION_PREVIOUS_MENU, self.ACTION_CLOSE_DIALOG, self.ACTION_NAV_BACK, self.ACTION_BACKSPACE, self.ACTION_SHOW_FULLSCREEN]:
            self.onStop()


        if Action == self.ACTION_MOUSE_LEFT_CLICK:
            [definition='1','0']log[/definition]('MOUSE CLICKED')
            self.onMouseLeftClick(self.getFocusId())
            return


        if Action == self.ACTION_MOVE_LEFT:  # left
            [definition='1','0']log[/definition]('event left')




        elif Action == self.ACTION_MOVE_RIGHT:  # right
            [definition='1','0']log[/definition]('event right')




        elif Action == self.ACTION_SELECT_ITEM:
            [definition='1','0']log[/definition]('event select')


        elif Action == self.ACTION_MOVE_UP:
            [definition='1','0']log[/definition]('event up')


        elif Action == self.ACTION_MOVE_DOWN:
            [definition='1','0']log[/definition]('event down')


    def onFocus(self, controlId):
        pass


    def onStop(self, close_window=True):
        [definition='1','0']log[/definition]('OBC: GUI Closing')
        self.isActive = False


        self.updateTimer.join()
        [definition='1','0']log[/definition]('Timers joined')


        if close_window:
            self.close()


        set_property('IBUSCOMMUNICATOR_OBC_LOADED', '0')


    def set_default_navigation(self):
        self.BUTTON111.setNavigation(self.BUTTON111, self.BUTTON112, self.BUTTON111, self.BUTTON111)
        self.BUTTON112.setNavigation(self.BUTTON111, self.BUTTON113, self.BUTTON112, self.BUTTON112)
        self.BUTTON113.setNavigation(self.BUTTON112, self.BUTTON114, self.BUTTON113, self.BUTTON113)
        self.BUTTON114.setNavigation(self.BUTTON113, self.BUTTON115, self.BUTTON114, self.BUTTON114)
        self.BUTTON115.setNavigation(self.BUTTON114, self.BUTTON116, self.BUTTON115, self.BUTTON115)
        self.BUTTON116.setNavigation(self.BUTTON115, self.BUTTON117, self.BUTTON116, self.BUTTON116)
        self.BUTTON117.setNavigation(self.BUTTON116, self.BUTTON118, self.BUTTON117, self.BUTTON117)
        self.BUTTON118.setNavigation(self.BUTTON117, self.BUTTON118, self.BUTTON118, self.BUTTON118)


    def update(self):
        # update your controls
        pass




def [definition='1','0']log[/definition](string, level=1):
    xbmc.[definition='1','0']log[/definition]('IBUSCOMMUNICATOR: %s' % string, xbmc.LOGNOTICE)




def note(heading, message=None, time=-1, icon=None):
    if time == -1:
        xbmcgui.Dialog().notification(heading='%s' % heading, message='%s' % message if message else ' ', icon='%s' % (icon if icon else ADDON_ICON))
    else:
        xbmcgui.Dialog().notification(heading='%s' % heading, message='%s' % message if message else ' ', icon='%s' % (icon if icon else ADDON_ICON), time=time)
    [definition='1','0']log[/definition]('NOTIFICATION: "%s%s"' % (heading, ' - %s' % message if message else ''))




def dialog_ok(label1, label2=None, label3=None):
    [definition='1','0']log[/definition]('DIALOG_OK: "%s%s%s"' % (label1, ' - %s' % label2 if label2 else '', ' - %s' % label3 if label3 else ''))
    xbmcgui.Dialog().ok(ADDON_NAME, label1, label2, label3)




def dialog_yesno(label1, label2=None, label3=None, nolabel='', yeslabel='', autoclose=0):
    return xbmcgui.Dialog().yesno(ADDON_NAME, line1=label1, line2=label2, line3=label3, nolabel=nolabel, yeslabel=yeslabel, autoclose=autoclose)




def get_addon_setting(id):
    setting = ADDON.getSetting(id)
    if setting.upper() == 'TRUE':
        return True
    if setting.upper() == 'FALSE':
        return False
    return '%s' % setting




def set_addon_setting(id, value):
    if type(value) == 'bool':
        ADDON.setSetting(id, 'true' if value else 'false')
    else:
        ADDON.setSetting(id, '%s' % value)




def get_property(property, id=10000):
    value = xbmcgui.Window(id).getProperty(property)
    return value




def set_property(property, value, id=10000):
    xbmcgui.Window(id).setProperty(property, value)




def clear_property(property, id=10000):
    xbmcgui.Window(id).clearProperty(property)




def get_addon_string(id):
    return ADDON.getLocalizedString(id)




def addon_skin_path():
    current_skin_name = xbmc.getSkinDir()
    if os.path.exists(os.path.join(ADDON_PATH, 'resources', 'skins', current_skin_name)):
        return '%s' % current_skin_name
    else:
        return 'default'




def text_color(text, color=None):
    if color:
        return '[COLOR %s]%s[/COLOR]' % (color, text)
    else:
        return text




def main():
    xbmc.[definition='1','0']log[/definition]('IBUSCOMMUNICATOR: Service Started', xbmc.LOGNOTICE)


    obc_gui = ObcGuiClass("OBC_SKIN.xml", ADDON_PATH, addon_skin_path(), '720p')
    obc_gui.doModal()


    xbmc.[definition='1','0']log[/definition]('IBUSCOMMUNICATOR: Service Stopped', xbmc.LOGNOTICE)




if __name__ == '__main__':
    main()

Alles anzeigen

ClaudiaF

Zitat von harryberlin

auch, wenn einiges nicht python konform ist

Ja, Du bist durchaus kreativ ,
aber letztendlich kommt es doch darauf an ob es tut was es soll

Cheers
Claudia

harryberlin

hmm naja. im python-forum wird man dafür gesteinigt.

ClaudiaF

Ich bin schon der Meinung das Code durchaus nach gewissen Masstäben geschrieben werden sollte,
ABER, es muß sich auch die Frage gestellt werden, wofür den der Code gedacht ist.

Wenn Du dass im Python-Forum postest mit dem Anspruch jemanden etwas über Pythonprgorammierung zu erklären,
oder ein Open-Source Projekt daraus machst das ggf. andere mitpflegen sollen, dann ja, ich bin ohne Sünde ich schmeiße dann mal

Aber wenn es um dein Projekt geht, dass Du schreibst und für Dich tut was es soll, dann gibt es kein pep8. PUNKT.
Du mußt wissen was das tut und wie Du damit am besten auskommst.
Was helfen List Comprehensions wenn ich mich immer wieder fragen muß was das nochmal war oder ...
Dein Projekt - dein Codingstyle - dein Problem nachher

Abgesehen davon, pep8 ist keine festgeschriebene Bibel, es ist eine Empfehlung und wer das nicht versteht
mag mir doch bitte mal erklären warum ein Zeilenumbruch, heutzutage, bei 80 Zeichen sein soll.

In dem Sinn - keep on coding (ha, dass ist ja auch noch zweideutig)

Cheers
Claudia

PvD

Nunja, ich finde den Code von @harryberlin durchaus gut strukturiert und lesbar. Ich hätte es jetzt auch nicht anders gemacht. Natürlich sollte man sich soweit es geht, an die Programmierkonventionen halten, allerdings ist es ja - wie schon bemerkt - keine Bibel. Man kann jetzt auch den Max raushängen lassen und alles schön 'pythonic' schreiben, m.M. nach ist sowas aber gerade für andere schwerer lesbar.

JM2C

Jetzt mitmachen!