Label mappings returns different structures depending on a csv being specified

[perellonieto]

Both functions label_mappings and label_mappings_localisation return different formats depending on a CSV being specified during the instantiation of a Wrapper. I would opt to return always the same format, and convert it to the required format if necessary while exporting the CSV file, however, I am not familiar with the code and I am not sure which would be the more generic format, and the implications that this may have.

@mkoz71 could you help me with this decision?

Also, this may be related to the issue #13

bHealth/bhealth/metric_wrappers.py

Line 531 in df5dadb

def label_mappings(self, container, is_duration, label_to_extract=None):

[mkoz71]

Hi Miquel,

You are right. This particular function I wrote quickly to incorporate the csv saving functionality. What you are saying makes more sense.

So long as you check that the csv is specified AND you change the format accordingly, the code should still run without issues.

Michal

[perellonieto]

I modified the code to return always the dictionaries with the specified values, this needs to be then modified while exporting the csv files.