Circuit sharing and pausing for Panels project TEA #119

JingWang-CUB · 2024-04-26T21:18:58Z

Reference the issue your PR is fixing
Circuit sharing and pausing control for panels project TEA #118
Assign at least 1 reviewer for your PR
Test with run_dwelling.py or other script
Update documentation as appropriate
Update changelog as appropriate

jmaguire1 · 2024-04-26T21:22:03Z

bin/run_multiple_panels.py

+# weather_path = os.path.join('path', 'to', 'weather_files')
+weather_path = os.path.join(os.getcwd(), 'ResStockFiles', 'Weather')
+
+end_use_names=['HVAC Heating Electric Power (kW)', 'HVAC Cooling Electric Power (kW)', 'Water Heating Electric Power (kW)',


Do we need pool/spa pumps/heaters on this list? I was leaning towards not including those because our model isn't up to snuff, let me confirm with LBNL next time we talk.

The only place this is used is to calculate the whole home amperage, where I think it deserves a discussion whether to monitor whole home power or whole home current. I think we might have discussed this last year but it's worth revisiting.

In that case, don't we need to include all the uncommon electrical loads? Things like well pumps and freezers are only in a small % of homes but definitely affect whole home power.

If you don't already have it I can get you a list of all the ones that'd come up in ResStock.

Yes, we should include all loads to make this general. I included these based on what loads there were for the Vermont homes that we simulated last year, but it's not an exhaustive list.

Here's everything that can show up but is in a small # of homes:

Extra Refrigerator
Freezer
Hot Tub Spa
Pool Heater
Pool Pump
Well Pump

If it's helpful I can also generate a .xml file for a home with all of these for testing through BEopt.

There's a few uncommon gas loads as well, like gas lighting. Not relevant here but just FYI.

…el-tea

jmaguire1

Looks great! Have we tested this at all? I'd be curious to see the results from a few test cases:

Circuit pausing where we had multiple pauses/year
Circuit sharing where CD is being paused.

jmaguire1 · 2024-07-11T22:16:32Z

bin/run_equipment.py

    # run_battery_controlled()
    # run_water_heater()
    # run_hvac()
-    # run_ev()
+    for i in range(1,2):


@mnblonsky: Do you think it's useful to have an example of running level 1 and level 2 charging, or should we revert/comment this out if we're eventually going to merge this? Along with some other EV changes above.

jmaguire1 · 2024-07-11T22:20:58Z

bin/run_multiple_panels.py

+        with Pool(n_parallel) as p:
+            p.starmap(run_single_building, map_args)
+
+    my_print('All processes finished, exiting.')


@mnblonsky: It might be useful to at least have examples of running multiple OCHRE runs locally? I know Andrew P was having to figure this out on his own too. Not like it's super hard but it's a common enough use case I'd like to package this into an example file (maybe not so project specific, we might have to modify this slightly and split it into it's own file for example purposes).

jmaguire1 · 2024-07-12T20:52:25Z

bin/run_multiple_panels.py

+from multiprocessing import Pool
+import pandas as pd
+
+from ochre import Dwelling, Analysis, ElectricVehicle


In general I think we should split this across multiple files, especially if we want to merge this into OCHRE development. I'll call out specifically how I'd refactor this below, but: none of those reorg changes are required before we start running, it's a longer term change.

jmaguire1 · 2024-07-12T20:54:55Z

bin/run_multiple_panels.py

+    my_print('All processes finished, exiting.')
+
+
+def run_multiple_local(main_folder, overwrite='False', n_parallel=1, n_max=None, *args):


This would be super useful to include as an example file, setting up multiple local runs. External people can't rely on HPC like we can.

This function was written by Michael and I also find it very useful. I use this to test the controllers locally.

jmaguire1 · 2024-07-12T20:58:00Z

bin/run_multiple_panels.py

+
+
+    # create OCHRE building based on der type
+    if der_type == 'ev':


Should we be loading existing EV runs to save time, rather than regenerating EV load profiles on the fly?

I also worry we might add/drop buildings from the analysis at some point and that'll mess up which home gets which EV across multiple runs. My preference would be to pregenerate EV load profiles and assign different buildings different EV profiles in a .csv so we can be sure it stays consistent across any runs we do for this project.

Yes we could use pregenerated EV profiles. This work was delayed/canceled due to lack of budget. We can re-open it if needed.

jmaguire1 · 2024-07-12T21:01:13Z

bin/run_multiple_panels.py

+    return dwelling
+
+
+def circuit_sharing_control(sim_type, dwelling, tech1, tech2, output_path):   


I'd love to split circuit sharing and circuit pausing into their own separate file in bin. (or @mnblonsky might have a better suggestions for where it should live). But this is absolutely something that's going to be useful for other projects and I'd like to get it into develop.

jmaguire1 · 2024-07-12T21:05:50Z

bin/run_multiple_panels.py

+
+    control_signal = None
+
+    if tech2 == 'Clothes Dryer':


This is all we need for this project, but: theoretically could we generalize this to other appliance end uses? I guess if it uses water as well as power that might get more complicated, but longer term it'd be nice to have this work for as many devices as possible.

If they are all cycle-based and follow the same control algorithm then we can expand this to more devices.

jmaguire1 · 2024-07-12T21:10:25Z

bin/run_multiple_panels.py

+        house_status = dwelling.update(control_signal=control_signal)
+
+        # get primary load power
+        if tech1 in ['Strip Heat']:


We excluded this, but good to have this capability. I think it's much harder to do in practice than it is to do in simulation.

jmaguire1 · 2024-07-12T21:18:09Z

One more thought @JingWang-CUB: Do we have all the metrics we need for unmet EV load? For water heater and HVAC we track unmet loads, and can look at the difference between baseline and our run. I'm not sure we have something similar for EVs though? Have you looked for this/done any test runs to check?

JingWang-CUB · 2024-07-17T19:23:15Z

One more thought @JingWang-CUB: Do we have all the metrics we need for unmet EV load? For water heater and HVAC we track unmet loads, and can look at the difference between baseline and our run. I'm not sure we have something similar for EVs though? Have you looked for this/done any test runs to check?

I have looked at EV SOC at departure as a discomfort metric. This can be done as part of the post-processing. Is this something you're thinking of?

mnblonsky · 2024-07-17T20:46:55Z

One more thought @JingWang-CUB: Do we have all the metrics we need for unmet EV load? For water heater and HVAC we track unmet loads, and can look at the difference between baseline and our run. I'm not sure we have something similar for EVs though? Have you looked for this/done any test runs to check?

We have EV unmet load already. It's based on the departure SOC being lower than it would have been given immediate charging.

…ideal capacity for WH and some HVAC

JingWang-CUB · 2024-08-13T21:22:46Z

ochre/Equipment/HVAC.py

@@ -187,6 +187,8 @@ def __init__(self, envelope_model=None, use_ideal_capacity=None, **kwargs):
        if use_ideal_capacity is None:
            use_ideal_capacity = self.time_res >= dt.timedelta(minutes=5) or self.n_speeds >= 4
        self.use_ideal_capacity = use_ideal_capacity
+        if self.n_speeds >= 4:
+            self.use_ideal_capacity = True


@jmaguire1 , should we take out this here as well or is it ok to keep it?

No, we should keep this as is. All variable speed equipment actually is capable of modulating like "use_ideal_capacity" works.

…ngWang-CUB

…ngWang-CUB for helping me find this!

This reverts commit fccbfc8.

This reverts commit f099d2a. We need the tempering valve for the time being. We can more formally integrate this later, but since in our test cases WH setpoint is 140F with a tempering valve set to 125 F this is necessary for the current runs.

Run multiple script for panels project, runnable initial version

5d56a12

JingWang-CUB linked an issue Apr 26, 2024 that may be closed by this pull request

Circuit sharing and pausing control for panels project TEA #118

Open

jmaguire1 self-requested a review April 26, 2024 21:20

jmaguire1 added enhancement New feature or request Medium Priority labels Apr 26, 2024

jmaguire1 reviewed Apr 26, 2024

View reviewed changes

JingWang-CUB and others added 5 commits May 7, 2024 13:39

Script for running multiple EV profiles

5c644a8

updating EV inputs for randomization

68c192b

Merge branch 'ev-batt-bugs' of https://github.com/NREL/OCHRE into pan…

117ab8c

…el-tea

Resolve conflicts

b956e61

Add tested circuit sharing and pausing control code

2fb64c8

jmaguire1 reviewed Jul 12, 2024

View reviewed changes

JingWang-CUB and others added 15 commits July 18, 2024 16:52

include csv containing all control scenarios and panel sizes

6eca453

force to use ideal capacity for variable speed HVAC devices

594ae80

1. change timestep 2. add timestamp of dryer interruption 3. not use …

9502348

…ideal capacity for WH and some HVAC

Add scenarios list w/o duplicated building ids

f16323e

Changes to run control scenarios from reading a file

88279b8

align end use names with OCHRE

14d1358

resolve conflict

4ac85b2

change level2 EV charger power

6ecf4f2

ingore "hot_water_showers" schedule (for now)

ea1e91e

merge dev

3ad06c8

Initial creation of 120V HPWH model

3d6916c

parse 120V HPWH based on temp logic in HPXML for panels branch

a5cac45

Trying to add a tempering valve, still some hard coded things for panels

212c4df

More 120V HPWH changes, update test run

0ba3e01

Merge remote-tracking branch 'remotes/origin/dev' into 120V_hpwh

314e900

remove unnecessary equipment auguments

78df804

JingWang-CUB commented Aug 13, 2024

View reviewed changes

JingWang-CUB and others added 28 commits August 13, 2024 16:03

remove simulation_name argument when initializing dwelling

75c8a9b

update output path

acbda73

bug fix for accumulating draw_total values

ce59ce5

Issue with the tempering valve that accumulated hot draws. Thanks @Ji…

1174c46

…ngWang-CUB for helping me find this!

add spreadsheet for timed out circuit sharing scenarios

fccbfc8

Revert "add spreadsheet for timed out circuit sharing scenarios"

c5b150a

This reverts commit fccbfc8.

add right timedout spreadsheet

20f697f

replace spreadsheet

8dd4a52

change some end use power to avoid unrealistic peaks

a6578ad

code for postprocessing results

09f4b3a

update log file

93acd03

Add in flat garage roofs

cc3fc99

Match roof insulation properties to .idf files

c08345a

add EV controller

9650140

Merge branch 'panel-tea' of https://github.com/NREL/OCHRE into panel-tea

fd375ab

test sh file for sbatch test run

8592674

adding shower draw for unmet loads

b781132

moved some low power HPWH inputs to hpxml.py

49350c6

set default tempering valve to off

f099d2a

merge 120V_hpwh

03f2a8a

uncomment to use hot_water_showers

90d12f2

prepare for HPC run

9aa89c3

Merge remote-tracking branch 'origin/120V_hpwh' into panel-tea

344fea7

One more tempering valve fix

aeb1c72

Merge remote-tracking branch 'origin/120V_hpwh' into panel-tea

ef5f15f

default the tempering valve to t_set

41cfdc3

Merge remote-tracking branch 'origin/120V_hpwh' into panel-tea

a7ed790

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Circuit sharing and pausing for Panels project TEA #119

Circuit sharing and pausing for Panels project TEA #119

JingWang-CUB commented Apr 26, 2024 •

edited

Loading

jmaguire1 Apr 26, 2024

JingWang-CUB Apr 26, 2024

jmaguire1 Apr 26, 2024

JingWang-CUB Apr 26, 2024

jmaguire1 Apr 26, 2024

jmaguire1 left a comment

jmaguire1 Jul 11, 2024

jmaguire1 Jul 11, 2024

jmaguire1 Jul 12, 2024

jmaguire1 Jul 12, 2024

JingWang-CUB Jul 17, 2024

jmaguire1 Jul 12, 2024

jmaguire1 Jul 12, 2024

JingWang-CUB Jul 17, 2024

jmaguire1 Jul 12, 2024

jmaguire1 Jul 12, 2024

JingWang-CUB Jul 17, 2024

jmaguire1 Jul 12, 2024

jmaguire1 commented Jul 12, 2024

JingWang-CUB commented Jul 17, 2024 •

edited

Loading

mnblonsky commented Jul 17, 2024

JingWang-CUB Aug 13, 2024

jmaguire1 Aug 13, 2024

		my_print('All processes finished, exiting.')


		def run_multiple_local(main_folder, overwrite='False', n_parallel=1, n_max=None, *args):



		# create OCHRE building based on der type
		if der_type == 'ev':

		return dwelling


		def circuit_sharing_control(sim_type, dwelling, tech1, tech2, output_path):

Circuit sharing and pausing for Panels project TEA #119

Are you sure you want to change the base?

Circuit sharing and pausing for Panels project TEA #119

Conversation

JingWang-CUB commented Apr 26, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jmaguire1 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jmaguire1 commented Jul 12, 2024

JingWang-CUB commented Jul 17, 2024 • edited Loading

mnblonsky commented Jul 17, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JingWang-CUB commented Apr 26, 2024 •

edited

Loading

JingWang-CUB commented Jul 17, 2024 •

edited

Loading