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

Минус ноль в UI консоли шаттла #715

Closed
wants to merge 1 commit into from
Closed

Минус ноль в UI консоли шаттла #715

wants to merge 1 commit into from

Conversation

Lazzi0706
Copy link

@Lazzi0706 Lazzi0706 commented Feb 5, 2024

Описание PR

Убран -0.0 с UI консоли управления шаттла
Медиа

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

@github-actions github-actions bot added the Changes: UI Изменение затронуло файлы ".xaml" label Feb 5, 2024
@Lazzi0706 Lazzi0706 changed the title acceleration rounding Минус ноль в UI консоли шаттла Feb 5, 2024
Copy link
Member

@TheArturZh TheArturZh left a comment

Choose a reason for hiding this comment

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

Округление до целого это слишком сильно, нужен 1 знак после запятой как раньше
Между 0.51 тайла в секунду и 1 тайлом в секунду есть существенная разница
https://learn.microsoft.com/ru-ru/dotnet/api/system.math.round?view=net-8.0#system-math-round(system-double-system-int32-system-midpointrounding)

GridLinearVelocity.Text = $"{gridVelocity.X:0.0}, {gridVelocity.Y:0.0}";
GridAngularVelocity.Text = $"{-gridBody.AngularVelocity:0.0}";
GridLinearVelocity.Text = $"{Math.Round(gridVelocity.X):0.0}, {Math.Round(gridVelocity.Y):0.0}";
GridAngularVelocity.Text = $"{Math.Round(gridBody.AngularVelocity):0.0}";
Copy link
Member

@TheArturZh TheArturZh Feb 5, 2024

Choose a reason for hiding this comment

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

зачем убран минус?
получается AngularVelocity направлен полностью в другую сторону

Copy link
Author

Choose a reason for hiding this comment

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

зачем убран минус? получается AngularVelocity направлен полностью в другую сторону

Окей, сейчас доделаю другие вещи и займусь этим

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Mar 16, 2024
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@ZoNeSRuS
Copy link

@Ady4ik Офы уже поменяли консоль шаттла. Думаю пр можно просто закрыть.

@Ady4ik
Copy link

Ady4ik commented Apr 15, 2024

@DexlerXD закрываем

@DexlerXD DexlerXD closed this Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: UI Изменение затронуло файлы ".xaml" Merge Conflict Some content has conflicts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants