Tcv get ids core profiles add e field
As part of an ids request, we need to provide profiles_1d/grid/e_field/parallel.
There is an ongoing discussion about how best to calculate this parameter. From email exchange "Fwd: TCV shots for DMP":
Luke Simons Olivier Sauter; Alessandro Pau; Süleyman Cenk Yildiz; Joan Decker; Michele Marin
I think we need to have a discussion about it. But from talking with Joan, we think (the field core_profiles.e_field.parallel), should be informed by reconstruction of the Efield parallel profile using Liuqe based on Vloop measurements. Not sure if that's what you mean by Vloop profile.
Cheers,
Luke
See redmine issue.
Merge request reports
Activity
assigned to @mmarin
added 1 commit
- 358d8c1c - Added ids for core_profiles.profiles_1d.e_field.parallel
requested review from @mmarin
- Resolved by Luke Simons
99 99 global_quantities_desc.ip = params_eff.data_request; 100 100 101 101 params_eff.data_request = 'ohm_data'; 102 102 current_non_inductive = gdat(params_cores_profiles.shot,params_eff); changed this line in version 4 of the diff
added 1 commit
- 73dafc4e - Changed current_inductive to current_non_inductive
added 1 commit
- 3f8b662c - Changed the calculation of current_non_inductive to be jtotal-johm-jbsav
added 1 commit
- 30ab0fb3 - Correcting currents and e-field. Still to remap from parallel to toroidal
When I run an example at the moment, I'm getting:
shot=[59151]; idss_bolo = gdat(shot,'ids','source','core_profiles'); Error in tcv_get_ids_core_profiles (line 394) global_quantities.current_bootstrap.bs.bs_data.cd_dens.data(:,it);
I think this is cause the data in global quantities hasn't been interpolated. Still may need toroidal to parallel transform
Yeah this is connected to issue !190 (merged) . I need to make the removal of Nans optional and then I should be able to merge. Use another shot to test
added 85 commits
-
30ab0fb3...a39c66b1 - 84 commits from branch
master
- f5753df2 - Merge branch 'master' into tcv_get_ids_core_profiles_add_e_field
-
30ab0fb3...a39c66b1 - 84 commits from branch
When you want! We should merge this one and then review, merge and test !190 (merged). A couple of lines might need to be changed there to accommodate for the changes here