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

Shading layer with zero thermal openings #73

Open
vidanovic opened this issue Oct 27, 2020 · 2 comments
Open

Shading layer with zero thermal openings #73

vidanovic opened this issue Oct 27, 2020 · 2 comments

Comments

@vidanovic
Copy link
Collaborator

Shading layer with zero thermal openings is not behaving the same as solid layer for tarcog calculations. As results following code was inserted.
std::shared_ptr
Layers::shading(double thickness,
double conductivity,
EffectiveLayers::EffectiveOpenness effectiveOpenness,
double frontEmissivity,
double frontTransmittance,
double backEmissivity,
double backTransmittance)
{
if(effectiveOpenness.isClosed())
{
return solid(thickness,
conductivity,
frontEmissivity,
frontTransmittance,
backEmissivity,
backTransmittance);
}

        return std::make_shared<CIGUShadeLayer>(
            thickness,
            conductivity,
            std::make_shared<CShadeOpenings>(effectiveOpenness.Atop,
                                             effectiveOpenness.Abot,
                                             effectiveOpenness.Al,
                                             effectiveOpenness.Ar,
                                             effectiveOpenness.Ah,
                                             effectiveOpenness.FrontPorosity),
            std::make_shared<CSurface>(frontEmissivity, frontTransmittance),
            std::make_shared<CSurface>(backEmissivity, backTransmittance));
    }

Check why std::make_shared with zero effective openings is not giving same results as
return solid(thickness,
conductivity,
frontEmissivity,
frontTransmittance,
backEmissivity,
backTransmittance);

@RDmitchell
Copy link

@vidanovic -- how did this come up?

Is this a high priority for THERM 8?

Or can we put it in the bucket for the NFRC (and AERC) fixes for November?

@vidanovic
Copy link
Collaborator Author

Was helping @StephenCzarnecki with his part. That is how we came to this.

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

No branches or pull requests

2 participants