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

Improvements to python server #442

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Conversation

deedy5
Copy link
Contributor

@deedy5 deedy5 commented Dec 24, 2023

  1. updated and simplified requirements.txt,
  2. requests have been replaced with httpx,
  3. ruff formatted for better compliance with python,
  4. unnecessary imports removed,
  5. minor feature enhancements.

@deedy5
Copy link
Contributor Author

deedy5 commented Dec 24, 2023

Unfortunately, there are no tests and I can't make sure everything works as it should.
But I have done all the operations from deprecated-do-not-use-start_server.sh - all dependencies are installed without conflicts, the server starts and responds to requests.

I think the server can be installed and started using deprecated-do-not-use-start_server.sh.
At least on linux it works now.

Will need to finalize it some more:

  1. Rename the names of all variables and functions to match pep8.
  2. Develop tests so that you can quickly check that everything is working as it should.

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.

1 participant