nextstrain / augur

Pipeline components for real-time phylodynamic analysis
https://docs.nextstrain.org/projects/augur/
GNU Affero General Public License v3.0
268 stars 129 forks source link

Allow spaces in BEAST annotations #1610

Closed watronfire closed 1 month ago

watronfire commented 1 month ago

Spaces can be included in discrete states reconstructed by BEAST, even if they aren't proper according to NEXUS specs.

Description of proposed changes

I've added a space to the regexes for comments, strings in comments, and sets in comments.

Related issue(s)

Resolves: #1609

Checklist

watronfire commented 1 month ago

Yeah, glad to help out! Let me know if that changelog message is alright?

jameshadfield commented 1 month ago

Thanks Nate - I merged in the changes here via #1615 in order to resolve some merge conflicts.

We typically make an augur release every week or so, so it shouldn't be too long before your changes are in the release version