Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new comfort region: Comfort Zone of the Building Bioclimatic Chart #15

Merged
merged 4 commits into from
Aug 9, 2024

Conversation

raph-topo
Copy link
Contributor

@raph-topo raph-topo commented Aug 6, 2024

Based on Milne and Givoni 1979 & ASHRAE 55-2017

Centered on PMV for:

  • Summer conditions:
    air velocity = 0.20 m/s
    clothing level = 0.60 clo (light to moderate clothes)
    metabolic rate = 1.25 met (seated with light activity)
    mean radiant temperature = 26°C (warm)

  • Winter conditions:
    air velocity = 0.20 m/s
    clothing level = 1.20 clo (long clothes)
    metabolic rate = 1.25 met (seated with light activity)
    mean radiant temperature = 19°C (normal)

@raph-topo raph-topo mentioned this pull request Aug 6, 2024
Based on Milne and Givoni 1979 & ASHRAE 55-2017.

Centered on PMV for:

- Summer conditions:
air velocity = 0.20 m/s
clothing level = 0.60 clo (light to moderate clothes)
metabolic rate = 1.25 met (seated with light activity)
mean radiant temperature = 26°C (warm)

- Winter conditions:
air velocity = 0.20 m/s
clothing level = 1.20 clo (long clothes)
metabolic rate = 1.25 met (seated with light activity)
mean radiant temperature = 19°C (normal)
@@ -100,8 +100,8 @@ export const plugin = new PanelPlugin<PsyOptions>(PsyPanel).setPanelOptions((bui
})
.addMultiSelect({
path: 'regions',
name: 'ASHRAE Comfort Regions',
Copy link
Collaborator

Choose a reason for hiding this comment

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

All the regions shown are developed by ASHRAE, but I think this is fine to remove, in case we include other standards.

@6nv 6nv self-assigned this Aug 8, 2024
@6nv
Copy link
Collaborator

6nv commented Aug 8, 2024

Excellent PR! I have no idea why the CI is failing. I just ran it on one of my old commits, and it also broke there. I figure there might be some issue with Grafana's docker image. I just made a post in Grafana's user Slack channel, and I'll merge this as soon as I hear a reply.

@6nv
Copy link
Collaborator

6nv commented Aug 9, 2024

CI was fixed here: grafana/plugin-tools#1046

@6nv 6nv merged commit 97f39fb into nicfv:main Aug 9, 2024
3 of 7 checks passed
@6nv
Copy link
Collaborator

6nv commented Aug 9, 2024

Update: Psychart v4.5.0 is waiting for review on Grafana.com. psychart.nicfv.com is already live with the new version!

@raph-topo
Copy link
Contributor Author

Thank you!

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