LBNL-ETA / Windows-CalcEngine

Thermal and optical routines for modeling properties of window and shading systems
Other
4 stars 2 forks source link

Shading layer with zero thermal openings #73

Open vidanovic opened 3 years ago

vidanovic commented 3 years ago

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 commented 3 years ago

@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 commented 3 years ago

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