-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Audit SDK and View/Model functions #3384
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @colin99d,
So document can be generated but I see duplicate lines and functions without docstring.
Example:
covid_model.get_global_cases
covid_model.get_case_slopes
Hey Chavi, I just generated a new csv. Both of the functions you listed are only included once, and both have docstrings. Did you change anything before running? |
2 similar comments
Hey Chavi, I just generated a new csv. Both of the functions you listed are only included once, and both have docstrings. Did you change anything before running? |
Hey Chavi, I just generated a new csv. Both of the functions you listed are only included once, and both have docstrings. Did you change anything before running? |
Ok, I did find some errors:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok to me.
Although this version not compatible with Windows.
Edit: it should have Windows support now.
Description
A tool the automatically compares the SDK and view/model functions. This allows users to see if either one has duplicate functions, and then see how they map to each other.
To run use
python sdk_audit.py
How has this been tested?
Ran the functionality
Also ran
openbb.crypto.dd.balance("BTC")
to test that my change did not break the command.Checklist:
Others