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

Fix Issue #10286 window material shade mis-alignment for Output:Constructions #10750

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/EnergyPlus/WindowManager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -6980,7 +6980,7 @@ namespace Window {
auto const *matShade = dynamic_cast<Material::MaterialShade const *>(mat);
assert(matShade != nullptr);

static constexpr std::string_view Format_703(" WindowMaterial:Shade,,{},{:.3R},{:.3R},{:.3R},{:.3R},{:.3R},{:.3R}\n");
static constexpr std::string_view Format_703(" WindowMaterial:Shade,{},{:.3R},{:.3R},{:.3R},{:.3R},{:.3R},{:.3R}\n");
print(state.files.eio,
Format_703,
matShade->Name,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,8 @@ This file documents the structural changes on the output of EnergyPlus that coul
### Description

This will eventually become a more structured file, but currently it isn't clear what format is best. As an intermediate solution, and to allow the form to be formed organically, this plain text file is being used. Entries should be clearly delimited. It isn't expected that there will be but maybe a couple each release at most. Entries should also include some reference back to the repo. At least a PR number or whatever.

### Output:Constructions for WindowMaterial:Shade
Data for Output:Constructions for WindowMaterial:Shade were misaligned with the column headings in the eio output and the table Initialization Summary report. Data has shifted one column to the left.

See PR [#10750](https://github.com/NREL/EnergyPlus/pull/10750)
Loading