Skip to content

IVS-430 - Header multiple MVDs - #339

Merged
Ghesselink merged 5 commits into
developmentfrom
IVS-1030-header-multiple-mvd
Oct 30, 2026
Merged

Ghesselink merged 5 commits into
developmentfrom
IVS-1030-header-multiple-mvd

Conversation

@Ghesselink

@Ghesselink Ghesselink commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor
  • All ViewDefinition names count, in one bracket or in separate strings.
  • Add-on views don't count as a base MVD.
  • More than one distinct base MVD is flagged; a duplicate name too
  • A second FILE_DESCRIPTION already fails the syntax check (tested)

@Ghesselink
Ghesselink requested a review from civilx64 October 29, 2026 18:28
@Ghesselink
Ghesselink requested a review from aothms October 29, 2026 18:31
@aothms

aothms commented Oct 29, 2026

Copy link
Copy Markdown
Collaborator

Ok, I read the guide again: https://standards.buildingsmart.org/documents/Implementation/ImplementationGuide_IFCHeaderData_Version_1.0.2.pdf

Apparently multiple strings are supported, also because they are limited to 256 chars (although that's a limit I think we can also choose not to enforce)

Not sure about the "duplicate name counts once" - I would prefer to flag that as well, just because it is silly.

@Ghesselink

Copy link
Copy Markdown
Contributor Author

Not sure about the "duplicate name counts once" - I would prefer to flag that as well, just because it is silly.

I've added it, also to the related MVD rules (open since today) buildingSMART/ifc-gherkin-rules#558)

@Ghesselink
Ghesselink merged commit eaf5bdf into development Oct 30, 2026
3 checks passed
@Ghesselink
Ghesselink deleted the IVS-1030-header-multiple-mvd branch October 30, 2026 12:45
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

Successfully merging this pull request may close these issues.

2 participants