Add option base model

[jeffweng8]

Checklist

• Added tests for changed code.
• Updated documentation for changed code.
• I've added a news fragment of my changes with the name,
  "{ISSUE_NUM}.{feature|bugfix|doc|removal|misc}""

Related Issue

#201

Description

• Updated several of the endpoints to be able to support options
• Streamline instrument getter functions
• Add option model class

[adithyabsk]

Just ping me when you want me to take a look again. Also, if you're having trouble with the linter, take a look at the docs to see how to set up pre-commit which should automatically fix most of the issues.

[jeffweng8]

Hey @adithyabsk

I am getting
pyrh/common.py:5: error: Cannot find implementation or library stub for module named 'yarl' pyrh/common.py:5: note: See https://mypy.readthedocs.io/en/latest/running_mypy.html#missing-imports pyrh/models/base.py:6: error: Cannot find implementation or library stub for module named 'marshmallow' pyrh/models/base.py:78: error: Class cannot subclass 'Schema' (has type 'Any') pyrh/models/portfolio.py:5: error: Cannot find implementation or library stub for module named 'marshmallow' pyrh/models/option.py:5: error: Cannot find implementation or library stub for module named 'marshmallow' pyrh/models/oauth.py:6: error: Cannot find implementation or library stub for module named 'marshmallow' pyrh/models/oauth.py:46: error: unused 'type: ignore' comment pyrh/models/sessionmanager.py:11: error: Cannot find implementation or library stub for module named 'marshmallow' pyrh/models/sessionmanager.py:14: error: Cannot find implementation or library stub for module named 'yarl' pyrh/models/sessionmanager.py:497: error: unused 'type: ignore' comment Found 10 errors in 6 files (checked 13 source files)
when I try to run the linter locally. Do you know how to resolve this issue?

[adithyabsk]

@jeffweng8 I think you might not have the packages installed locally, perhaps? (or at least not in a place mypy can find them) pre-commit is installed in a separate virtualenv so that it doesn't depend on your env but mypy needs those installations to properly static check our code. Can you try manually running mypy pyrh from the project base directory?

[jeffweng8]

Ah that makes sense, I just installed yarl and marshmallow in the virtual environment and that fixed the issue. Thanks!

[adithyabsk]

@jeffweng8 you likely want to install all of our dev dependencies either using pip install . or using poetry install -vvv (poetry is reccomended)

[jeffweng8]

Hmm I am getting this error when I try to run the poetry install command:

Traceback (most recent call last):
File "/Users/jeff/anaconda3/bin/poetry", line 7, in
from poetry.console import main
ModuleNotFoundError: No module named 'poetry'

Any idea where that is coming from? I ran pip install . and it successfully ran.

Also, when I try to run pytest, I get the following:
ERROR: usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: --xdoc --cov=pyrh --cov-config=setup.cfg --cov-report=term --cov-report=html
inifile: /Users/jeff/Documents/pyrh/setup.cfg
rootdir: /Users/jeff/Documents/pyrh

Is this related to the above issue?

[jeffweng8]

Also should I close this PR and make a new PR where I squash my commits?

[adithyabsk]

Nope, no worries. I can squash when I merge @jeffweng8

[adithyabsk]

Yep @jeffweng8, so I would actually suggest you not use your anaconda version of python when installing poetry since anaconda comes with a bunch of packages pre-installed.

$ which python

Should tell you the info you need there. I would use your system version of python (pre-installed on macs or brew version of python or better yet use pyenv to manage your python version)

You do this by looking at your PATH, which you likely modified to be able to use anaconda (putting it first in front of other directories your system would want to search)

[adithyabsk]

Additionally if you use poetry it will automatically manage your virtual environment so you can just do poetry run [CMD] and access you venv easily. (E.g. poetry run pytest

[adithyabsk]

It is also important to note how you install poetry, you shouldn’t install it just using pip, take a look at their docs and there should be a one line command that your run which uses your defined python version in your path to set it up.

[adithyabsk]

@jeffweng8 just to keep you in the loop, I'm starting to work on re-factoring the endpoints part of the project. That's the next big task. I've started the a draft PR #226. Take a look to get a sense of the direction that I'm heading there.

[jeffweng8]

Hey @adithyabsk

I just pushed the skeleton for the chain model. I need to fix the session manager for get_chain, but I'm not too sure what exactly that is supposed to be

[adithyabsk]

@jeffweng8 I'll be quite busy for the next few weeks but I'll try to take a look this weekend. Also let me know when this PR is in a good state for CR and I'll try to get that in.

Also, to give you a sense of my next steps, I know I need to step up a better way to run tests on this project especially with Robinhood constantly updating their API. To that end, I'm likely going to try and set up vcr.py and pytest-vcr so that writing tests is as easy as calling the desired endpoints and caching the results. It will be somewhat difficult to initially set it up to work with oauth but once that is done it should be smooth sailing from there.

[adithyabsk]

I need to fix the session manager for get_chain

Also, what specifically is broken?

[jeffweng8]

No rush, I was just unsure what exactly a session manager is supposed to be. Is it just getting the results from a page? If so should I create a default one since most of them should just be grabbing it from the "results" key?

[adithyabsk]

SessionManager allows you to authenticate with the Robinhood API. Take a look at how I structured the Portfolio modules. I made a PortfolioManager which the main Robinhood class inherits from. The idea is that all "managers" should inherit from it and as a result get access to the managed forms of the get and post methods which allow you to hit the endpoints that are restricted to only logged in users.

Eventually, we should be able to mark specific methods as not requiring log-in which should work without the authentication flow.

[jeffweng8]

Hi @adithyabsk, I think this PR is pretty much ready. Could you help me see why this is failing some of the checks? When I run pytest locally, everything passes, but it appears that this is happening for some of the checks here:

Creating virtualenv pyrh-6aazBjZI-py3.7 in /Users/runner/.virtualenvs [FileNotFoundError] [Errno 2] No such file or directory ##[error]Process completed with exit code 1.

[adithyabsk]

@jeffweng8 that's related to this: #233

If you notice, your build passed on Linux. Thanks for the PR, I'll try to do a proper CR in the next week.

[jeffweng8]

Hey @adithyabsk have you had a chance to look at the changes yet?

[jeffweng8]

Hi @adithyabsk , Are you still planning on working on or maintaining this project?

[adithyabsk]

@jeffweng8 Sorry for the long delay in response, I'll look at this in the coming week.

[stale]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

Closing this pull request automatically because it has not had any activity since it has been marked as stale. If you think it is still relevant and should be addressed, feel free to open a new one.