mvrdevelopment / spec

DIN SPEC 15800 General Device Type Format (GDTF) and My Virtual Rig (MVR) File Format description DIN SPEC 15801
64 stars 13 forks source link

GDTF files edited and provided via MVR #146

Closed petrvanekrobe closed 1 year ago

petrvanekrobe commented 1 year ago

1) When a GDTF file is modified and exported to GDTF, often there is no revision - indicating this change - added, thus the user may think that the file is the original one as provided (by the manufacturer). Revisions are optional and at this point are only enforced upon uploading to the Share. We should add a mandatory revision requirement to make it clear that the file has been modified.

2) Shouldn't the GDTFSpec → FileName be more closely aligned to the GDTF counterpart? https://github.com/mvrdevelopment/spec/blob/main/gdtf-spec.md#file-format-definition . To the point above, the revision of the file is suggested to be part of the file name.

3) GDTFSpec → FileName does not specify that the extension must be part of the FileName. We have seen issues in the past with GDTF resource files which used upper case or mixed upper/lower case in the extension. It would be good to remove this ambiguity, by requiring the extension to be part of FileName.

4) General GDTF quality issues caused by modifications and subsequent export to MVR:

This is more on quality/process which is hard to capture and enforce, yet the user experience is strongly downgraded if that happens and we should put emphasis on ensuring that data quality is protected.

moritzstaffel commented 1 year ago

Shouldn't the GDTFSpec → FileName be more closely aligned to the GDTF counterpart? https://github.com/mvrdevelopment/spec/blob/main/gdtf-spec.md#file-format-definition . To the point above, the revision of the file is suggested to be part of the file name.

Already by the spec

moritzstaffel commented 1 year ago

GDTFSpec → FileName does not specify that the extension must be part of the FileName. We have seen issues in the past with GDTF resource files which used upper case or mixed upper/lower case in the extension. It would be good to remove this ambiguity, by requiring the extension to be part of FileName

Day by the spec already