Merge pull request #2466 from RosettaCommons/roccomoretti/parse_tag_interface_change
Change the interface to parse_my_tags
This is the final merge in a series of merges to reduce the interface of parse_my_tag(). This PR removes the Pose parameter from the parse_my_tag() function on Movers and Filters, leaving it as `void parse_my_tag( utility::tag::TagCOP tag, basic::datacache::DataMap &)` (like most other RosettaScripts objects.).
The Movers map and Filters map parameters have been merged into the DataMap object, and the Pose parameter has been removed completely. -- In general, there's no reason to rely on the state of the Pose at parse time. There's no guarantee that the state of the Pose at parse time will match that at apply time. Instead, you should cache the information (residue selections, chain designations, etc.) and then evaluate it on the pose passed to apply. There are various objects which allow you to do this deferral more generally.
Correspondingly, APPLY_TO_POSE has been removed. An empty APPLY_TO_POSE section is fine, but any content in that section will result in an error. See https://www.rosettacommons.org/docs/latest/scripting_documentation/RosettaScripts/Updating-RosettaScripts for more information about how to convert an XML using APPLY_TO_POSE to something that will work with the new scheme.