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

ToDos/Notes for template #21

Open
1 task done
gabroko opened this issue Apr 13, 2024 · 5 comments
Open
1 task done

ToDos/Notes for template #21

gabroko opened this issue Apr 13, 2024 · 5 comments

Comments

@gabroko
Copy link
Contributor

gabroko commented Apr 13, 2024

  • allow only predefined values for Type of building in META home (see definition)
@retostauffer
Copy link
Collaborator

retostauffer commented Apr 13, 2024

@gabroko

Implemented in version 0.2-13 (current main branch on github)

  • Updated XLSX template, "Type of building" colored in dark gray and added data validation (dropdown) to only allow for the values specified in the Definitions table. Leaving it empty is not an option.
  • annex_validate() checks if this column is filled and throws warnings otherwise.
    • Note: annex_validate() does not test if the content of the cells is one of the allowed types, Excel is doing that. This is the same as we use for other variables (e.g., Ventilation type).

@gabroko
Copy link
Contributor Author

gabroko commented Apr 13, 2024

Thanks! But actually it isn't a mandatory field, so no coloring in dark gray and no warning needed (only a note during the valodate). Just like for the field Method of vent. rate determination, here we also have predefined values but not mandatory.
Yes, if excel checks, that is certainly enough... No big deal, if it stays a while like that, but for the next change...

@retostauffer
Copy link
Collaborator

@gabroko META-Home "Ventilation type" was already set "mandatory" and annex_validate() will warn the user if the cells are empty, thus I thought "Type of building" should be the same, thus my implementation.

If I understand you correctly we should make both "non-mandatory" and not check the content? Excel will still provide the dropdown where the users can set one of the pre-specified values - or nothing.

Just let me know what you prefer and I'll quickly adjust it - easy fix :).

@gabroko
Copy link
Contributor Author

gabroko commented Apr 13, 2024

Yes, Ventilation type should stay mandatory, but Type of building is not mandatory, but if filled out it should adhere to the standard nomeclature as defined in Definitions shhet... Thanks :-)

@retostauffer
Copy link
Collaborator

Done; data 'validation' is done by Excel (dropdown menu with data validation), annex_validate() will not check the content and allows them to be empty (non-mandatory).

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