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

requirements.txt update - add deserialize #246

Closed
iMartyn opened this issue Sep 1, 2020 · 2 comments
Closed

requirements.txt update - add deserialize #246

iMartyn opened this issue Sep 1, 2020 · 2 comments

Comments

@iMartyn
Copy link

iMartyn commented Sep 1, 2020

Clean python install doesn't necessarily have deserialize :

On a clean python 3.7 on debian buster, for instance with this Dockerfile :

FROM python:3.7-buster
RUN git clone https://github.com/maximumstock/poe-currency-flip-planner.git
RUN cd poe-currency-flip-planner ; pip install -r requirements.txt

You still have to pip install deserialize :

root@174e58b83184:/poe-currency-flip-planner# python cli.py --league "Harvest"
Traceback (most recent call last):
  File "cli.py", line 5, in <module>
    from src.config.user_config import UserConfig
  File "/poe-currency-flip-planner/src/config/user_config.py", line 8, in <module>
    from src.trading import StackSizeHelper
  File "/poe-currency-flip-planner/src/trading/__init__.py", line 1, in <module>
    from .items import Item, ItemList, UnknownBackendException, UnsupportedItemException
  File "/poe-currency-flip-planner/src/trading/items.py", line 24, in <module>
    import deserialize
ModuleNotFoundError: No module named 'deserialize'
root@174e58b83184:/poe-currency-flip-planner# pip install deserialize
Collecting deserialize
  Downloading deserialize-1.8.0-py3-none-any.whl (15 kB)
Installing collected packages: deserialize
Successfully installed deserialize-1.8.0
root@174e58b83184:/poe-currency-flip-planner# python cli.py --league "Harvest"
Fetching Harvest offers for 120 pairs
@iMartyn iMartyn changed the title requirements.txt update - add deserialize requirements.txt update - add deserialize Sep 1, 2020
@maximumstock
Copy link
Owner

Should be fixed via 2eea1e6 as well.

@iMartyn
Copy link
Author

iMartyn commented Sep 1, 2020

Confirmed working here.

@iMartyn iMartyn closed this as completed Sep 1, 2020
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