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

Debug code #15

Open
Suor opened this issue Mar 2, 2016 · 2 comments
Open

Debug code #15

Suor opened this issue Mar 2, 2016 · 2 comments

Comments

@Suor
Copy link
Collaborator

Suor commented Mar 2, 2016

I noticed lots of debug code like this:

debug and df.to_csv("%s.analysis_df.csv" % analysis.analysis_name)

Also debug argument needs to be parsed everywhere, which complicates code. Is this necessary?

You may use ipdb or debug in PyCharm instead.

@iDrDex
Copy link
Owner

iDrDex commented Mar 2, 2016

Not necessary. But convenient. We need the same engine running analyses on website as star_api. We also need to submit to website from API. What's the best way to get the tel in sync.

On Mar 2, 2016, at 4:33 AM, Alexander Schepanovski notifications@github.com wrote:

I noticed lots of debug code like this:

debug and df.to_csv("%s.analysis_df.csv" % analysis.analysis_name)
Also debug argument needs to be parsed everywhere, which complicates code. Is this necessary?

You may use ipdb or debug in PyCharm instead.


Reply to this email directly or view it on GitHub.

@Suor
Copy link
Collaborator Author

Suor commented Mar 3, 2016

I wrote this library to help not break anything while refactoring and optimizing. The idea is to cache (save) function result first time and then check it all subsequent times. If result changes app stops, you are presented with explanation of what changed and a debugger promt, where you can examine result and saved_result and compare them.

But to really make everything in sync we should share code. I will create starbase library and extract as much as possible to it. I'll create separate issue for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants