- Tracking of variables: adding, clearing, loading implemented
- Syntax highlighter uses extracted functions and variables
- Caching of modules variables
Details
- Reviewers
filipesaraiva - Group Reviewers
Cantor - Maniphest Tasks
- T1753: Add support for Julia (julialang-backend)
- Commits
- R55:df70016a0e02: Variable management
Diff Detail
- Repository
- R55 Cantor
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
@ivanlakhtanov, please could you confirm if the feature works just using this patch? I applied it, I can see the variable management widget but when I create a variable in the worksheet (like typing and running x = 3), Cantor starts the command processing but never stop.
I have checkouted to Variable management commit and everything works fine. I can take this on video if you don't belive me. Can you try to run unit-tests for command execution? And are you sure you have added all dependent patches like latest command execution?
Question: can I push --force to own remote feature branch? That way I can send it to server, so you can checkout to desired commit.
Have just checked it with arc patch D2198. Everything is OK. Which version of Julia do you have?
Ok, I found the problem. It was just the path to julia bin was incorrect, now it is working properly.
src/backends/julia/julia.qrc | ||
---|---|---|
6–10 ↗ | (On Diff #5959) | Ok, like the previous review request, remove this qrc file. Use the same approach of Octave backend: create a script folder, put your scripts there and create a new CMakeLists.txt file in the folder. |
src/backends/julia/juliaextensions.cpp | ||
59–73 | After remove that QRC file you must to change those calls of fromSource. | |
src/backends/julia/juliautils.h | ||
1–29 ↗ | (On Diff #5959) | Does this header get more functions in the other patches? If not maybe you could to change the name for something more specific like "loadScripts". |