search

dzhvansky / pepato

GNU General Public License v3.0
1 stars 2 forks source link

adjust output files #11

Closed aremazeilles closed 3 years ago

aremazeilles commented 3 years ago

Attempt to add a CI process to validate the code, veryfing that giving the reference input, we get the expected output.

As mentioned in https://github.com/dzhvansky/pepato/issues/9, I committed the output file that I have, which correspond to the following changes:

─anthony@PRT0962AL ~/code/eurobench/pepato ‹ci*› 
╰─$ diff output test_data/output
diff --color output/subject_0_run_0_emg_reco_quality.yaml test_data/output/subject_0_run_0_emg_reco_quality.yaml
2c2
< value: [0.98381, 0.93659, 0.9487]
---
> value: [0.98382, 0.93655, 0.9487]
diff --color output/subject_0_run_0_pattern_coa.yaml test_data/output/subject_0_run_0_pattern_coa.yaml
2c2
< value: [[92.6378, 6.5332, 38.1322, 36.8123, 88.5832], [93.1011, 0.27894, 33.4706, 45.7178, 87.2971], [4.3559, 16.2496, 35.3299, 92.9091, 90.7257]]
---
> value: [[92.6476, 5.2742, 38.1158, 36.5813, 87.8575], [94.2312, 99.5407, 33.5845, 48.1029, 86.6531], [4.3559, 16.2496, 35.3299, 92.9091, 90.7257]]
diff --color output/subject_0_run_0_pattern_fwhm.yaml test_data/output/subject_0_run_0_pattern_fwhm.yaml
2c2
< value: [[19, 16, 32.5, 35, 26.5], [9, 30, 28, 19.5, 29], [15.5, 34, 21.5, 15, 11.5]]
---
> value: [[18.5, 16, 32.5, 37, 25.5], [9, 30.5, 28, 20, 30.5], [15.5, 34, 21.5, 15, 11.5]]
diff --color output/subject_0_run_0_patterns_similarity.yaml test_data/output/subject_0_run_0_patterns_similarity.yaml
2c2
< value: [[[0.46952, -0.61304, 0.87954, 0.64039], [0.93869, -0.28111, 0.38969, 0.88667], [-0.25053, 0.95214, -0.70183, -0.51035], [0.10085, 0.54473, -0.6915, -0.27004], [0.1191, -0.45409, 0.77165, 0.34175]], [[0.68391, -0.4158, 0.82883, 0.024115], [0.8097, -0.38895, 0.34239, 0.64038], [-0.24222, 0.95568, -0.61256, -0.46461], [-0.3614, 0.50409, -0.43918, -0.44096], [-0.056503, -0.43976, 0.46824, 0.71175]], [[0.96272, -0.19, 0.39977, 0.1049], [-0.094108, 0.47302, -0.30215, 0.16371], [-0.29254, 0.96314, -0.4772, -0.37951], [0.38344, -0.38879, 0.26328, 0.96102], [0.69427, -0.43879, 0.94155, 0.12186]]]
---
> value: [[[0.45183, -0.58407, 0.87952, 0.62441], [0.9301, -0.33932, 0.44498, 0.90376], [-0.24891, 0.95165, -0.70346, -0.50956], [0.11404, 0.53947, -0.69225, -0.25796], [0.079566, -0.4328, 0.75243, 0.3004]], [[0.7191, -0.42046, 0.81787, 0.032704], [0.75209, -0.40221, 0.33518, 0.71035], [-0.2513, 0.95184, -0.64022, -0.45555], [-0.38971, 0.41522, -0.34307, -0.41223], [-0.057862, -0.44623, 0.51404, 0.65396]], [[0.96272, -0.19, 0.39977, 0.1049], [-0.094108, 0.47302, -0.30215, 0.16371], [-0.29254, 0.96314, -0.4772, -0.37951], [0.38344, -0.38879, 0.26328, 0.96102], [0.69427, -0.43879, 0.94155, 0.12186]]]
diff --color output/subject_0_run_0_synergies_similarity.yaml test_data/output/subject_0_run_0_synergies_similarity.yaml
2c2
< value: [[[-0.20307, -0.31054, 0.75179, -0.18561], [-0.19826, -0.33986, -0.19823, 0.96878], [-0.4307, 0.98286, -0.44353, -0.41714], [0.50763, -0.16744, -0.21778, -0.098392], [-0.30704, -0.23647, 0.51936, 0.10162]], [[-0.11623, -0.3863, 0.83371, -0.090038], [0.34181, -0.31228, -0.30613, 0.21786], [-0.43388, 0.98993, -0.42052, -0.31509], [0.41283, -0.016439, -0.24564, -0.26732], [-0.41964, -0.25837, 0.15012, 0.6803]], [[0.62042, -0.10301, -0.25463, -0.24342], [-0.40109, 0.035689, -0.27831, 0.57156], [-0.16758, 0.91063, -0.43696, -0.44041], [-0.17187, -0.3796, -0.39359, 0.90217], [-0.066135, -0.3468, 0.74966, -0.21439]]]
---
> value: [[[-0.21932, -0.29709, 0.77036, -0.20801], [-0.22844, -0.35739, -0.14998, 0.97363], [-0.42351, 0.98259, -0.46987, -0.39429], [0.50952, -0.1765, -0.21782, -0.087232], [-0.28656, -0.2389, 0.49838, 0.1056]], [[-0.053236, -0.29479, 0.74234, -0.19809], [0.3049, -0.38693, -0.30375, 0.34826], [-0.39735, 0.98658, -0.44122, -0.33735], [0.40064, -0.067904, -0.19446, -0.23391], [-0.4367, -0.23521, 0.17052, 0.6562]], [[0.62042, -0.10301, -0.25463, -0.24342], [-0.40109, 0.035689, -0.27831, 0.57156], [-0.16758, 0.91063, -0.43696, -0.44041], [-0.17187, -0.3796, -0.39359, 0.90217], [-0.066135, -0.3468, 0.74966, -0.21439]]]

Before merging, if you agree, let us wait if the cI process gets correctly triggered.

aremazeilles commented 3 years ago

The CI process has passed, so I think, if you accept the change on the reference outptut files, that we can merge this.