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

improve dynamic attributes of symbolic expressions #14802

Closed
burcin opened this issue Jun 21, 2013 · 19 comments
Closed

improve dynamic attributes of symbolic expressions #14802

burcin opened this issue Jun 21, 2013 · 19 comments

Comments

@burcin
Copy link

burcin commented Jun 21, 2013

Symbolic functions can define dynamic attributes that will be added to symbolic expressions representing evaluated forms of these functions (#9556).

This patch extends the existing functionality to allow these attributes to take arbitrary arguments. It also passes the function class and the symbolic expression as arguments.

Apply: attachment: trac_14802-dynamic_attribute_args.patch

Depends on #14780

CC: @vbraun @eviatarbach

Component: symbolics

Keywords: sd48

Author: Burcin Erocal

Branch/Commit: 5c579d8

Reviewer: Volker Braun

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

@burcin burcin added this to the sage-5.11 milestone Jun 21, 2013
@burcin burcin self-assigned this Jun 21, 2013
@burcin
Copy link
Author

burcin commented Jun 22, 2013

Dependencies: #14780

@burcin

This comment has been minimized.

@vbraun
Copy link
Member

vbraun commented Jun 26, 2013

comment:4

I'm getting doctest failures:

sage -t sage/symbolic/function_factory.py  # 1 doctest failed
sage -t sage/symbolic/expression.pyx  # 3 doctests failed

Missing dependency?

@burcin
Copy link
Author

burcin commented Jun 27, 2013

comment:5

Attachment: trac_14802-dynamic_attribute_args.patch.gz

Replying to @vbraun:

I'm getting doctest failures:

sage -t sage/symbolic/function_factory.py  # 1 doctest failed
sage -t sage/symbolic/expression.pyx  # 3 doctests failed

Missing dependency?

I forgot to update the documentation after adding the new function class argument. Updated patch attached.

@vbraun
Copy link
Member

vbraun commented Jun 27, 2013

comment:6

looks good to me

@vbraun
Copy link
Member

vbraun commented Jun 27, 2013

Reviewer: Volker Braun

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Jul 21, 2013
@jdemeyer jdemeyer removed this from the sage-5.12 milestone Aug 2, 2013
@vbraun
Copy link
Member

vbraun commented Mar 31, 2014

@vbraun
Copy link
Member

vbraun commented Mar 31, 2014

comment:10

Merge conflict with sage-6.2.beta6 will have to be resolved


New commits:

a6bea64Import of trac_14802-dynamic_attribute_args.patch from

@vbraun
Copy link
Member

vbraun commented Mar 31, 2014

Commit: a6bea64

@rwst
Copy link

rwst commented Apr 3, 2014

@rwst
Copy link

rwst commented Apr 3, 2014

Changed commit from a6bea64 to 80851f6

@rwst
Copy link

rwst commented Apr 3, 2014

comment:12

rebased on 6.2.beta6


New commits:

80851f6Merge branch 'u/vbraun/improve_dynamic_attributes_of_symbolic_expressions' of ssh://trac.sagemath.org/sage into ticket/14802

@rwst
Copy link

rwst commented Apr 8, 2014

comment:13

rebased on 6.2.beta7


New commits:

32e801aImport of trac_14802-dynamic_attribute_args.patch from

@rwst
Copy link

rwst commented Apr 8, 2014

Changed branch from public/14802 to public/14802-1

@rwst
Copy link

rwst commented Apr 8, 2014

Changed commit from 80851f6 to 32e801a

@rwst rwst added t: bug and removed t: enhancement labels Apr 8, 2014
@rwst rwst added this to the sage-6.2 milestone Apr 8, 2014
@rwst rwst removed the pending label Apr 8, 2014
@vbraun
Copy link
Member

vbraun commented Apr 8, 2014

comment:15

Tabs are not allowed, can you convert it to spaces?

sage -t --long src/sage/symbolic/pynac.pyx  # Tab character found

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 9, 2014

Changed commit from 32e801a to 5c579d8

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 9, 2014

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:

5c579d814802: remove tabs

@vbraun
Copy link
Member

vbraun commented Apr 9, 2014

Changed branch from public/14802-1 to 5c579d8

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