Skip to content

adapted to centos#1

Open
Menestre1 wants to merge 7 commits intostsp:masterfrom
Menestre1:feature
Open

adapted to centos#1
Menestre1 wants to merge 7 commits intostsp:masterfrom
Menestre1:feature

Conversation

@Menestre1
Copy link
Copy Markdown

Fix a 2 bug in control command selection and determination of the quadrant in the flight operations. Tested on CentOS 8 operating system.

@stsp
Copy link
Copy Markdown
Owner

stsp commented Jan 9, 2021

Fix a 2 bug

... with a huge patch of over 1000 lines!
Please remove white-space errors from
the patch.

As for documentation - would it be possible
to translate an xml file rather than the generated
txt file?

@Menestre1
Copy link
Copy Markdown
Author

Fix a 2 bug

... with a huge patch of over 1000 lines!
Please remove white-space errors from
the patch.

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?

As for documentation - would it be possible
to translate an xml file rather than the generated
txt file?

It possible on the next holidays, my be in marth, 22-29.

doc/doc_rus.txt Outdated
@@ -0,0 +1,717 @@
ИНСТРУКЦИЯ К ИГРЕ STARTREK

Клингоны ждут
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.

А чего они ждут то? :)
Может, "идут"?

Copy link
Copy Markdown
Author

@Menestre1 Menestre1 Jan 9, 2021

Choose a reason for hiding this comment

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

это коммандеры идут, а клингоны ждут - так и буквально написано в инструкции как цитата так и по правилам игры, изложенным в инструкции Еще по духу клингонов, они могут ждать того, чтобы подраться.Но можно и идут написать, тоже хорошо.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Кажется с удалением пробелов все получилось?

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.

так и буквально написано в инструкции

А, ну в конце первой главы такая фраза есть.
А чего она в начале-то всплыла?
В начале ведь не это написано.

Copy link
Copy Markdown
Author

@Menestre1 Menestre1 Jan 10, 2021

Choose a reason for hiding this comment

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

А, ну в конце первой главы такая фраза есть. А чего она в начале-то всплыла? В начале ведь не это написано.

Кажется, Вы заинтересованы соблюсти близую к оригиналу структуру, но оригинальное руководство это неструктурированный текст (правила по оригинальному руководству разбросаны несколько хаотично), в предлагаемой инструкции облегчено восприятие по отношению к оригинальному руководству. Например без изучения советов в конце изложения более ранний текст не понятен - поэтому данные советы хочется перенести и выстроить в логическую последовательность. Ощущается что оригинальное руководство дописывается в конце по мере разработки программы без соблюдения структуры. Так же длинная история программы легко может быть найдена в интернет на английском и русском языке, она не нужна для того, чтобы получить удовольствие от игры и не является инструкцией. Это обоснование целесообразности данного текста. Я не говорю о том, что оригинал не нужен, но мой вариант имеет цель удобнее помогать игрокам чем оригинал.

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.

Да не в этом дело.
Просто непонятно, что фраза "клингоны
ждут" может делать в начале. В конце -
понятно: там она означает, "ну хватит
слов, пора играть. Клингоны ждут".
А вот в начале - должны быть совсем
другие фарзы, типа, "Клингоны идут,
Вселенная в опасности", и всё в таком
духе. Не забываем, что это не только
инструкция, но и литературное произведение.

Так же длинная история программы легко может быть найдена в интернет на
английском и русском языке, она не нужна для того, чтобы получить удовольствие
от игры и не является инструкцией.

Согласен, и переводить её на русский
совершенно не требуется.
Если хотите, можете её в отдельный
текстовый файл вынести и из оригинала
тоже, тут проблем никаких.

@stsp
Copy link
Copy Markdown
Owner

stsp commented Jan 9, 2021

Thanks, I want to fix it, but this white-space errors created by IDLE3 Environment.

According to this:
https://docs.python.org/3/library/idle.html
You can go to Format menu and select
Strip trailing whitespace.

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!
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.

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!
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.

This is a white-space patch.

@stsp
Copy link
Copy Markdown
Owner

stsp commented Jan 9, 2021

Нет, не получилось.
Надо убрать лишние патчи, а не исправлять
их новыми.

@Menestre1
Copy link
Copy Markdown
Author

Нет, не получилось.
Надо убрать лишние патчи, а не исправлять
их новыми.
проверьте пожалуйста результат:
@Menestre1
Revert "adapted to centosRevert "adapted to centos"
если не устраивает, подскажите как конкретно можно убрать лишние патчи, из нескольких опробованных способов все ведут к ошибкам.


def quadrant(self):
#print "Location %s -> %s" % (self, (self / QUADSIZE).roundtogrid())
<<<<<<< HEAD
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.

Ну что это?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Это исторический комментарий, я обнаружил что в первой версии своей правки потерял его, теперь вернул. Однако конструкция "(self / QUADSIZE).roundtogrid()" не рабочая возможно из-за эволюции среды Пайтон, у меня версия Python 3.6.8

@stsp
Copy link
Copy Markdown
Owner

stsp commented Jan 10, 2021

Убирать патчи можно через git rebase -i.
Там можно не только убирать, но и объединять
несколько в 1, изменять сообщения и тд.

Copy link
Copy Markdown
Author

@Menestre1 Menestre1 left a comment

Choose a reason for hiding this comment

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

fix crash because of missing quadrant() call

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