Skip to content

Simplified heat tests - #3484

Open
jort-wolda wants to merge 3 commits into
masterfrom
simplified-heat-tests
Open

Simplified heat tests#3484
jort-wolda wants to merge 3 commits into
masterfrom
simplified-heat-tests

Conversation

@jort-wolda

@jort-wolda jort-wolda commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Context

Due to simplified sliders in built environment the old spec was deleted. These are queries for the new spec.

Implemented changes

  • Made new turk_... queries that refer to existing general queries
  • Made 1 new general query
  • Deleted turk_hhp_... queries that were not used anywhere

Related

Goes with pull requests

Checklist

  • I have tested these changes
  • I have updated documentation as needed
  • I have tagged the relevant people for review

@jort-wolda
jort-wolda marked this pull request as ready for review September 1, 2026 09:05
@jort-wolda
jort-wolda requested a review from kndehaan September 1, 2026 09:05
@kndehaan
kndehaan force-pushed the simplified-heat-tests branch from 31f8b13 to 395af4d Compare September 2, 2026 12:44

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this new query required? Can the existing query costs_building_and_installations_households_insulation.gql be used for the same purpose?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Existing query also divides the costs by the technical lifetime resulting in CAPEX/year. I thought it would be more 'clean' if it only used the CAPEX but you're right it is not necessary. I deleted them and now refer to the existing queries.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same question here, can the existing costs_building_and_installations_buildings_insulation.gql query be used instead of adding a new query?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Idem

And refer to the existing ones
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants