architecture-building-systems / CityEnergyAnalyst

The City Energy Analyst (CEA)
https://www.cityenergyanalyst.com/
MIT License
195 stars 65 forks source link

Correct electromobility explanation #2757

Closed AlexJew closed 4 years ago

AlexJew commented 4 years ago

Describe the bug

Curently, the variable _EvkWveh is erroneously explained in the Input Editor. In the glossary, there isn't any information. This is important, as currently there is little explanation about the electric mobility feature, when I believe it represents an increasingly important feature.

To Reproduce

Steps to reproduce the behavior:

  1. Go to Input Editor
  2. Select internal-loads
  3. Go to _EvkWveh column label

Expected behaviour

I believe a correct explanation would be: "Effective electric load (rated power) per vehicle when charging [kW/vehicle]"

In such cases, I believe a good default value would be 3,7 kW, a typical value for domestic electric car chargers. It corresponds to a single phase (RMS voltage = 230 V) at 16A

Screenshots

What is currently shown in the app:

Screen Shot 2020-06-04 at 15 45 35

Hardware (please complete the following information):

daren-thomas commented 4 years ago

@AlexJew thank you for pointing this out! This kind of attention to detail really helps us improve the CEA :)

@JIMENOFONSECA can you look into this please? Might be as simple as editing one field in schemas.yml and also updating the databases (if we really want to change the default value)...

jimenofonseca commented 4 years ago

@daren-thomas sure, easy to solve, I made a PR #2768 for this. I hope this solves the issue.