Reorder prior reading

This MR reorders the set of things attempted when trying to parse the prior argument strings. The previous method was causing issues, e.g., the one identified in !617 (merged).

I hope the new order makes a bit more logical sense than the old way. A more descriptive error message is raised and I added a docstring so that future developers will hopefully follow better what is going on.

This will strip the r'' from any parameters that are passed as raw strings, e.g., r'foo' will just render as foo.

@sylvia.biscoveanu can you test this on the prior you had issues with.

added Documentation Priors Refactoring labels

mentioned in merge request !617 (merged)

@colm.talbot it might make sense just to add Sylvia's test to the C.I. tests (and any other variants one can think of) so that future developers won't break this :D

I added the case that caused the error.

approved this merge request

Ok, I've tested this with my code and both the r' is rendered correctly and the parentheses: https://ldas-jobs.ligo.caltech.edu/~sylvia.biscoveanu/tupak/tests/test_latex_label1_corner.png https://ldas-jobs.ligo.caltech.edu/~sylvia.biscoveanu/tupak/tests/test_latex_label2_corner.png Thanks for the fix @colm.talbot! I will update my MR with the documentation changes.

approved this merge request

changed milestone to %0.5.9

merged

mentioned in commit 3e706657