-
Notifications
You must be signed in to change notification settings - Fork 24
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
Metadata in Synthetics Calculation with FK in MTUQ #291
Comments
Hi Felix,
Thanks for the report. Let me check more later, but here at first glance
is what I think is going on.
Previously the metadata "stats" from the observed data were getting carried
over to the synthetics. This works well many times, but doesn't
generalize to all cases/ data formats.
Now the metadata from the Green's functions themselves get carried over to
the synthetics. The Green's function metadata is just what is supplied
when you call the get_greens_tensors() method. At least this was the
desired default behavior, let me check.
You could also override the default behavior above and control directly
what metadata gets passed to the synthetics via the "stats" argument to
get_synthetics().
…-Ryan
On Wed, Jan 15, 2025 at 4:18 PM Felix Rodriguez Cardozo < ***@***.***> wrote:
Probably this commit may be helpful for this discussion: 7668898
<7668898>
. Thanks @thurinj <https://github.com/thurinj>
—
Reply to this email directly, view it on GitHub
<#291 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABCGSSSPORMGYNGWAFCGUW32K3UDJAVCNFSM6AAAAABVGSS3RGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOJUGEZTAOJVGY>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Hi Felix, You mentioned a specific commit (7668898). Could you double check that you are using this version or a more recent one? I am finding that for such versions, the synthetic traces have the requested metadata |
For reference, I used the current upstream version (f3d6cc7) and added these two lines to SerialGridSearch.DoubleCouple.py:
The resulting standard output is
I also tried adding
and got the same result. |
So at least get_synthetics() seems to give the requested metadata in examples/SerialGridSearch.DoubleCouple.py If the behavior still continues in your case, could you post fuller example? |
Hello,
I am facing an interesting situation when I am calculating synthetics in MTUQ with an FK database. Basically with these two lines I am doing all the job:
The synthetics calculation worked. However, when I tried to save the synthetics, I realized that the streams in synthetics list do not contain most of the metadata that is required for obtaining the station name, network, station location, etc. Interestingly, this only happens in the newest version of MTUQ. When I tried with an old version, the synthetics contains a comprehensive amount of metadata.
With old version:
print(pair[0].stats)
With new version:
In both cases, I used the same script and the same data, I only changed the MTUQ environment. On the other hand, this problem is not seen when I use Specfem3D GFs database. I also noticed that the starttime and endtime are also different despite I used the same origin time in both cases: '2017-12-01T02:32:44.000000Z'
If you can shed some light on what happened wit the .get_synthetics subroutine in the most recent version, I would appreciate it a lot. Without the metadata, it is difficult to figure out to what station corresponds a given synthetic trace.
Thanks for your help,
Felix
The text was updated successfully, but these errors were encountered: