Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 1463 by Toon.Verstraelen: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
I've tried to send the message below to sympy-patches, but it doesn't seem
to get through.
Hi,
This is my first at
Updates:
Status: Accepted
Labels: -Type-Defect Type-Enhancement NeedsReview
Comment #1 on issue 1463 by ondrej.certik: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
Hi Toon,
thanks a lot! It was waiting for a moderation, as we are getting qu
Updates:
Owner: ondrej.certik
Comment #2 on issue 1463 by ondrej.certik: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
I am reviewing it now.
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you
Comment #3 on issue 1463 by ondrej.certik: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
The code looks good, all tests pass, so the patches are +1 from me, it's a
good
beginning. I'll wait some time if others want to review it too, before I
push it in.
Comment #4 on issue 1463 by ondrej.certik: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
* we should add an option to codegen() to also generate some
makefiles/setup.py/cmake/... files to actually compile the project,
together with a
sample main program. S
Comment #5 on issue 1463 by Vinzent.Steinberg: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
Do we really want to support all the build systems? Shouldn't the user care
about this?
--
You received this message because you are listed in the owner
or CC f
Comment #6 on issue 1463 by Toon.Verstraelen: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
Thanks for the suggestions, I'll make a new set of patches. Maybe it is
even better
to let codegen return the contents of the files as strings by default. With
an
Comment #7 on issue 1463 by Toon.Verstraelen: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
With respect to 0001-Allow-SYMPY_TEST_CLEAN_TEMP-never.patch: Is there a
way to print
the locations of the temporary directories on screen? I have the impression
Comment #8 on issue 1463 by ondrej.certik: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
Yeah, ignore the build system for now, I agree.
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this
Comment #9 on issue 1463 by fab...@fseoane.net: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
some ideas:
please try not to define symbols globally on the tests (in
sympy/utilities/tests/test_codegen.py).
and the module should be referenced from ccode, s
Updates:
Labels: -NeedsReview Milestone-Release0.6.5
Comment #10 on issue 1463 by ondrej.certik: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
Ping us when you have some updates.
--
You received this message because you are listed in the owner
or CC
Comment #11 on issue 1463 by Toon.Verstraelen: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
I think the symbols in the tests are global variables.
Will add the comment in ccode.
I have considered the idea to extend the printing routines for sympy
fun
Comment #12 on issue 1463 by Toon.Verstraelen: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
I'm a bit busy, but expect new patches soon. I recently finished my PhD and
promised
quite a few people to do X for them after the defense and the vacation. :-)
Comment #13 on issue 1463 by fab...@fseoane.net: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
Thanks for the clarification
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You
Comment #14 on issue 1463 by Toon.Verstraelen: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
Ok, here is the following set of patches. It is basically the original
patch + taking
into account all the comments above. There is one more thing that was
add
Updates:
Status: Started
Labels: NeedsReview
Comment #15 on issue 1463 by Vinzent.Steinberg: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
(No comment was entered for this change.)
--
You received this message because you are listed in the
Comment #16 on issue 1463 by Toon.Verstraelen: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
Maybe an additional note for this patch. The idea is to keep the
functionality
minimal at first instance. The next step will be an extension to a wide
range of
Updates:
Owner: ---
Labels: -Milestone-Release0.6.5 -NeedsReview
Comment #17 on issue 1463 by ondrej.certik: Initial codegen implementation
http://code.google.com/p/sympy/issues/detail?id=1463
The patches look good now, I tested both on 64 and 32 bits, tests pass so,
I pushed
t
Updates:
Status: Fixed
Comment #18 on issue 1463 by Toon.Verstraelen: Initial codegen
implementation
http://code.google.com/p/sympy/issues/detail?id=1463
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fields of this iss
19 matches
Mail list logo
