economidis-nick / createXSDforxMCF

2 stars 2 forks source link

Enable <tangential_direction/> for more connection types. #64

Closed DrCaFr closed 3 years ago

DrCaFr commented 3 years ago

Cited from minutes of 2021-05-20 working group meeting:

  • So far, the attribute tangential direction has been defined along with for 0D connection elements but only used for robscans. It was observed that rivets can also be non-axisymmetric and hence it was requested to enable both attributes for all 0D connection types.
  • It was agreed by the working group.
DrCaFr commented 3 years ago

@economidis-nick: Please add more detail, e.g. examples for rivet types etc.! Are there arguments for assigning it to V3.1.1?

economidis-nick commented 3 years ago

I updated the xmcf document: Allowed <tangentiaL_direction> (and <normal_direction>) for every <connection_0d> type. However, it is conceivable only for <rivet> and <clinch>, apart from <robscan>, <nail> and <clip> where it already existed. Consider 'rejecting' the changes from some subtypes if group does not agree with the change.

DrCaFr commented 3 years ago

@economidis-nick : At <spotweld/>, I found the explanation "If a sense of direction or orientation is required for the spotwelding technique, they can optionally be provided using the and elements.", whereas something similar is missing at the other occasions. There, tangential_direction has been added to the lists of nested elements, only.

Is this intentional?

economidis-nick commented 3 years ago

@DrCaFr , yes. I added this text as an attempt to justify the existence of the 'directions' table under spotweld. But I was not proud of the text so I did not paste it everywhere. If we agree to introduce directions in every connection_0d and we agree to a decent wording, we can paste it where appropriate. Otherwise, if we decide to add directions only to some specific subtypes, a better text could be provided.

DrCaFr commented 3 years ago

Decision:

DrCaFr commented 3 years ago

Issue successfully reviewed. Suggest to include it into v3.1.1 / milestone Draft_PAS_with_new_annex.

DrCaFr commented 3 years ago

Change was accepted by VDA AK 25 meeting on 2021-10-21.