search

slimgroup / TimeProbeSeismic.jl

Memory efficient seismic inversion via trace estimation
MIT License
9 stars 2 forks source link

Add seed as an optional input to the operators #3

Open ziyiyin97 opened 3 years ago

ziyiyin97 commented 3 years ago

Maybe for future reference, we can add seed as an input to lsrtm_objective, fwi_objective, judiJacobian, etc so that we have more control of randomization of probing vectors. Now seed is only in qr_data and not included by any operator.

mloubout commented 3 years ago

the seed is not used, it's for future use if ever needed. I don't think it will so not gonna add it, randomization doesn't make much of a difference it's just used to span the range of the sample covariance of the data.