Remove taup.tau.remove-create_taup_model() which is no longer used

[seisman]

What does this PR do?

Remove a function taup.tau.remove-create_taup_model().

Why was it initiated? Any relevant Issues?

This function is never used in the source codes. It is also not tested at all. The function calls a method TauPCreate.main, which doesn't exists. The function is replaced by taup.taup_create.build_taup_model().

PR Checklist

• Correct base branch selected? master for new features, maintenance_... for bug fixes
• This PR is not directly related to an existing issue (which has no PR yet).
• If the PR is making changes to documentation, docs pages can be built automatically.
  Just remove the space in the following string after the + sign: "+DOCS"
• If any network modules should be tested for the PR, add them as a comma separated list
  (e.g. clients.fdsn,clients.arclink) after the colon in the following magic string: "+TESTS:"
  (you can also add "ALL" to just simply run all tests across all modules)
• All tests still pass.
• Any new features or fixed regressions are be covered via new tests.
• Any new or changed features have are fully documented.
• Significant changes have been added to CHANGELOG.txt .
• First time contributors have added your name to CONTRIBUTORS.txt .

[megies]

Interesting.. I can't even find evidence that this code ever worked in any release version.

[megies]

Can you please do this PR against master though? This PR won't fix anything maintenance-wise, but changes the exception that will be raised when trying to use this function from an AttributeError to an ImportError, potentially. Also please remove the now unused import of TauPCreate.

[megies]

Can you please do this PR against master though?

I can cherry-pick your commit on master if it's too much hassle for you..

Thanks for the PR :)

[seisman]

@megies I've changed the base branch to master and removed the unused import of TauPCreate.

[megies]

Looks good, thanks a lot again!