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

Non-ASCII character in file #28505

Closed
zimmermann6 opened this issue Sep 16, 2019 · 8 comments
Closed

Non-ASCII character in file #28505

zimmermann6 opened this issue Sep 16, 2019 · 8 comments

Comments

@zimmermann6
Copy link

with Sage 8.8 (the version shipped with Debian):

$ sage
┌────────────────────────────────────────────────────────────────────┐
│ SageMath version 8.8, Release Date: 2019-06-26                     │
│ Using Python 2.7.15. Type "help()" for help.                       │
└────────────────────────────────────────────────────────────────────┘
sage: from scipy.stats import norm
   File "/usr/local/SageMath/local/lib/python2.7/site-packages/scipy/stats/_continuous_distns.py", line 3345
     pp. 1–13, 1997.
                    ^
SyntaxError: Non-ASCII character '\xe2' in file /usr/local/SageMath/local/lib/python2.7/site-packages/scipy/stats/_continuous_distns.py on line 3346, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details

The following line should be added in the file:

# -*- coding: utf-8 -*-

Same for _stats_mstats_common.py

Component: misc

Issue created by migration from https://trac.sagemath.org/ticket/28505

@zimmermann6 zimmermann6 added this to the sage-8.9 milestone Sep 16, 2019
@fchapoton
Copy link
Contributor

comment:1

Works fine with python3-built sage. Time to switch ?

@embray
Copy link
Contributor

embray commented Dec 30, 2019

comment:2

Ticket retargeted after milestone closed

@embray embray modified the milestones: sage-8.9, sage-9.1 Dec 30, 2019
@mkoeppe
Copy link
Member

mkoeppe commented Jan 17, 2020

comment:3

Is this fixed with #28154?

@fchapoton
Copy link
Contributor

comment:4

Works fine for me.

┌────────────────────────────────────────────────────────────────────┐
│ SageMath version 9.1.beta0, Release Date: 2020-01-10               │
│ Using Python 3.7.3. Type "help()" for help.                        │
└────────────────────────────────────────────────────────────────────┘
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ Warning: this is a prerelease version, and it may be unstable.     ┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
sage: from scipy.stats import norm
sage: 

@zimmermann6
Copy link
Author

comment:5

I will try with Sage 9.0, if I manage to compile it from source on my machine, which was not the case for 8.8 and 8.9. See #28768.

@mkoeppe
Copy link
Member

mkoeppe commented May 1, 2020

comment:6

Moving tickets to milestone sage-9.2 based on a review of last modification date, branch status, and severity.

@mkoeppe mkoeppe modified the milestones: sage-9.1, sage-9.2 May 1, 2020
@mkoeppe
Copy link
Member

mkoeppe commented Jun 14, 2020

comment:7

Likely outdated after our switch to Python 3.

@mkoeppe mkoeppe removed this from the sage-9.2 milestone Jun 14, 2020
@zimmermann6
Copy link
Author

comment:8

I confirm from scipy.stats import norm works with 9.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants