Skip to content

B-1#1

Open
sofiiako wants to merge 34 commits intoeffect:masterfrom
sofiiako:master
Open

B-1#1
sofiiako wants to merge 34 commits intoeffect:masterfrom
sofiiako:master

Conversation

@sofiiako
Copy link

No description provided.

@matklad
Copy link
Collaborator

matklad commented Apr 10, 2014

Отлично 👍, первый pull request!

Сначала надо причесать код -- убрать не нужные закомментированные строки и по возможности отформатировать код в соответствии с PEP8(http://legacy.python.org/dev/peps/pep-0008/).

Например, не стоит ставить пробелы перед (), т.е, лучше писать queue.popleft() а не queue.popleft ().
В PyCharm это можно сделать так: http://www.jetbrains.com/pycharm/webhelp/reformat-code-dialog.html

@sofiiako
Copy link
Author

спасибо, сделаю! =)))

2014-04-10 21:40 GMT+04:00 Aleksey Kladov notifications@github.com:

Отлично [image: 👍], первый pull request!

Сначала надо причесать код -- убрать не нужные закомментированные строки и
по возможности отформатировать код в соответствии с PEP8(
http://legacy.python.org/dev/peps/pep-0008/).

Например, не стоит ставить пробелы перед (), т.е, лучше писать
queue.popleft() а не queue.popleft ().
В PyCharm это можно сделать так:
http://www.jetbrains.com/pycharm/webhelp/reformat-code-dialog.html

Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-40114822
.

@sofiiako
Copy link
Author

Почистила вот код к задачам из блока А :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Сначала все импорты, потом всё остальное

@matklad
Copy link
Collaborator

matklad commented Apr 13, 2014

Вот ещё про что совсем забыл -- лучше все функции писать в начале файла, а ввод/вывод -- в конце

def f1():
    pass

def f2():
    pass

def f3():
    pass

n = input_flie....
use f1, f2, f3
calculate answer

output_file.write(answer)

@sofiiako
Copy link
Author

sofiiako commented May 1, 2014

так, я не знаю, что это тут такое все и откуда оно взялось.. последнее, что я хотела добавить - вот это - sofiiako@9a928e4

@sofiiako
Copy link
Author

B8 >_< я почему-то его пропустила и вовремя не отправила

@sofiiako
Copy link
Author

sofiiako commented Jun 1, 2014

У меня сломался комп и сейчас я с другого, поэтому там и написано "Juliarep" вместо "sofiiako". (Это мама моя - комп ее)))
a Pcms2 как-то совсем завис

Copy link
Owner

Choose a reason for hiding this comment

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

Вообще в этом задании подразумевается использование и одного, и другого классификатора.

Copy link
Author

Choose a reason for hiding this comment

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

это не тоооо задание))) это вообще не по тому предмету!

On Thu, Jun 5, 2014 at 7:01 PM, Pavel Fedotov notifications@github.com
wrote:

In kolchanova/regression3.py:

@@ -0,0 +1,57 @@
+#!/usr/bin/python
+
+#I chose to perform the classification task using the KNN method

Вообще в этом задании подразумевается использование и одного, и другого
классификатора.


Reply to this email directly or view it on GitHub
https://github.com/effect/bii-14s/pull/1/files#r13444335.

Copy link
Author

Choose a reason for hiding this comment

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

там три файла - задачи по алгоритмам(( я не знаю, почему сложила их в ту же
папку, но теперь уже ничего сделать не могу, потому что отправляла Федору
на них ссылки

2014-06-06 1:48 GMT+04:00 Sofiia Kolchanova 19rubisco93@gmail.com:

это не тоооо задание))) это вообще не по тому предмету!

On Thu, Jun 5, 2014 at 7:01 PM, Pavel Fedotov notifications@github.com
wrote:

In kolchanova/regression3.py:

@@ -0,0 +1,57 @@
+#!/usr/bin/python
+
+#I chose to perform the classification task using the KNN method

Вообще в этом задании подразумевается использование и одного, и другого
классификатора.


Reply to this email directly or view it on GitHub
https://github.com/effect/bii-14s/pull/1/files#r13444335.

@sofiiako
Copy link
Author

sofiiako commented Jun 6, 2014

C7: we've done and discussed this together with Kirill, so the variables and stuff are much alike, but the approaches are different.
as a proof for that - on Pcms the solutions were submitted almost simultaneously :)

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