Discussion on capturing required functionalities within package about hiqga.jl HOT 6 CLOSED

Thanks @yusenley and @NMoghaddam for your feedback. In brief, I will respond here:

• read_survey_files exists for each AEM system we can currently handle out of the box, e.g., here and here so in API form and for workflow purposes, this is generic. Each system needs different input fields, and there has to be some way to supply them. Currently, there is an electronics_halt.jl or some file which supplies extremely specific fields like waveforms, gate times and high_altitude noise for each system, but again, this is generic to the workflow irrespective of AEM system.
• Users are free to make any modifications they wish to any part of the code as it is under the MIT license! If specific requests are made for any functions, issues can be opened and we can take care of them, or better, still, fork it or branch it and open a pull request! Most functions for a particular geophysical method exist within the enclosing module e.g., for Tempest in TEMPEST1DInversion.jl and the common ones are in CommonToAll.jl
• If you would like to show AEM forward responses as part of the inverted section, please open a pull request. This will require some familiarity with the forward modeling part of the code, and I encourage you to look here for SkyTEM and here for Tempest.

from hiqga.jl.

No worries, another thing which maybe useful @NMoghaddam @yusenley ... the Julia methods function to help you discover where functions are ...
for example

using HiQGA.transD_GP
methods(transD_GP.getchi2forall)
# 2 methods for generic function "getchi2forall":
[1] getchi2forall(optn_in::HiQGA.transD_GP.OptionsNuisance; nchains, figsize, fsize, alpha, nxticks, gridon) in HiQGA.transD_GP.CommonToAll at /g/data/z67/ar0754/juliadev/HiQGA/src/CommonToAll.jl:263
[2] getchi2forall(opt_in::HiQGA.transD_GP.Options; nchains, figsize, fsize, alpha, nxticks, gridon, omittemp) in HiQGA.transD_GP.CommonToAll at /g/data/z67/ar0754/juliadev/HiQGA/src/CommonToAll.jl:306

from hiqga.jl.

Cool @a2ray. I tried searching for the location of a couple of items as follows:

methods(transD_GP.dfn2hdr)
# 1 method for generic function "dfn2hdr":
[1] dfn2hdr(dfnfile::String) in HiQGA.transD_GP.CommonToAll at /g/data/z67/nfm547/package/HiQGA/src/CommonToAll.jl:1002
methods(transD_GP.SkyTEM1DInversion.read_survey_files)
# 2 methods for generic function "read_survey_files":
[1] read_survey_files(; fname_dat, fname_specs_halt, frame_height, frame_dz, frame_dx, frame_dy, LM_Z, HM_Z, LM_�, HM_�, relerror, units, figsize, makesounding, dotillsounding, startfrom, skipevery, multnoise, X, Y, Z, fid, linenum) in HiQGA.transD_GP.SkyTEM1DInversion at /g/data/z67/nfm547/package/HiQGA/src/SkyTEM1DInversion.jl:248
[2] read_survey_files(dfnfile::String; fname_specs_halt, frame_height, frame_dz, frame_dx, frame_dy, LM_Z, HM_Z, LM_�, HM_�, X, Y, Z, fid, linenum, LM_drop, HM_drop, relerror, units, figsize, makesounding, dotillsounding, startfrom, skipevery, multnoise) in HiQGA.transD_GP.SkyTEM1DInversion at /g/data/z67/nfm547/package/HiQGA/src/SkyTEM1DInversion.jl:172

from hiqga.jl.

A few open issues that can be referenced in our discussion are:

• Amplitude inversion plots for TEMPEST
  a2ray/transD_GP#18
• Read column names for tempest instead of manually assigning column nos
  a2ray/transD_GP#21
• Quit on no improvement in gradient-based
  a2ray/transD_GP#24
• use pmap for processing SkyTEM or Tempest transD plots
  a2ray/transD_GP#26

from hiqga.jl.

Thanks Ananad, Its clear. Negin has helped me fork a branch so I will work with this.

from hiqga.jl.

The links you provided are very helpful @a2ray. Thank you.

from hiqga.jl.