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

add notebook support for loading of spyx and pyx files #606

Closed
mwhansen opened this issue Sep 7, 2007 · 6 comments
Closed

add notebook support for loading of spyx and pyx files #606

mwhansen opened this issue Sep 7, 2007 · 6 comments

Comments

@mwhansen
Copy link
Contributor

mwhansen commented Sep 7, 2007

From the notebook,

%sh
cat > hello.pyx << EOF
def hello(name):
    """
    Print hello with the given name.
    """
    print("Hello %s"%name)
EOF
load "hello.pyx"
Loading of file
"/opt/sage-2.8.3.rc3/sage_notebook/worksheets/admin/0/cells/0/hello.pyx"
has type not implemented.

There is a function that currently just checks to see if the filename extension is .py or .sage before it tries to load the file.

Component: notebook

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

@mwhansen
Copy link
Contributor Author

mwhansen commented Sep 7, 2007

Attachment: 606.patch.gz

@mwhansen
Copy link
Contributor Author

mwhansen commented Sep 7, 2007

comment:1

Patch added and tested.

@mwhansen mwhansen assigned mwhansen and unassigned boothby Sep 7, 2007
@williamstein
Copy link
Contributor

comment:2

This patch is completely bogus.

@williamstein
Copy link
Contributor

comment:4

This is a good spyx file to test:

was@ubuntu:~$ more a.spyx
def foo(int n):
    return n*n

@williamstein
Copy link
Contributor

Attachment: 606-2.patch.gz

changed version that i like

@williamstein
Copy link
Contributor

comment:5

Attachment: 6199.patch.gz

@williamstein williamstein modified the milestones: sage-2.9, sage-2.8.4 Sep 7, 2007
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

3 participants