start fixing complete_IDS so can ask questions on gitlab...

avoid time at 2 places

matlab/IMAS/complete_IDS_CORSICA.m

@@ -65,6 +65,9 @@ end
 tmp = data_circuits();
 Ncircuits = size(tmp,1);
 Ncoils = numel(IDS_out.pf_active.coil);
+% time will be set in circuit.current.time so should not put it at top and set homogeneous to 0?
+IDS_out.pf_active.time = [];
+IDS_out.pf_active.ids_properties.homogeneous_time = 0;
 for ii=1:Ncircuits
     IDS_out.pf_active.circuit{ii}.name = tmp{ii,1}{1};
     IDS_out.pf_active.supply{ii}.name = tmp{ii,1}{1};

[Olivier Sauter]

@carpanes Seems pf_active.time (51 points) is the same as pf_active.coil{index_coil(1)}.current.time so if set in coil should remove from top and set homogeneous to 0. OK?

[Francesco Carpanese]

At a first glance I would say that all the currents share the same time, so we could also put homogeneous = 1 , and provide the time directly there, removing the other one. However, the field pf_active.coil{ii}.current.data and pf_active.coil{ii}.current.time was filled this way by CORSICA data originally so we might not want to change remove the original data.

[Olivier Sauter]

Yes better keep local tim/data