Skip to content

Add logging and multi threading#20

Open
daveecode wants to merge 1 commit into
gallexis:masterfrom
daveecode:master
Open

Add logging and multi threading#20
daveecode wants to merge 1 commit into
gallexis:masterfrom
daveecode:master

Conversation

@daveecode
Copy link
Copy Markdown

Create functionality for run the module by run.sh with a torrent file as a parameter so that cannot modify the main.py in case of new torrents.
Add logging to extend debugging.
Add multi threading to tracker.py for better performance.

Comment thread main.py
Comment thread run.sh
@@ -0,0 +1,3 @@
#!/usr/bin/env bash
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nice to have, but I dont think it is needed by everyone. Consider removing please.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, can you remove this line?

Copy link
Copy Markdown
Owner

@gallexis gallexis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! It should improve the code thank you

Comment thread run.sh
@@ -0,0 +1,3 @@
#!/usr/bin/env bash
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, can you remove this line?

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.

3 participants