Zenmo / HOLON

Model made for HOLON project
MIT License
2 stars 0 forks source link

Feature casus2 output #243

Closed petertue closed 1 year ago

petertue commented 1 year ago

Lokale mergen met main is paar keer mislukt. Dus ik kijk of het via deze route kan. Dan kan ik ook weer van mijn branch af.

In principe zijn er 2 updates in mijn branch:

GillisHommen commented 1 year ago

Lokale mergen met main is paar keer mislukt. Dus ik kijk of het via deze route kan. Dan kan ik ook weer van mijn branch af.

In principe zijn er 2 updates in mijn branch:

* diesel vehicles met een verbruik van 0.08 wordt naar 0.8 gezet

* Terraced houses geven terug of ze geisoleerd hebben (tbv ETM query)

Waarom zet je het dieselverbruik van de voertuigen niet direct in de diesel_vehicle energy asset? Nu is het hardcoded als ik het goed begrijp?

petertue commented 1 year ago

Deze merge request is al gesloten. Misschien dat er daarom iets mis gaat?

Omdat het tijdelijk hardcoded is tot er het datamodel/fixture weer mogen en kunnen aanpassen. Alle hardcoded dingen staan ongeveer op zelfde plek. Daarom niet in de J_asset gezet.

GillisHommen commented 1 year ago

Een diesel vehicle asset heeft toch gewoon het attribuut EnergyConsumption kWhpkm, instelbaar in het cms? https://holontool.nl/wt/admin/holon/dieselvehicleasset/900/change/?_changelist_filters=p%3D4

petertue commented 1 year ago

Ja pas jij dat op de pizzaoven aan bij mijn veertig huishoudens en de 240 van Jorrit, terwijl het eigenlijk niet werkt wat de site loopt vast. En opnieuw als we migreren, en wel de goede vehicle assets in de lijst met 1000 vinden ;)

Dat was de situatie twee weken terug toen we dit op deze manier doorvoerde, met 1 regel code die we tzt weer aan konden passen ;)

GillisHommen commented 1 year ago

En die vervolgens een andere casus in de war gooit. Zou niet de eerste keer zijn...

GillisHommen commented 1 year ago

Met een interactief element kun je ook eenvoudig een attribuut (in dit geval kWhpkm) aanpassen voor al je (diesel-) verhicles.