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

idfobject.extensible_500 = xxx where extensible_500 extends beyond the fields in the IDD #395

Closed
santoshphilip opened this issue Oct 9, 2022 · 1 comment
Assignees

Comments

@santoshphilip
Copy link
Owner

idfobject.extensible_500 = xxx where extensible_500 extends beyond the fields in the IDD

Consider the idfobject windowmaterial

print(windowmaterial)

WINDOWMATERIAL:GLAZINGGROUP:THERMOCHROMIC,
    Gumby,                    !- Name
    1,                        !- Optical Data Temperature 1
    G1,                       !- Window Material Glazing Name 1
    2,                        !- Optical Data Temperature 2
    G2;                       !- Window Material Glazing Name 2

windowmaterial.Optical_Data_Temperature_5000 = 55

Will throw the exception BadEPFieldError because the IDD does not extend to 5000 variables
The last variable in the IDD is windowmaterial.Optical_Data_Temperature_45

eppy should tranparently extend the variable in the IDD to the right amount

@santoshphilip santoshphilip self-assigned this Oct 9, 2022
santoshphilip pushed a commit that referenced this issue Oct 9, 2022
:Problem: dunder of setattr, getattr, setitem, getitem fail for an extensible field that is not in the IDD
:Solution: updated the dunders to extend the fields in IDD that is in eppy's memory

dunder = double underscore such as __setattr__
santoshphilip pushed a commit that referenced this issue Oct 9, 2022
:Problem: dunder of setattr, getattr, setitem, getitem of bunch_subclass fail for an extensible field that is not in the IDD
:Solution: updated the dunders to extend the fields in IDD that is in eppy's memory

dunder = double underscore such as __setattr__

commit message corrected for clarity. Previous message did not mention bunch_subclass
@santoshphilip
Copy link
Owner Author

fixed issue #395

:Problem: dunder of setattr, getattr, setitem, getitem fail for an extensible field that is not in the IDD
:Solution: updated the dunders to extend the fields in IDD that is in eppy's memory

dunder = double underscore such as __setattr__

santoshphilip pushed a commit that referenced this issue Nov 9, 2022
    modified:   HISTORY.rst
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

1 participant