-
Notifications
You must be signed in to change notification settings - Fork 26
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
Remove redundant code #46
Comments
@djmgit i've done this, so when u merged the typo from me from rahuldcoder (cause i corrected this on my local already and don't want to steal his solution :D), then i will commit it and open a PR ;) |
@LoneOne123 yes, by flag I mean an additional argument. |
so sry have had an exam yesterday and my ssd died :( |
@djmgit here we go, but it still just works with [array1][array2] and not with [array1],[array2] |
@LoneOne123 no problem, its great that you are still continuing to work on this :) |
In lib/plotutil.py, the functions plot and plot_line has many elements in common. Try to merge plot_line into plot. Use a flag (or any other method) to determine whether a function is to be plotted or two lists of x and y has to be plotted and do both the stuff in the same function. In this way the repeated code can be removed.
The text was updated successfully, but these errors were encountered: