-
Notifications
You must be signed in to change notification settings - Fork 529
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
rebrand IRIS -> EarthScope pt. 2 #3456
Conversation
deprecation warning when using "iris-federator"
I just realized Python is not even showing Gonna change these warnings to |
Python has been hiding DeprecationWarnings by default for ages now, for whatever silly reason
@chad-earthscope did some more changes, will merge this, let me know if there's anything else, but I think this is everything now besides renaming |
Looks good to me, thanks @megies! |
What does this PR do?
This is a follow up for #3448 following #3448 (comment)
This changes some more docstring wordings to EarthScope and will show deprecation warnings when using
'IRIS'
short URL to initialize an FDSN client and'iris-federator'
when initializing an federator client.Why was it initiated? Any relevant Issues?
see #3448
PR Checklist
master
for new features,maintenance_...
for bug fixesno_ci
label can be added to skip CI buildsJust add the
build_docs
tag to this PR.Docs will be served at docs.obspy.org/pr/{branch_name} (do not use master branch).
Please post a link to the relevant piece of documentation.
clients.fdsn
) should be tested for the PR,just add the
test_network
tag to this PR.CHANGELOG.txt
.CONTRIBUTORS.txt
.from all the CI builds look correct. Add the "upload_plots" tag so that plotting
outputs are attached as artifacts.
CODEOWNERS
with your github handleready for review
label when you are ready for the PR to be reviewed.