Skip to content

Commit f64efb8

Browse files
authored
chore: remove unnecessary assertions (#22297)
1 parent a2b8c1c commit f64efb8

File tree

2 files changed

+0
-13
lines changed

2 files changed

+0
-13
lines changed

core/site.go

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -539,9 +539,6 @@ func (site *Site) updatePvMeters() {
539539

540540
for i, dev := range site.pvMeters {
541541
meter := dev.Instance()
542-
if _, ok := meter.(api.Meter); !ok {
543-
panic("not a meter: pv")
544-
}
545542

546543
power := mm[i].Power
547544
if power < -500 {
@@ -582,10 +579,6 @@ func (site *Site) updatePvMeters() {
582579
// update solar yield
583580
for i, dev := range site.pvMeters {
584581
// use stored devices, not ui-updated instances!
585-
if _, ok := dev.(config.Device[api.Meter]); !ok {
586-
panic(fmt.Sprintf("not a device: pv %d", i+1))
587-
}
588-
589582
name := dev.Config().Name
590583

591584
if mm[i].Energy > 0 {
@@ -614,9 +607,6 @@ func (site *Site) updateBatteryMeters() {
614607

615608
for i, dev := range site.batteryMeters {
616609
meter := dev.Instance()
617-
if _, ok := meter.(api.Meter); !ok {
618-
panic("not a meter: battery")
619-
}
620610

621611
// battery soc and capacity
622612
var batSoc, capacity float64

core/site_battery.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,6 @@ func (site *Site) requiredBatteryMode(batteryGridChargeActive bool, rate api.Rat
9292
func (site *Site) applyBatteryMode(mode api.BatteryMode) error {
9393
for _, dev := range site.batteryMeters {
9494
meter := dev.Instance()
95-
if _, ok := meter.(api.Meter); !ok {
96-
panic("not a meter: battery")
97-
}
9895

9996
if batCtrl, ok := meter.(api.BatteryController); ok {
10097
if err := batCtrl.SetBatteryMode(mode); err != nil && !errors.Is(err, api.ErrNotAvailable) {

0 commit comments

Comments
 (0)