search

inab / benchmarking-data-model

OpenEBench Benchmarking Data Model repository
Creative Commons Attribution Share Alike 4.0 International
2 stars 7 forks source link

TestAction "_metadata" vs "metadata" properties #113

Closed redmitry closed 3 years ago

redmitry commented 4 years ago

Hello,

There are two metadata properties in the TestAction. Is this supposed?

Cheers,

D.

jmfernandez commented 4 years ago

If you read the description of them, they have very different purposes. _metadata is generic to almos all the entries, and metadata is specific to TestAction. But it is true their very similar names can mislead the people.

So, the most specific one should either disappear or be renamed. What do you think @javi-gv94 @jlgelpi @scapella ?

javi-gv94 commented 4 years ago

if we had a clear purpose for "metadata", we should rename it. But I think it is not the case - we haven't used it so far, so I suggest to remove it an move its content (Relevant metadata used to call the tool) to "_metadata".

jlgelpi commented 4 years ago

+1

jmfernandez commented 3 years ago

Removed metadata from TestAction in c4b3808bb4be8bb3ed6416ab4cc9eab4ce70f298