Skip to content

Fix SyntaxWarning (resolves #943).#944

Merged
dwhswenson merged 1 commit into
openpathsampling:masterfrom
bdice:fix/syntaxwarning-literal
Nov 17, 2020
Merged

Fix SyntaxWarning (resolves #943).#944
dwhswenson merged 1 commit into
openpathsampling:masterfrom
bdice:fix/syntaxwarning-literal

Conversation

@bdice

@bdice bdice commented Nov 17, 2020

Copy link
Copy Markdown
Contributor

This PR resolves #943 by fixing a SyntaxWarning that affects Python >=3.8. Also affects #864, #871.

@dwhswenson dwhswenson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix! Since you mention in #943 that you're starting out with OPS, a quick heads-up -- we're planning to do a little online tutorial (YouTube live stream) on some new aspects on OPS sometime in early December (I need to finish a few things to release 1.4 first). Topics will include the new storage and/or the command line interface, both of which will be especially useful if you're getting started on a new project. Check the issues here (I'll pin an issue when date/time are set) or watch the OPS Twitter (@pathsampling) for details!

@dwhswenson dwhswenson merged commit 6ee9733 into openpathsampling:master Nov 17, 2020
@bdice bdice deleted the fix/syntaxwarning-literal branch November 17, 2020 20:44
@bdice

bdice commented Nov 17, 2020

Copy link
Copy Markdown
Contributor Author

@dwhswenson Wonderful, thanks! 😄

@dwhswenson dwhswenson mentioned this pull request Dec 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SyntaxWarning in Python 3.8

2 participants