Skip to content

HW 10 (1)HW11 (1)#152

Open
Fed909 wants to merge 4 commits into
mainfrom
feature
Open

HW 10 (1)HW11 (1)#152
Fed909 wants to merge 4 commits into
mainfrom
feature

Conversation

@Fed909
Copy link
Copy Markdown
Collaborator

@Fed909 Fed909 commented Jul 26, 2023

HW11 (1)
Пока всё что успел.

HW11 (1)
Пока всё что успел.
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Хорошо. Есть предложения, как сделать ещё лучше!

Comment thread Practice/Fedorov/HW_10/10.1.py Outdated
Comment thread Practice/Fedorov/HW_10/10.1.py Outdated
Comment thread Practice/Fedorov/HW_10/10.1.py Outdated
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.

Ожидаются ответы на вопросы в задании. Можно в виде комментариев.

Comment thread Practice/Fedorov/HW_11/11.1_server.py Outdated
Comment thread Practice/Fedorov/HW_11/11.1_client.py Outdated
HW12 (1)
HW13 (1,2)
HW14 (1)
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Добавил небольшой комментарий. Ну и старые замечания остались по 10.1.

Comment thread Practice/Fedorov/HW_12/12.1.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Остались вопросы только по 10.1

Comment thread Practice/Fedorov/HW_10/10.1.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

ОК. Добавил новые замечания.

my_lst = threading.Thread(target=find_primes, args=(num1, num2))
else:
my_lst = multiprocessing.Process(target=find_primes, args=(num1, num2))
my_lst.start()
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.

Так my_lst - это поток, процесс или список?

my_lst = multiprocessing.Process(target=find_primes, args=(num1, num2))
my_lst.start()
my_lst.append(my_lst)
num1 += 10000
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.

3 + 10000 => 10003, а не 10001

def krasivo(my_lst, istread):
num1 = 3
num2 = 10000
for x in range(2):
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.

Может, range(3) ?

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.

2 participants