Conversation
... with a huge patch of over 1000 lines! As for documentation - would it be possible |
Thanks, I want to fix it, but this white-space errors created by IDLE3 Environment. What software i need to make correct diff? My be special text editor do not break it?
It possible on the next holidays, my be in marth, 22-29. |
doc/doc_rus.txt
Outdated
| @@ -0,0 +1,717 @@ | |||
| ИНСТРУКЦИЯ К ИГРЕ STARTREK | |||
|
|
|||
| Клингоны ждут | |||
There was a problem hiding this comment.
А чего они ждут то? :)
Может, "идут"?
There was a problem hiding this comment.
это коммандеры идут, а клингоны ждут - так и буквально написано в инструкции как цитата так и по правилам игры, изложенным в инструкции Еще по духу клингонов, они могут ждать того, чтобы подраться.Но можно и идут написать, тоже хорошо.
There was a problem hiding this comment.
Кажется с удалением пробелов все получилось?
There was a problem hiding this comment.
так и буквально написано в инструкции
А, ну в конце первой главы такая фраза есть.
А чего она в начале-то всплыла?
В начале ведь не это написано.
There was a problem hiding this comment.
А, ну в конце первой главы такая фраза есть. А чего она в начале-то всплыла? В начале ведь не это написано.
Кажется, Вы заинтересованы соблюсти близую к оригиналу структуру, но оригинальное руководство это неструктурированный текст (правила по оригинальному руководству разбросаны несколько хаотично), в предлагаемой инструкции облегчено восприятие по отношению к оригинальному руководству. Например без изучения советов в конце изложения более ранний текст не понятен - поэтому данные советы хочется перенести и выстроить в логическую последовательность. Ощущается что оригинальное руководство дописывается в конце по мере разработки программы без соблюдения структуры. Так же длинная история программы легко может быть найдена в интернет на английском и русском языке, она не нужна для того, чтобы получить удовольствие от игры и не является инструкцией. Это обоснование целесообразности данного текста. Я не говорю о том, что оригинал не нужен, но мой вариант имеет цель удобнее помогать игрокам чем оригинал.
There was a problem hiding this comment.
Да не в этом дело.
Просто непонятно, что фраза "клингоны
ждут" может делать в начале. В конце -
понятно: там она означает, "ну хватит
слов, пора играть. Клингоны ждут".
А вот в начале - должны быть совсем
другие фарзы, типа, "Клингоны идут,
Вселенная в опасности", и всё в таком
духе. Не забываем, что это не только
инструкция, но и литературное произведение.
Так же длинная история программы легко может быть найдена в интернет на
английском и русском языке, она не нужна для того, чтобы получить удовольствие
от игры и не является инструкцией.
Согласен, и переводить её на русский
совершенно не требуется.
Если хотите, можете её в отдельный
текстовый файл вынести и из оригинала
тоже, тут проблем никаких.
According to this: |
sst.py
Outdated
| game.probe.moves -= 1 # One less to travel | ||
| if game.probe.arrived() and game.isarmed and pdest.stars: | ||
| supernova(game.probe) # fire in the hole! | ||
| supernova(game.probe.quadrant()) # fire in the hole! |
There was a problem hiding this comment.
Please add an informative log msg
to commit. Ideally it should contain
the python's crash message.
Like:
fix crash because of missing quadrant() call
This fixes the following crash:
<crash msg here>
| game.probe.moves -= 1 # One less to travel | ||
| if game.probe.arrived() and game.isarmed and pdest.stars: | ||
| supernova(game.probe.quadrant()) # fire in the hole! | ||
| supernova(game.probe.quadrant()) # fire in the hole! |
|
Нет, не получилось. |
" This reverts commit 3eef073.
|
|
|
||
| def quadrant(self): | ||
| #print "Location %s -> %s" % (self, (self / QUADSIZE).roundtogrid()) | ||
| <<<<<<< HEAD |
There was a problem hiding this comment.
Это исторический комментарий, я обнаружил что в первой версии своей правки потерял его, теперь вернул. Однако конструкция "(self / QUADSIZE).roundtogrid()" не рабочая возможно из-за эволюции среды Пайтон, у меня версия Python 3.6.8
|
Убирать патчи можно через |
Menestre1
left a comment
There was a problem hiding this comment.
fix crash because of missing quadrant() call
Fix a 2 bug in control command selection and determination of the quadrant in the flight operations. Tested on CentOS 8 operating system.