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.

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

Fix bytestring issues in Python3 #2898

Merged
merged 1 commit into from Jun 12, 2019
Merged

Fix bytestring issues in Python3 #2898

merged 1 commit into from Jun 12, 2019

Conversation

jngrad
Copy link
Member

@jngrad jngrad commented Jun 11, 2019

Fixes #2709

Description of changes:

  • use to_str() whenever the core returns a string in P3M, DipolarP3M, MMM1D
  • use to_str() when formatting Python exception messages based on error messages from the core

@codecov
Copy link

codecov bot commented Jun 11, 2019

Codecov Report

Merging #2898 into python will decrease coverage by <1%.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           python   #2898   +/-   ##
======================================
- Coverage      82%     82%   -1%     
======================================
  Files         519     519           
  Lines       26958   26958           
======================================
- Hits        22312   22310    -2     
- Misses       4646    4648    +2
Impacted Files Coverage Δ
src/core/electrostatics_magnetostatics/p3m.cpp 85% <0%> (-1%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b88304...cea3da1. Read the comment docs.

@RudolfWeeber
Copy link
Contributor

bors r+

bors bot added a commit that referenced this pull request Jun 12, 2019
2898: Fix bytestring issues in Python3 r=RudolfWeeber a=jngrad

Fixes #2709

Description of changes:
 - use `to_str()` whenever the core returns a string in P3M, DipolarP3M, MMM1D
- use `to_str()` when formatting Python exception messages based on error messages from the core


Co-authored-by: Jean-Noël Grad <jgrad@icp.uni-stuttgart.de>
@bors
Copy link
Contributor

bors bot commented Jun 12, 2019

Build succeeded

@bors bors bot merged commit cea3da1 into espressomd:python Jun 12, 2019
@jngrad jngrad deleted the fix-2709 branch January 18, 2022 12:11
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.

Cython and Python3: string conversion
2 participants