Add Enphase Envoy component by jesserizzo · Pull Request #15081 · home-assistant/core

jesserizzo · 2018-06-21T22:16:32Z

Description:

Add component to get energy consumption and solar panel energy production from an Enphase Envoy

Related issue (if applicable): fixes #

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.io#5638

Example entry for `configuration.yaml` (if applicable):

# Example configuration.yaml entry
sensor:
  - platform: enphase_envoy
    ip: LOCAL_IP_OF_ENVOY
    name: NAME_TO_DISPLAY_IN_FRONTEND
    monitored_conditions:
    - production
    - daily_production
    - 7_days_production
    - lifetime_production
    - consumption
    - daily_consumption
    - 7_days_consumption
    - lifetime_consumption

Checklist:

The code change is tested and works locally.
Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

New dependencies have been added to the REQUIREMENTS variable (example).
New dependencies are only imported inside functions that use them (example).
New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
New files were added to .coveragerc.

If the code does not interact with devices:

Tests have been added to verify that the new code works.

homeassistant · 2018-06-21T22:16:33Z

Hi @jesserizzo,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

houndci-bot · 2018-06-21T22:16:38Z

homeassistant/components/sensor/enphase_envoy.py

+        """Initialize the sensor."""
+
+
+        self._url = "http://{}/production.json".format(ip_address)


too many blank lines (2)

houndci-bot · 2018-06-21T22:16:38Z

homeassistant/components/sensor/enphase_envoy.py

+
+
+
+class Envoy(Entity):


too many blank lines (3)

houndci-bot · 2018-06-21T22:16:38Z

homeassistant/components/sensor/enphase_envoy.py

+    #out of the config, or that the given sensor is in the monitored conditions list
+    for i in range(8):
+        if monitored_conditions == {} or SENSOR_TYPES[i] in monitored_conditions:
+            add_devices([Envoy(ip_address, DEFAULT_NAMES[i], SENSOR_TYPES[i])], True)


line too long (85 > 79 characters)

houndci-bot · 2018-06-21T22:16:38Z

homeassistant/components/sensor/enphase_envoy.py

+    #Iterate through the list of sensors, adding it if either monitored conditions has been left
+    #out of the config, or that the given sensor is in the monitored conditions list
+    for i in range(8):
+        if monitored_conditions == {} or SENSOR_TYPES[i] in monitored_conditions:


line too long (81 > 79 characters)

houndci-bot · 2018-06-21T22:16:38Z

homeassistant/components/sensor/enphase_envoy.py

+    monitored_conditions = config.get(CONF_MONITORED_CONDITIONS, {})
+
+    #Iterate through the list of sensors, adding it if either monitored conditions has been left
+    #out of the config, or that the given sensor is in the monitored conditions list


block comment should start with '# '
line too long (84 > 79 characters)

houndci-bot · 2018-06-21T22:16:39Z

homeassistant/components/sensor/enphase_envoy.py

+
+DEFAULT_NAMES = ["Envoy Current Energy Production", "Envoy Today's Energy Production", "Envoy Last Seven Days Energy Production",
+"Envoy Lifetime Energy Production", "Envoy Current Energy Consumption", "Envoy Today's Energy Consumption",
+"Envoy Last Seven Days Energy Consumption", "Envoy Lifetime Energy Consumption"]


continuation line under-indented for visual indent
line too long (80 > 79 characters)

houndci-bot · 2018-06-21T22:16:39Z

homeassistant/components/sensor/enphase_envoy.py

+CONF_MONITORED_CONDITIONS = 'monitored_conditions'
+
+DEFAULT_NAMES = ["Envoy Current Energy Production", "Envoy Today's Energy Production", "Envoy Last Seven Days Energy Production",
+"Envoy Lifetime Energy Production", "Envoy Current Energy Consumption", "Envoy Today's Energy Consumption",


continuation line under-indented for visual indent
line too long (107 > 79 characters)

houndci-bot · 2018-06-21T22:16:39Z

homeassistant/components/sensor/enphase_envoy.py

+CONF_IP_ADDRESS = 'ip'
+CONF_MONITORED_CONDITIONS = 'monitored_conditions'
+
+DEFAULT_NAMES = ["Envoy Current Energy Production", "Envoy Today's Energy Production", "Envoy Last Seven Days Energy Production",


line too long (129 > 79 characters)

houndci-bot · 2018-06-21T22:16:39Z

homeassistant/components/sensor/enphase_envoy.py

+from homeassistant.helpers.entity import Entity
+from homeassistant.components.sensor import PLATFORM_SCHEMA
+import homeassistant.helpers.config_validation as cv
+from homeassistant.const import CONF_NAME


'homeassistant.const.CONF_NAME' imported but unused

houndci-bot · 2018-06-21T22:16:40Z

homeassistant/components/sensor/enphase_envoy.py

@@ -0,0 +1,126 @@
+"""
+Support for monitoring energy usage and solar panel energy production using the Enphase Envoy.


line too long (94 > 79 characters)

houndci-bot · 2018-06-21T22:31:24Z

homeassistant/components/sensor/enphase_envoy.py

+            add_devices([Envoy(ip_address, DEFAULT_NAMES[i], \
+            SENSOR_TYPES[i])], True)
+
+class Envoy(Entity):


expected 2 blank lines, found 1

houndci-bot · 2018-06-21T22:31:24Z

homeassistant/components/sensor/enphase_envoy.py

+        if monitored_conditions == {} or \
+        SENSOR_TYPES[i] in monitored_conditions:
+            add_devices([Envoy(ip_address, DEFAULT_NAMES[i], \
+            SENSOR_TYPES[i])], True)


continuation line under-indented for visual indent

houndci-bot · 2018-06-21T22:31:24Z

homeassistant/components/sensor/enphase_envoy.py

+    for i in range(8):
+        if monitored_conditions == {} or \
+        SENSOR_TYPES[i] in monitored_conditions:
+            add_devices([Envoy(ip_address, DEFAULT_NAMES[i], \


the backslash is redundant between brackets

houndci-bot · 2018-06-21T22:31:25Z

homeassistant/components/sensor/enphase_envoy.py

+#is in the monitored conditions list
+    for i in range(8):
+        if monitored_conditions == {} or \
+        SENSOR_TYPES[i] in monitored_conditions:


continuation line missing indentation or outdented

houndci-bot · 2018-06-21T22:31:25Z

homeassistant/components/sensor/enphase_envoy.py

+
+#Iterate through the list of sensors, adding it if either monitored
+#conditions has been left out of the config, or that the given sensor
+#is in the monitored conditions list


block comment should start with '# '

houndci-bot · 2018-06-21T22:31:25Z

homeassistant/components/sensor/enphase_envoy.py

+    ip_address = config.get(CONF_IP_ADDRESS)
+    monitored_conditions = config.get(CONF_MONITORED_CONDITIONS, {})
+
+#Iterate through the list of sensors, adding it if either monitored


block comment should start with '# '

houndci-bot · 2018-06-21T22:31:25Z

homeassistant/components/sensor/enphase_envoy.py

+
+SENSOR_TYPES = ["production", "daily_production", "7_days_production",
+    "lifetime_production", "consumption", "daily_consumption",
+    "7_days_consumption", "lifetime_consumption"]


continuation line under-indented for visual indent

houndci-bot · 2018-06-21T22:31:25Z

homeassistant/components/sensor/enphase_envoy.py

+    "Envoy Lifetime Energy Consumption"]
+
+SENSOR_TYPES = ["production", "daily_production", "7_days_production",
+    "lifetime_production", "consumption", "daily_consumption",


continuation line under-indented for visual indent

houndci-bot · 2018-06-21T22:31:26Z

homeassistant/components/sensor/enphase_envoy.py

+DEFAULT_NAMES = ["Envoy Current Energy Production",
+    "Envoy Today's Energy Production",
+    "Envoy Last Seven Days Energy Production",
+    "Envoy Lifetime Energy Production",


continuation line under-indented for visual indent

houndci-bot · 2018-06-21T22:31:26Z

homeassistant/components/sensor/enphase_envoy.py

+
+DEFAULT_NAMES = ["Envoy Current Energy Production",
+    "Envoy Today's Energy Production",
+    "Envoy Last Seven Days Energy Production",


continuation line under-indented for visual indent

houndci-bot · 2018-06-21T22:37:22Z

homeassistant/components/sensor/enphase_envoy.py

+        if monitored_conditions == {} or \
+            SENSOR_TYPES[i] in monitored_conditions:
+            add_devices([Envoy(ip_address, DEFAULT_NAMES[i],
+                SENSOR_TYPES[i])], True)


continuation line under-indented for visual indent

houndci-bot · 2018-06-21T22:37:22Z

homeassistant/components/sensor/enphase_envoy.py

+# is in the monitored conditions list
+    for i in range(8):
+        if monitored_conditions == {} or \
+            SENSOR_TYPES[i] in monitored_conditions:


continuation line with same indent as next logical line

houndci-bot · 2018-06-21T22:37:22Z

homeassistant/components/sensor/enphase_envoy.py

+from homeassistant.helpers.entity import Entity
+from homeassistant.components.sensor import PLATFORM_SCHEMA
+import homeassistant.helpers.config_validation as cv
+#from homeassistant.const import CONF_NAME


block comment should start with '# '

exxamalte · 2018-06-22T08:50:51Z

homeassistant/components/sensor/enphase_envoy.py

+        import requests
+
+        try:
+            response = requests.get(self._url, timeout=5)


Why not use the RestData class (components/sensor/rest.py) here? May save a bit of code duplication.
Or maybe even use one instance of RestData and share across all sensors?

exxamalte · 2018-06-22T09:54:55Z

I have an Envoy-S myself and so far have simply been using rest sensors to fetch the data and display it in HA. In my experience, having HA to retrieve the JSON from the Envoy device 8 times in a very short time sometimes led to one or two requests to fail. Hence I am suggesting to use RestData to fetch the data and share this object across all sensors.

exxamalte · 2018-06-22T10:10:29Z

I'd also like to suggest to add some unit tests. The JSON format used by the Envoy is not a public API, and having a few current sample JSON responses to test your code against would help just in case Enphase decides to change the format; you would then just update the test JSON and modify the code until the tests work again.

exxamalte · 2018-06-22T10:19:59Z

homeassistant/components/sensor/enphase_envoy.py

+        if type == 'production' or type == 'consumption':
+            self._unit_of_measurement = 'watts'
+        else:
+            self._unit_of_measurement = "watt hours"


I would prefer W and Wh here. Looking at homeassistant/const.py it seems to be the consensus for other units of measurement to use the abbreviated version. Maybe these two could be added there?

A couple of really good points, thank you. I'm not really sure how to do unit tests. Is there somewhere you can suggest to learn more about that?

Maybe the easiest way is by looking at the existing unit tests. Most unit tests focus on two areas - setting up the component, and then testing the various functionalities, transformation, error handling, etc. You can load a static JSON file with load_fixture.
The developer documentation tells you how to run tests in your environment.

houndci-bot · 2018-06-29T20:17:15Z

homeassistant/components/sensor/enphase_envoy.py

+        elif self._type == "daily_consumption":
+            self._state = int(response_parsed["consumption"][0]["whToday"])
+        elif self._type == "7_days_consumption":
+            self._state = int(response_parsed["consumption"][0]["whLastSevenDays"])


line too long (83 > 79 characters)

houndci-bot · 2018-06-29T20:17:15Z

homeassistant/components/sensor/enphase_envoy.py

+        elif self._type == "daily_production":
+            self._state = int(response_parsed["production"][1]["whToday"])
+        elif self._type == "7_days_production":
+            self._state = int(response_parsed["production"][1]["whLastSevenDays"])


line too long (82 > 79 characters)

houndci-bot · 2018-06-29T20:17:15Z

homeassistant/components/sensor/enphase_envoy.py

+        self._name = name
+
+
+        if sensor_type == 'production' or sensor_type == 'consumption':


too many blank lines (2)

houndci-bot · 2018-06-29T20:17:15Z

homeassistant/components/sensor/enphase_envoy.py

+import json
+import voluptuous as vol
+
+from homeassistant.components.sensor.rest import RestData


'homeassistant.components.sensor.rest.RestData' imported but unused

houndci-bot · 2018-06-29T20:34:46Z

homeassistant/components/sensor/enphase_envoy.py

+            self._state = int(response_parsed["consumption"][0]["whToday"])
+        elif self._type == "7_days_consumption":
+            self._state = \
+            int(response_parsed["consumption"][0]["whLastSevenDays"])


continuation line missing indentation or outdented

houndci-bot · 2018-06-29T20:34:46Z

homeassistant/components/sensor/enphase_envoy.py

+            self._state = int(response_parsed["production"][1]["whToday"])
+        elif self._type == "7_days_production":
+            self._state = \
+            int(response_parsed["production"][1]["whLastSevenDays"])


continuation line missing indentation or outdented

houndci-bot · 2018-06-29T20:34:47Z

homeassistant/components/sensor/enphase_envoy.py

+        if monitored_conditions == {} or \
+                    SENSOR_TYPES[i] in monitored_conditions:
+                add_devices([Envoy(ip_address, DEFAULT_NAMES[i],
+                        SENSOR_TYPES[i])], True)


continuation line under-indented for visual indent

houndci-bot · 2018-06-29T20:55:56Z

homeassistant/components/sensor/enphase_envoy.py

+        if monitored_conditions == {} or \
+                    SENSOR_TYPES[i] in monitored_conditions:
+            add_devices([Envoy(ip_address, DEFAULT_NAMES[i],
+                                SENSOR_TYPES[i])], True)


continuation line over-indented for visual indent

frenck · 2018-06-30T20:44:40Z

Docs PR spotted. Removed docs-missing label.

MartinHjelmare

The requests and parsing should be broken out into standalone library published on pypi.

https://developers.home-assistant.io/docs/en/creating_platform_code_review.html#6-communication-with-devices-services

MartinHjelmare

Nice! See further comments below.

MartinHjelmare · 2018-07-30T05:33:23Z

homeassistant/components/sensor/enphase_envoy.py

+For more details about this platform, please refer to the documentation at
+https://home-assistant.io/components/sensor.enphase_envoy/
+"""
+


Please remove this blank line.

MartinHjelmare · 2018-07-30T05:33:38Z

homeassistant/components/sensor/enphase_envoy.py

+"""
+
+import logging
+import voluptuous as vol


Please add a blank line between standard library and 3rd party imports and 3rd party and homeassistant imports.

MartinHjelmare · 2018-07-30T05:35:55Z

homeassistant/components/sensor/enphase_envoy.py

+REQUIREMENTS = ['envoy_reader==0.1']
+_LOGGER = logging.getLogger(__name__)
+
+CONF_IP_ADDRESS = 'ip'


Please import and use CONF_IP_ADDRESS from const.py.

MartinHjelmare · 2018-07-30T05:37:08Z

homeassistant/components/sensor/enphase_envoy.py

+_LOGGER = logging.getLogger(__name__)
+
+CONF_IP_ADDRESS = 'ip'
+CONF_MONITORED_CONDITIONS = 'monitored_conditions'


Import and use CONF_MONITORED_CONDITIONS from const.py.

MartinHjelmare · 2018-07-30T05:44:04Z

homeassistant/components/sensor/enphase_envoy.py

+
+PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend({
+    vol.Required(CONF_IP_ADDRESS): cv.string,
+    vol.Optional(CONF_MONITORED_CONDITIONS): vol.All(cv.ensure_list)


Make sure the selected conditions are in the SENSOR_TYPES list and default to the whole list.

vol.Optional(CONF_MONITORED_CONDITIONS, default=SENSOR_TYPES): vol.All( cv.ensure_list, [vol.In(SENSOR_TYPES)])

MartinHjelmare · 2018-07-30T06:13:48Z

homeassistant/components/sensor/enphase_envoy.py

+# Iterate through the list of sensors, adding it if either monitored
+# conditions has been left out of the config, or that the given sensor
+# is in the monitored conditions list
+    for i in range(8):


Just iterate monitored_conditions.

MartinHjelmare · 2018-07-30T06:15:58Z

homeassistant/components/sensor/enphase_envoy.py

+    "Envoy Last Seven Days Energy Consumption",
+    "Envoy Lifetime Energy Consumption"]
+
+SENSOR_TYPES = [


It's cleaner to make this a list of lists or tuples, where the items in the list or tuple are the sensor type and the corresponding sensor name. Then we don't have to keep track of that the order in the list of types matches the order of the list of names. See eg:
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/sensor/airvisual.py#L43-L47

MartinHjelmare · 2018-07-30T06:20:26Z

homeassistant/components/sensor/enphase_envoy.py

+# conditions has been left out of the config, or that the given sensor
+# is in the monitored conditions list
+    for i in range(8):
+        if monitored_conditions == {} or \


Validation should already be done in the config schema, so we don't need to check that here.

MartinHjelmare · 2018-07-30T06:21:27Z

homeassistant/components/sensor/enphase_envoy.py

+        """Initialize the sensor."""
+        self._ip_address = ip_address
+        self._name = name
+        if sensor_type == 'production' or sensor_type == 'consumption':


Move the unit of measurement type to the nested list of sensor types. If no unit should be used, put None in the list.

MartinHjelmare · 2018-07-30T06:23:24Z

requirements_all.txt

 # homeassistant.components.zha
 zigpy==0.1.0
+
+# homeassistant.components.sensor.enphase_envoy


Run script/gen_requirements.py to update the requirements file correctly.

jesserizzo · 2018-08-02T14:48:43Z

Ok @MartinHjelmare I did most of the changes you suggested. Thanks for being so detailed. The only thing I did differently is I made the sensors a dictionary with value a tuple of the default name and unit of measurement.

SENSORS = {
    "production": ("Envoy Current Energy Production", 'W'),  
    "daily_production": ("Envoy Today's Energy Production", "Wh"),  
    "7_days_production": ("Envoy Last Seven Days Energy Production", "Wh"),  
    "lifetime_production": ("Envoy Lifetime Energy Production", "Wh"),  
    "consumption": ("Envoy Current Energy Consumption", "W"),  
    "daily_consumption": ("Envoy Today's Energy Consumption", "Wh"),  
    "7_days_consumption": ("Envoy Last Seven Days Energy Consumption", "Wh"),  
    "lifetime_consumption": ("Envoy Lifetime Energy Consumption", "Wh")  
    }

I think this accomplishes what you wanted so we aren't trying to keep multiple lists synced up.

Also, I can't get the gen_requirements_all.py to run. I'm getting

No such file or directory: '../homeassistant/package_constraints.txt'

Any suggestions? Thanks.

MartinHjelmare · 2018-08-02T15:02:46Z

Did you run script/setup first?

https://developers.home-assistant.io/docs/en/development_environment.html#setting-up-virtual-environment

MartinHjelmare · 2018-08-02T15:08:07Z

homeassistant/components/sensor/enphase_envoy.py

+from homeassistant.components.sensor import PLATFORM_SCHEMA
+import homeassistant.helpers.config_validation as cv
+from homeassistant.const import (CONF_IP_ADDRESS)
+from homeassistant.const import (CONF_MONITORED_CONDITIONS)


Group imports from the same module. I use isort to do this automatically for new modules. Most advanced editors have plugins that can do this.

from homeassistant.const import ( CONF_IP_ADDRESS, CONF_MONITORED_CONDITIONS)

MartinHjelmare · 2018-08-02T15:09:09Z

homeassistant/components/sensor/enphase_envoy.py

+    ip_address = config[CONF_IP_ADDRESS]
+    monitored_conditions = config[CONF_MONITORED_CONDITIONS]
+
+# Iterate through the list of sensors


Indent the comment with the code that the comment is aimed at.

MartinHjelmare · 2018-08-02T15:10:52Z

homeassistant/components/sensor/enphase_envoy.py

+https://home-assistant.io/components/sensor.enphase_envoy/
+"""
+import logging
+import voluptuous as vol


There should be a blank line between logging and voluptuous.

jesserizzo · 2018-08-02T16:07:41Z

@MartinHjelmare Did you run script/setup first?

Yep, that was it. Thanks.

MartinHjelmare

Great!

machielw · 2018-08-05T20:19:24Z

Error with envoy which does not measure the Consumption (Envoy s standard). I Left the last 4 lines away from the sensor configuration.

Log Details (ERROR)
Sun Aug 05 2018 17:12:29 GMT+0200 (CEST)

Invalid config for [sensor.enphase_envoy]: required key not provided @ data['ip_address']. Got None. (See ?, line ?). Please check the docs at https://home-assistant.io/components/sensor.enphase_envoy

MartinHjelmare · 2018-08-05T20:40:07Z

Please open an issue.

jesserizzo added 2 commits June 21, 2018 16:07

add enphase envoy component

c7ca273

Add Enphase Envoy component for energy monitoring

75c87fa

homeassistant added cla-needed new-platform platform: sensor.enphase_envoy labels Jun 21, 2018

houndci-bot reviewed Jun 21, 2018

View reviewed changes

ghost added the in progress label Jun 21, 2018

homeassistant added cla-signed and removed cla-needed labels Jun 21, 2018

Fix formatting problems

30ece48

houndci-bot reviewed Jun 21, 2018

View reviewed changes

Fix formatting errors

e549f4e

houndci-bot reviewed Jun 21, 2018

View reviewed changes

jesserizzo added 2 commits June 21, 2018 17:40

Fix formatting errors

4adfbde

Fix formatting errors

66d0df8

exxamalte reviewed Jun 22, 2018

View reviewed changes

Change unit of measurement to W or Wh. Return sensor states as integers

ad17cb1

houndci-bot reviewed Jun 29, 2018

View reviewed changes

Fix formatting errors

b8d3738

houndci-bot reviewed Jun 29, 2018

View reviewed changes

Fix formatting errors

fe90ffd

houndci-bot reviewed Jun 29, 2018

View reviewed changes

Fix formatting errors

a38ebe7

frenck added the docs-missing label Jun 30, 2018

Move import json to update function

a325c8a

jesserizzo mentioned this pull request Jun 30, 2018

Add documentation for new Enphase Envoy platform home-assistant/home-assistant.io#5638

Merged

2 tasks

frenck removed the docs-missing label Jun 30, 2018

MartinHjelmare requested changes Jul 18, 2018

View reviewed changes

jesserizzo added 2 commits July 29, 2018 16:41

Add new component to requirements_all.txt

77e5355

Move API call to third party library on PyPi

d43d1bc

homeassistant added the cla-signed label Jul 29, 2018

MartinHjelmare requested changes Jul 30, 2018

View reviewed changes

home-assistant deleted a comment from houndci-bot Jul 30, 2018

Refactor

d90bd63

homeassistant added the cla-signed label Aug 2, 2018

MartinHjelmare reviewed Aug 2, 2018

View reviewed changes

jesserizzo added 2 commits August 2, 2018 11:03

Run gen_requirements_all.py

8d88fa2

Minor refactor

bf47d88

homeassistant added the cla-signed label Aug 2, 2018

Fix indentation

5004d3c

homeassistant added the cla-signed label Aug 2, 2018

Fix indentation

11cc650

MartinHjelmare approved these changes Aug 2, 2018

View reviewed changes

MartinHjelmare merged commit affd4e7 into home-assistant:dev Aug 2, 2018

ghost removed the in progress label Aug 2, 2018

home-assistant locked as resolved and limited conversation to collaborators Aug 5, 2018

ghost added integration: enphase_envoy and removed platform: sensor.enphase_envoy labels Mar 21, 2019

		"""Initialize the sensor."""


		self._url = "http://{}/production.json".format(ip_address)

		@@ -0,0 +1,126 @@
		"""
		Support for monitoring energy usage and solar panel energy production using the Enphase Envoy.

		self._name = name


		if sensor_type == 'production' or sensor_type == 'consumption':

Uh oh!

Conversation

jesserizzo commented Jun 21, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description:

Example entry for configuration.yaml (if applicable):

Checklist:

Uh oh!

homeassistant commented Jun 21, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

exxamalte Jun 22, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

exxamalte commented Jun 22, 2018

Uh oh!

exxamalte commented Jun 22, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jesserizzo commented Jun 21, 2018 •

edited

Loading

Example entry for `configuration.yaml` (if applicable):

exxamalte Jun 22, 2018 •

edited

Loading