Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Script interface maintenance #4234

Merged
merged 4 commits into from
Apr 24, 2021
Merged

Script interface maintenance #4234

merged 4 commits into from
Apr 24, 2021

Conversation

jngrad
Copy link
Member

@jngrad jngrad commented Apr 21, 2021

Description of changes:

  • improve code coverage of the script interface
  • cluster analysis methods that return void in the core now return None in Python instead of True
  • unrecognized strings passed to method call_method() of cluster analysis now return None in Python instead of False

@jngrad jngrad added this to the Espresso 4.2 milestone Apr 21, 2021
@jngrad jngrad added the automerge Merge with kodiak label Apr 24, 2021
@kodiakhq kodiakhq bot merged commit b5f9101 into espressomd:python Apr 24, 2021
@jngrad jngrad deleted the script_interface_maintenance branch April 24, 2021 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reinaual reinaual reinaual approved these changes

Assignees
No one assigned
Labels
ApiChange automerge Merge with kodiak Testcase
Projects
None yet
Milestone
Espresso 4.2.0
Development

Successfully merging this pull request may close these issues.
2 participants
@jngrad @reinaual