pyActigraphy icon indicating copy to clipboard operation
pyActigraphy copied to clipboard

Add support for Italian RPX files

Open ugGit opened this issue 2 years ago • 3 comments

This PR adds support for RPX files which were exported in Italian as suggested in issue https://github.com/ghammad/pyActigraphy/issues/136. This extends the existing languages English, French, and German.

ugGit avatar Aug 22 '23 09:08 ugGit

I think, it would be great to add another test case. However, I suspect that it might be good to aim for consistency with respect to the test files. Therefore, it would be great if @ghammad could probably generate the file in the same manner as the other RPX files used for tests.

ugGit avatar Aug 22 '23 09:08 ugGit

I think, it would be great to add another test case. However, I suspect that it might be good to aim for consistency with respect to the test files. Therefore, it would be great if @ghammad could probably generate the file in the same manner as the other RPX files used for tests.

@ugGit : could you send an Italian file example you use? In addition, we still need to figure out how "Mobility" and "S/W Status" are translated in Italian and include these translations in the dict.

ghammad avatar Oct 06 '23 08:10 ghammad

Here you go for the file: example_it.csv

The thing with the Mobility and S/W Status is that they are not listed in the columns in my examples. The same already seem to have applied when somebody added support for the german language. Thus, I did not bother me too much.

ugGit avatar Oct 16 '23 09:10 ugGit