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

Консоль для мониторинга температуры и давления #1138

Merged
merged 109 commits into from
Apr 19, 2024

Conversation

VentelR
Copy link

@VentelR VentelR commented Mar 26, 2024

Что этот PR делает

Добавляет консоль с новым анимированным спрайтом экрана консоли, новым TGUI для отображения графиков температуры и давления, без необходимости игроку быть рядом с консолью (process)

Почему это хорошо для игры

Позволяет с помощью мультиметра подключить к консоли Gas Sensor, и Meter (RPD => devices), и видеть на графике изменение показателей температуры и давления в динамике.
Полезно для инженерного отдела, потому как наглядная визуализация всегда лучше сухих цифр.

Изображения изменений

СМ еще никогда не горел так информативно
1
2

5850147e85193d37
Плата с наименованием Atmospheric Graph Monitor, для печати необходимы programin =2, engineering =3
circuit
green

UPD

Консоли расположены в следующих точках:
Дельта (пришлось украсть стол из пластали и перенести сварочную маску)
дельта
Кибериада
Кибериада
Метастейшон
Цереброн
подсказка

Тестирование

Тестировал на локальном сервере:
Протестировано создание платы в автомате для печати плат;
Протестированы отображение показателей в динамике;
Протестировано отображение спрайта экрана консоли в разных направлениях;

Для тестирования с помощью Game Panel необходимо найти консоль pt_monitor.

Changelog

🆑
add: Добавлена консоль для мониторинга температуры и давления. Датчики к ней можно подключить с помощью мультитула.
/:cl:

VentelR and others added 30 commits March 21, 2024 01:35
+ инициализируются требуемого размера (null значениями)
+ изменен интервал записи и размер массива
+ в отсутствие данных с сенсора записывается null значение
This reverts commit a9de59c.
+ инициализируются требуемого размера (null значениями)
+ изменен интервал записи и размер массива
+ в отсутствие данных с сенсора записывается null значение
@VentelR VentelR marked this pull request as draft April 9, 2024 15:14
@VentelR VentelR marked this pull request as ready for review April 9, 2024 15:24
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Apr 9, 2024
@VentelR
Copy link
Author

VentelR commented Apr 9, 2024

Еще раз конфликт тгуиуи

Готово, секунды исправлены, бандл и мердж сделан.

Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

Мне всё ещё не нравится копипизднутый код чарта, но да похуй, вроде всё норм

Comment on lines 46 to 50
const maxWingowHeight = 900;
const getWindowHeight = Math.min(
maxWingowHeight,
Object.keys(data.sensors).length * 220 + 150
);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
const maxWingowHeight = 900;
const getWindowHeight = Math.min(
maxWingowHeight,
Object.keys(data.sensors).length * 220 + 150
);
const getWindowHeight = Math.min(
900,
Object.keys(data.sensors).length * 220 + 150
);

@VentelR VentelR marked this pull request as draft April 9, 2024 19:45
@VentelR VentelR marked this pull request as ready for review April 13, 2024 10:11
Comment on lines 284 to 290
horizontalLinesCount = 2,
verticalLinesCount = data.length - 2,
gridColor = 'rgba(255, 255, 255, 0.1)',
gridWidth = 2,
pointTextColor = 'rgba(255, 255, 255, 0.8)',
pointTextSize = '0.8em',
labelViewBoxSize = 400,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
horizontalLinesCount = 2,
verticalLinesCount = data.length - 2,
gridColor = 'rgba(255, 255, 255, 0.1)',
gridWidth = 2,
pointTextColor = 'rgba(255, 255, 255, 0.8)',
pointTextSize = '0.8em',
labelViewBoxSize = 400,
horizontalLinesCount = 0,
verticalLinesCount = 0,
gridColor = 'rgba(255, 255, 255, 0.1)',
gridWidth = 2,
pointTextColor = 'rgba(255, 255, 255, 0.8)',
pointTextSize = '0.8em',
labelViewBoxSize = 0,

Я бы сделал по умолчанию как-то так, а нужные значения уже пропсами выставить

@github-actions github-actions bot added the Merge Conflict Ну блять... label Apr 18, 2024
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Apr 19, 2024
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

Жалоб на производительность во время ТМа не было, рантаймов я сам на нашел в логисе, интерфейс в целом нормик, больше нечего добавить

@AyIong
Copy link
Collaborator

AyIong commented Apr 19, 2024

Fuck it we balls

@AyIong AyIong merged commit 5ac0f86 into ss220club:master Apr 19, 2024
13 checks passed
@VentelR VentelR deleted the Test_PT_monitor branch April 19, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден Testmerge Required TGUI Кто-то решил потрогать интерфейсы. 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation) Цереброн Этот ПР изменил что-то на Цереброне (MetaStation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants