Skip to content

c-api: do not propagate definition mask from fileref (#102)#132

Open
CarawaySeed42 wants to merge 1 commit into
masterfrom
c-api-do-not-propagate-defmask-from-fileref
Open

c-api: do not propagate definition mask from fileref (#102)#132
CarawaySeed42 wants to merge 1 commit into
masterfrom
c-api-do-not-propagate-defmask-from-fileref

Conversation

@CarawaySeed42

Copy link
Copy Markdown
Contributor

Propagating the definition mask from a filref to the referenced crg file lead to header information being partially ignored. Evaluation results thus differed from the original file.

Resolves: #102

Propagating the definition mask from a filref to the referenced
crg file lead to header information being partially ignored.
Evaluation results thus differed from the original file.

Signed-off-by: Patrick Kuemmerle <patrick.kuemmerle@3d-mapping.de>
@CarawaySeed42 CarawaySeed42 added this to the v.2.1.0 milestone Jun 7, 2026
@CarawaySeed42 CarawaySeed42 requested a review from a team June 7, 2026 02:51
@CarawaySeed42

Copy link
Copy Markdown
Contributor Author

I could not find any drawbacks in my testing.
But without proper and pre-defined tests, there might be an edge case were this change might not produce the desired results. On the other hand, this fixes the usual case of how filerefs are used

@CarawaySeed42 CarawaySeed42 added the isState:ReadyforCCBreview CCB will review it and change the status to ReadyForMerge if everything is ok label Jun 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

isState:ReadyforCCBreview CCB will review it and change the status to ReadyForMerge if everything is ok

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inconsistent Evaluation Results for Referencing .crg Files in OpenCRG

1 participant