Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add seed as an optional input to the operators #3

Open
ziyiyin97 opened this issue Apr 7, 2021 · 1 comment
Open

Add seed as an optional input to the operators #3

ziyiyin97 opened this issue Apr 7, 2021 · 1 comment

Comments

@ziyiyin97
Copy link
Member

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
Copy link
Member

mloubout commented Apr 7, 2021

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants