Skip to content
This repository was archived by the owner on May 21, 2020. It is now read-only.

Revision SymPy#56

Merged
AunSiro merged 2 commits intomasterfrom
revision_sympy_marzo_18
Mar 22, 2018
Merged

Revision SymPy#56
AunSiro merged 2 commits intomasterfrom
revision_sympy_marzo_18

Conversation

@AunSiro
Copy link
Copy Markdown
Member

@AunSiro AunSiro commented Mar 21, 2018

Algunos cambios menores, cero que no me dajo nada. En el issue #22 se menciona que hay que actualizarlos al estilo nuevo, pero a parte de cambiar el título y añadir un # aeropython: preserve, no sé qué más habría que modificar.

Algunos cambios menores
@astrojuanlu
Copy link
Copy Markdown
Member

Revisando!

@astrojuanlu
Copy link
Copy Markdown
Member

Importante: tengo las últimas versiones de todos los paquetes y me ha aparecido este error

sympy/sympy#13319

He puesto un comentario a ver si nos dicen cuáles son los paquetes afectados.

@astrojuanlu
Copy link
Copy Markdown
Member

Si lo he entendido bien, este parche lo arregla https://github.com/sympy/sympy/pull/14443.patch pero entiendo que hay alguna versión de IPython o pyzmq o sympy o algo que ha producido el error... ¿Qué versiones de estos paquetes tienes tú @AunSiro ?

Copy link
Copy Markdown
Member

@astrojuanlu astrojuanlu left a comment

Choose a reason for hiding this comment

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

El código está bien, pero por favor comprobad el tema de los paquetes que he comentado, no vaya a explotar en clase.

Se importan las funciones y variables de manera diferente para que
funcione mientras no esté disponible init_session
"from sympy import init_session\n",
"init_session(use_latex=True)```\n",
"\n",
" Esta función ya se encarga de importar todas las funciones básicas y preparar las salidas gráficas. Sin embargo, en este momento, esta función se encuentra en mantenimiento para su uso dentro de los notebooks por lo que activaremos la salida gráfica e importaremos las funciones de la manera usual. Puedes consultar el estado de la corrección en: https://github.com/sympy/sympy/pull/13300 y https://github.com/sympy/sympy/issues/13319 ."
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.

¡Muy bien!

@astrojuanlu
Copy link
Copy Markdown
Member

Otra cosa: el AttributeError: 'tuple' object has no attribute 'right' del notebook de Mecánica, ¿es esperado? ya no lo recordaba

@astrojuanlu
Copy link
Copy Markdown
Member

Es este issue: sympy/sympy#13354

@AunSiro AunSiro merged commit d5e1662 into master Mar 22, 2018
@AunSiro AunSiro deleted the revision_sympy_marzo_18 branch March 22, 2018 15:10
@astrojuanlu
Copy link
Copy Markdown
Member

Ya se ha arreglado el problema con init_session en la rama máster de SymPy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants