Tinker writer

[micaela-matta]

Fixes #

Changes made in this Pull Request:

• added Tinker trajectory writer

PR Checklist

• Tests?
• Docs?
• CHANGELOG updated?
• Issue raised/referenced?

[jbarnoud]

Thank you for the pull request. I do not have time to look at it right now, but I'll have a look as soon as possible if nobody reviewed the PR in the meantime.

For the tests to run, you need to add new_hexane.arc in MDAnalysisTests.datafiles as ARC_PBC. I suggest you run the tests locally to catch this kind of errors without waiting for travis. The wiki documents how to run the tests: https://github.com/MDAnalysis/mdanalysis/wiki/UnitTests.

[jbarnoud]

I did not see it was related to #2011.

[micaela-matta]

Thanks for pointing that out. Maybe after I've corrected that @richardjgowers can have a look.

[tylerjereddy]

Catching all exception classes, including keyboard interruptions, might not be desired

[tylerjereddy]

I did not see it was related to #2011.

Indeed, clarifying if this PR is still relevant / needed might be useful given period of inactivity.

[hmacdope]

Closing as stale, feel free to re-open if you want to continue.