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

provide the calling graph of the processing in the log window #18

Open
shelper opened this issue Jul 19, 2016 · 3 comments
Open

provide the calling graph of the processing in the log window #18

shelper opened this issue Jul 19, 2016 · 3 comments

Comments

@shelper
Copy link

shelper commented Jul 19, 2016

It would be very useful if I tested the processing procedure and it works, now i want to add the processing procedure (API calls) to my application source code, i can just open the log window, copy and paste to my source code editor, followed by minor modifications to fit into the code context.
that will save a lot of time for developing.

i would also suggest to include both python and/or c++ calls

@dinusv
Copy link
Contributor

dinusv commented Jul 20, 2016

Hi, thank you for your suggestion, sounds interesting and it would be very useful for debugging as well. I would create a separate window for the call graph, the log would be filled with too much stuff in some cases.

I think we can manage this with a global CallGraph class instance that each algorithm calls just after it does the processing. We can store the matrix references there as well, so we can follow how each matrix gets transferred thorugh the calls.

@shelper
Copy link
Author

shelper commented Jul 20, 2016

i used your program a little bit more and i think it could be useful if the program separates edit window from the preview images, or support of 3rd part editors, like vim would be even better

i can open new issues if needed but i want just explain here first.

@dinusv
Copy link
Contributor

dinusv commented Jul 20, 2016

True, I actually thought about using separate editors at some point, but never had the chance to look into it. There's also the preview mode which I want to add I think it falls in the same category.

I've created 2 issues from this: #19, #20.
Thanks for your input, you're welcome to open new issues if you have more suggestions

@dinusv dinusv added the editor label Jul 21, 2016
@dinusv dinusv modified the milestone: Version 1.3.0 Jul 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants