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

Tweak: Changed max light for borgs #6314

Merged

Conversation

Koko4yXa
Copy link
Contributor

@Koko4yXa Koko4yXa commented Jan 4, 2025

Описание

Немного изменил мощность лампы боргов, сделав что-то среднее между старым и новым

Причина создания ПР / Почему это хорошо для игры

При реворке света был нещадно урезан фонарь боргов, компенсируем, но не до конца из-за приколов нового света

Демонстрация изменений

Старая макс яркость теперь на 4 мощности
image
image

Тесты

включил лампочку, выключил лампочку

Усреднил старый радиус освещения боргов с новым
@github-actions github-actions bot added the DMI Affects DMI files label Jan 4, 2025
@Koko4yXa
Copy link
Contributor Author

Koko4yXa commented Jan 4, 2025

Кто нибудь объясните откуда миллион пустых правок и куда я забыл добавить новую иконку

@FlitchTime
Copy link
Contributor

FlitchTime commented Jan 5, 2025

Кто нибудь объясните откуда миллион пустых правок и куда я забыл добавить новую иконку

Где-то тут
image

@Koko4yXa
Copy link
Contributor Author

Koko4yXa commented Jan 5, 2025

Кто нибудь объясните откуда миллион пустых правок и куда я забыл добавить новую иконку

Где-то тут image

Иконка лежит, загружал по идее

@Koko4yXa
Copy link
Contributor Author

Koko4yXa commented Jan 5, 2025

image
вот в списке файлов

@BeebBeebBoob
Copy link
Member

Пересоздай light_554.dmi , айкондифф не может прочитать иконку

@Koko4yXa
Copy link
Contributor Author

Koko4yXa commented Jan 6, 2025

Пересоздай light_554.dmi , айкондифф не может прочитать иконку

Сделал. В чем проблема была не понял, но теперь не ругается

@BeebBeebBoob BeebBeebBoob merged commit 433c4a1 into ss220-space:master220 Jan 6, 2025
12 checks passed
Koko4yXa added a commit to Koko4yXa/MyParadise that referenced this pull request Jan 7, 2025
@Koko4yXa
Copy link
Contributor Author

Koko4yXa commented Jan 7, 2025

@BeebBeebBoob Если мне надо зафиксить уже замерженую ветку, мне новую создавать или так норм?

Koko4yXa added a commit to Koko4yXa/MyParadise that referenced this pull request Jan 7, 2025
@BeebBeebBoob
Copy link
Member

Да. Обнови у себя master220 до актуалки, потом новую ветку и там фикс, делаешь ПР и готово

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DMI Affects DMI files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants