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

Remove unused imports #3

Open
wants to merge 3 commits into
base: development
Choose a base branch
from
Open

Remove unused imports #3

wants to merge 3 commits into from

Conversation

durvalcarvalho
Copy link

Description

Removed all unused imports pointed by flake8.
Fix undefined variables on data_reader/transport.py
Removing seed_db.py script from start.sh file (this script is importing a model that does not exist)

@durvalcarvalho
Copy link
Author

In the file data_reader/transport.py, the open_socket method of the TransportProtocol class uses the timeout and port variables, I believe these are the self.timeout and self.port instance atributes

image

@durvalcarvalho
Copy link
Author

In the file seed_db.py is imported the class TransductorModel from transductor_model.models. I checked and this model dont exists.

image

This script was raising an ImportError, so i commented the line that ran this script from the start.sh file for the container to be created without errors

image

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

Successfully merging this pull request may close these issues.

None yet

1 participant