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.

Sign up for GitHub

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

Jump to bottom

mopad integration #421

Closed
megies opened this issue Oct 9, 2012 · 2 comments
Closed

mopad integration #421

megies opened this issue Oct 9, 2012 · 2 comments
Labels
external It's not our fault! .imaging Issues with our plotting functionalities task

Comments

@megies
Copy link
Member

megies commented Oct 9, 2012

geophysics/MoPaD should be included as a submodule in obspy.imaging.
@megies
Copy link
Member Author

megies commented Oct 9, 2012

also see geophysics/MoPaD#2

megies referenced this issue Aug 5, 2013
@krischer
Refactored mopad script.
47019b4 
Now adheres to PEP8. Also removed unused code parts and commented code.
Furthermore restructured the imports slightly and replaces some
expressions.

Tests still pass and I also tested it with lots of other moment tensors
which also look ok.
@megies
Copy link
Member Author

megies commented Aug 7, 2013

see comments in 47019b4

@mbyt mbyt mentioned this issue Oct 15, 2013
Open
@megies megies closed this as completed Oct 19, 2013
@krischer krischer mentioned this issue Jul 16, 2014
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
external It's not our fault! .imaging Issues with our plotting functionalities task
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@megies