Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A wild corgi appears! А у нас внезапный фриз на карты и код который хоть как-то связан со словом атмос. #1745

Closed
wants to merge 1 commit into from

Conversation

ZVee
Copy link
Contributor

@ZVee ZVee commented Aug 18, 2017

Это относится ко всем ПРам которые будут созданы ровно после этого.

Снятие фриза ориентировочно после мержа этого ПР #1771 и по прошествии немного времени когда все устаканится.

  • Коротко о главном

    • ПРы в которых задеты карты - не принимаются и даже после снятия фриза не будут приняты, решить такого рода конфликт и при этом не испортить карту - будет крайне сложно.
    • ПРы в которых задет код конфликтующий с изменениями - решение будет вынесено индивидуально, т.к напрямую зависит от того, что изменено.
    • Не затевайте ПРы в которых вам надо задеть карту, даже если это вида - надо поправить 1 турф. Касательно кода - с ним проще, но если ваши фичи хоть как-то пересекаются с машинерией, атмосом, вы можете поймать тонну конфликтов.
  • О времени

  • О картах

    • Т.к. я в курсе того, что у нас там занимаются переделыванием карт, то в принципе текущие ваши планы с изменением карт (которые еще на гит не попали, но в процессе работы) можно пропустить конечно, но это все будет удлинять фриз. Я не буду даже браться за карты, пока их не перестанут трогать, т.к перемапливать весь трубопровод с нуля - задача не из приятных.
  • Под карты попадает попросту папка .../maps. Причина проста - решить конфликты когда половина карты будет задето, задача не из приятных. Собственно фриз по большей части именно из-за карт.

  • Под атмос - любой код в котором хоть какая-то есть связь с атмосом, его машинерией, газами и конечно же папкой .../code/modules/atmospherics.

  • Что принесет этот фриз и то из-за чего он затеян

    • Подробности пока не раскрою, но думаю, выше, подсказок полно.
    • Почему с картами так сложно? Потому что бъенд, и потому что мапмержер, и решение конфликтов может привести к потере данных на карте, если человек решающий их не имеет опыта, да и тут изменений будет очень не мало, особенно в z1.dmm.

Для тех кто до сих пор не понял на что фриз (даже прочитав название и описание):

Карты, атмос-машинерия и строки кода/объекты которые имеют отношение к атмосу и засу (т.е газы, температура и все такое), например шатать систему дыхания мобов которая взаимодействует с газами, перепил пожаров, внедрение фичи тепловыделения машин и тому подобное.

Что-то еще хотел написать, да из головы вылетело как обычно...

@Belfogor
Copy link
Contributor

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

@ZVee
Copy link
Contributor Author

ZVee commented Aug 22, 2017

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

Нет, шатать я буду переименование всяких стрингов и то, каким образом выдираются данные по тем же газам. Например получение кол.ва молей кислорода сейчас как var/n = air.oxygen, а будет скажем как var/n = air.gas["oxygen"]. Поэтому шатание механа типа кол.ва использования кислорода мобами легких, это немного не то, что стоит изначально в задачах.
Коротко - обещать изменений такого рода не могу.

@Zap-zapper
Copy link
Contributor

То есть добавлять и удалять что-то с карты теперь навсегда запрещено? Даже, если я там придумал новую машинерию для, допустим, роботики, накодил её, и хочу поставить в роботику, то я не смогу этого теперь сделать?

@P-20
Copy link
Contributor

P-20 commented Sep 6, 2017

@Zap-zapper



Разве что для тебя запрещено :pepe:
А если ты придумал новую машинерию, для роботики, ты можешь ее сделать, и на временной основе добавить в билд ТОЛЬКО для строительства, и не трогая карту. Когда фриз пройдет, добавить и на карту.А пока, игроки сами смогут построить твою машинерию(ВОУ ИГРОКИ МОГУТ СТРОИТЬ??? Я НЕ ЗНАЛ!)

@P-20
Copy link
Contributor

P-20 commented Sep 16, 2017

Теперь трубы скручиваются СИНГУЛЯРНОСТЬЮ!
@ZVee

@ZVee
Copy link
Contributor Author

ZVee commented Sep 16, 2017

@P-20 я проверил (на локалке по крайней мере), даже 4-я стадия ничего не "скручивает", да и в коде https://github.com/TauCetiStation/TauCetiClassic/pull/1771/files#diff-37b445d0e5194e89fff026424b6a9784R221 четко прописано на пятую стадию. Так что или в репорте причина была в другом или какую-то деталь не договорили.

@volas
Copy link
Member

volas commented Sep 19, 2017

Атмос смержен, думаю можно закрыть и закончить фриз

@volas volas closed this Sep 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants