UO-Geophysics / MudPy

Kinematic and static rupture forward modeling and inversion code
Other
114 stars 63 forks source link

New issue when specifying the hypocenter #91

Open taranye96 opened 1 year ago

taranye96 commented 1 year ago

A new bug has been recently introduced where if you specify the hypocenter, there is an error saying something like "hypocenter[2] index out of range". The hypocenter parameter is getting redefined somewhere to some weird long array. The issue is happening around line of generate_ruptures_parallel.py. I made a temporary fix on my end to change line 228 to be hypocenter = hypocenter.

dmelgarm commented 1 year ago

I also noticed that all the ruptures now have hypocenters in the middle instead of randomly distributed throughout the rupture. Do you see this too?

On Wed, May 10, 2023 at 3:42 PM taranye96 @.***> wrote:

A new bug has been recently introduced where if you specify the hypocenter, there is an error saying something like "hypocenter[2] index out of range". The hypocenter parameter is getting redefined somewhere to some weird long array. The issue is happening around line of generate_ruptures_parallel.py. I made a temporary fix on my end to change line 228 to be hypocenter = hypocenter.

— Reply to this email directly, view it on GitHub https://github.com/dmelgarm/MudPy/issues/91, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAUJBIEWGLHPGVEYNSN2DXTXFQKWNANCNFSM6AAAAAAX5J66VI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Un mate para la ansiedad http://igpppublic.ucsd.edu/~dmelgarm

taranye96 commented 1 year ago

I don't think Ive noticed this. You mean the "random" hypocenters are always now in the midle of the fault?

dmelgarm commented 1 year ago

Yes!

On Wed, May 10, 2023 at 3:56 PM taranye96 @.***> wrote:

I don't think Ive noticed this. You mean the "random" hypocenters are always now in the midle of the fault?

— Reply to this email directly, view it on GitHub https://github.com/dmelgarm/MudPy/issues/91#issuecomment-1542908936, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAUJBIGQ5B6ORWYSPCLAYTDXFQMIPANCNFSM6AAAAAAX5J66VI . You are receiving this because you commented.Message ID: @.***>

-- Un mate para la ansiedad http://igpppublic.ucsd.edu/~dmelgarm