search

architecture-building-systems / CityEnergyAnalyst

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

fix preprocessing_main to include decentralized_main to avoid shipping of files with reference case #649

Closed BhargavaKrishnaSreepathi closed 7 years ago

BhargavaKrishnaSreepathi commented 7 years ago

Activating decentralized_main to generate 'DiscOP_buildingname_result' files

BhargavaKrishnaSreepathi commented 7 years ago

Hi @JIMENOFONSECA , this is the error I am facing, when I run the optimization_main.py in branch i649-decentralized-buildings-preprocessing. Please help me out . Thanks

File "C:/Users/Bhargava/Documents/GitHub/CEAforArcGIS/cea/optimization/optimization_main.py", line 84, in Run decentralized model for buildings run_as_script(r'C:\reference-case-zug\baseline') File "C:/Users/Bhargava/Documents/GitHub/CEAforArcGIS/cea/optimization/optimization_main.py", line 79, in run_as_script moo_optimization(locator=locator, weather_file= weather_file, gv=gv) File "C:/Users/Bhargava/Documents/GitHub/CEAforArcGIS/cea/optimization/optimization_main.py", line 50, in moo_optimization weather_file, gv) File "C:\Users\Bhargava\Documents\GitHub\CEAforArcGIS\cea\optimization\preprocessing\preprocessing_main.py", line 75, in preproccessing decentralized_buildings.decentralized_main(locator, building_names, gv) AttributeError: 'module' object has no attribute 'decentralized_main'

BhargavaKrishnaSreepathi commented 7 years ago

@JIMENOFONSECA : a follow up question. After running the decentralized_buildings.py, there are files generated corresponding to the number of buildings, but each file has 14 rows, 1 row corresponding to header and 13 (i don't know). The number 13 is hard coded in the code and no explanation is provided for why it is chosen as 13. It says at the start it is a combinatorial problem. Can you give your insights on this.

jimenofonseca commented 7 years ago

So this is something we need to "decipher" form the code. but in essence, it is making a ranking of the best solution out of combining all possible supply systems for decentralized buildings. Such algorithm should be translated into a MILP formulation which could be a great point to get this skills of your on board ; ). I advice to look at that issue and update the decentralized buildings script. Which should be an optimization at the building scale of conversion technologies.

From: BhargavaKrishnaSreepathi [notifications@github.com] Sent: Wednesday, May 17, 2017 3:43 PM To: architecture-building-systems/CEAforArcGIS Cc: Jimeno Fonseca; Mention Subject: Re: [architecture-building-systems/CEAforArcGIS] fix preprocessing_main to include decentralized_main to avoid shipping of files with reference case (#649)

@JIMENOFONSECAhttps://github.com/jimenofonseca : a follow up question. After running the decentralized_buildings.py, there are files generated corresponding to the number of buildings, but each file has 14 rows, 1 row corresponding to header and 13 (i don't know). The number 13 is hard coded in the code and no explanation is provided for why it is chosen as 13. It says at the start it is a combinatorial problem. Can you give your insights on this.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/architecture-building-systems/CEAforArcGIS/issues/649#issuecomment-302012090, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIjrguSBxFbMR3ndje_cuYmCAczbXqZsks5r6qUJgaJpZM4NdS-P.