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

Порт Р.П.А. с Сирены #729

Merged
merged 10 commits into from
May 26, 2024

Conversation

Us3rn4me228
Copy link
Contributor

@Us3rn4me228 Us3rn4me228 commented May 24, 2024

Описание PR

Продолжаем портировать годноту с Сирены. На этот раз настала очередь Р.П.А. - пинпоинтера аномалий, который является хайриском Научрука. Крайне полезная вещь, не теряйте.

Медиа
image
image
image
image

Проверки

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

Изменения
🆑 Username228

  • add: Добавлен Р.П.А. - пинпоинтер аномалий.
  • add: Добавлено задание для предателя на кражу Р.П.А.
  • add: Для Р.П.А. сделан новый спрайт.

kingmonkeyx and others added 6 commits May 24, 2024 00:05
Добавлен Р.П.А. - ручной поисковик аномалий (by That_is_not_Sloni122). Пинпоинтер, что указывает на все сущности с компонентом `Anomaly`. Также он добавлен в шкаф научрука. А ещё добавлена триторская цель на него.

*P.S. сборка не проверялась. Тесты не проводились.*

Co-authored-by: kingmonkeyx <kingmonkeyx@email>
Reviewed-on: https://codeberg.org/NekoDar/SS14-Sirena/pulls/40
Co-authored-by: kingmonkeyx <[email protected]>
Co-committed-by: kingmonkeyx <[email protected]>
Copy link
Contributor

github-actions bot commented May 24, 2024

RSI Diff Bot; head commit 7bfa966 merging into 2e3746e
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Objects/Devices/pinpointer.rsi

State Old New Status
pinpointer-anomaly Added

Edit: diff updated after 7bfa966

@Ratyyy
Copy link
Contributor

Ratyyy commented May 24, 2024

я не думаю, что это хорошая идея...

Copy link
Collaborator

@Schrodinger71 Schrodinger71 left a comment

Choose a reason for hiding this comment

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

Ошибки в билдтестах. Ещё бы для баланса этому устройству нужно бы сделать потребление энергии от батарейки

  Failed NoCargoBountyArbitrageTest [4 s]
  Stack Trace:
  
1)    at Content.IntegrationTests.Pair.TestPair.OnDirtyDispose() in /home/runner/work/space_station/space_station/Content.IntegrationTests/Pair/TestPair.Recycle.cs:line 32
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Content.IntegrationTests.Pair.TestPair.OnDirtyDispose()
   at Content.IntegrationTests.Pair.TestPair.DisposeAsync() in /home/runner/work/space_station/space_station/Content.IntegrationTests/Pair/TestPair.Recycle.cs:line 102
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Content.IntegrationTests.Pair.TestPair.DisposeAsync()
   at Content.IntegrationTests.Tests.CargoTest.NoCargoBountyArbitrageTest() in /home/runner/work/space_station/space_station/Content.IntegrationTests/Tests/CargoTest.cs:line 93
   at System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1.AsyncStateMachineBox`1.ExecutionContextCallback(Object s)
Failed NoStaticPriceAndStackPrice [42 ms]
  Error Message:
     Assert.That(delta, Is.EqualTo(targetDelta))
  Expected: 1
  But was:  0

  Stack Trace:
     at Content.IntegrationTests.Pair.TestPair.SyncTicks(Int32 targetDelta) in /home/runner/work/space_station/space_station/Content.IntegrationTests/Pair/TestPair.Timing.cs:line 60
   at Content.IntegrationTests.PoolManager.GetServerClientPair(PoolSettings poolSettings) in /home/runner/work/space_station/space_station/Content.IntegrationTests/PoolManager.cs:line 248
   at Content.IntegrationTests.PoolManager.GetServerClientPair(PoolSettings poolSettings) in /home/runner/work/space_station/space_station/Content.IntegrationTests/PoolManager.cs:line 266
   at Content.IntegrationTests.PoolManager.GetServerClient(PoolSettings poolSettings) in /home/runner/work/space_station/space_station/Content.IntegrationTests/PoolManager.cs:line 192
   at Content.IntegrationTests.Tests.CargoTest.NoStaticPriceAndStackPrice() in /home/runner/work/space_station/space_station/Content.IntegrationTests/Tests/CargoTest.cs:line 99
   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
   at NUnit.Framework.Internal.AsyncToSyncAdapter.Await[TResult](Func`1 invoke)
   at NUnit.Framework.Internal.AsyncToSyncAdapter.Await(Func`1 invoke)
   at NUnit.Framework.Internal.Commands.TestMethodCommand.RunTestMethod(TestExecutionContext context)
   at NUnit.Framework.Internal.Commands.TestMethodCommand.Execute(TestExecutionContext context)
   at NUnit.Framework.Internal.Execution.SimpleWorkItem.<>c__DisplayClass3_0.<PerformWork>b__0()
   at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)

1)    at Content.IntegrationTests.Pair.TestPair.SyncTicks(Int32 targetDelta) in /home/runner/work/space_station/space_station/Content.IntegrationTests/Pair/TestPair.Timing.cs:line 60
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Content.IntegrationTests.Pair.TestPair.SyncTicks(Int32 targetDelta)
   at Content.IntegrationTests.PoolManager.GetServerClientPair(PoolSettings poolSettings) in /home/runner/work/space_station/space_station/Content.IntegrationTests/PoolManager.cs:line [248](https://github.com/xtray85/space_station/actions/runs/9217012454/job/25358304114?pr=729#step:10:249)
   at System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1.AsyncStateMachineBox`1.ExecutionContextCallback(Object s)

@Us3rn4me228
Copy link
Contributor Author

Ошибки в билдтестах. Ещё бы для баланса этому устройству нужно бы сделать потребление энергии от батарейки

  Failed NoCargoBountyArbitrageTest [4 s]
  Stack Trace:
  
1)    at Content.IntegrationTests.Pair.TestPair.OnDirtyDispose() in /home/runner/work/space_station/space_station/Content.IntegrationTests/Pair/TestPair.Recycle.cs:line 32
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Content.IntegrationTests.Pair.TestPair.OnDirtyDispose()
   at Content.IntegrationTests.Pair.TestPair.DisposeAsync() in /home/runner/work/space_station/space_station/Content.IntegrationTests/Pair/TestPair.Recycle.cs:line 102
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Content.IntegrationTests.Pair.TestPair.DisposeAsync()
   at Content.IntegrationTests.Tests.CargoTest.NoCargoBountyArbitrageTest() in /home/runner/work/space_station/space_station/Content.IntegrationTests/Tests/CargoTest.cs:line 93
   at System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1.AsyncStateMachineBox`1.ExecutionContextCallback(Object s)
Failed NoStaticPriceAndStackPrice [42 ms]
  Error Message:
     Assert.That(delta, Is.EqualTo(targetDelta))
  Expected: 1
  But was:  0

  Stack Trace:
     at Content.IntegrationTests.Pair.TestPair.SyncTicks(Int32 targetDelta) in /home/runner/work/space_station/space_station/Content.IntegrationTests/Pair/TestPair.Timing.cs:line 60
   at Content.IntegrationTests.PoolManager.GetServerClientPair(PoolSettings poolSettings) in /home/runner/work/space_station/space_station/Content.IntegrationTests/PoolManager.cs:line 248
   at Content.IntegrationTests.PoolManager.GetServerClientPair(PoolSettings poolSettings) in /home/runner/work/space_station/space_station/Content.IntegrationTests/PoolManager.cs:line 266
   at Content.IntegrationTests.PoolManager.GetServerClient(PoolSettings poolSettings) in /home/runner/work/space_station/space_station/Content.IntegrationTests/PoolManager.cs:line 192
   at Content.IntegrationTests.Tests.CargoTest.NoStaticPriceAndStackPrice() in /home/runner/work/space_station/space_station/Content.IntegrationTests/Tests/CargoTest.cs:line 99
   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
   at NUnit.Framework.Internal.AsyncToSyncAdapter.Await[TResult](Func`1 invoke)
   at NUnit.Framework.Internal.AsyncToSyncAdapter.Await(Func`1 invoke)
   at NUnit.Framework.Internal.Commands.TestMethodCommand.RunTestMethod(TestExecutionContext context)
   at NUnit.Framework.Internal.Commands.TestMethodCommand.Execute(TestExecutionContext context)
   at NUnit.Framework.Internal.Execution.SimpleWorkItem.<>c__DisplayClass3_0.<PerformWork>b__0()
   at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)

1)    at Content.IntegrationTests.Pair.TestPair.SyncTicks(Int32 targetDelta) in /home/runner/work/space_station/space_station/Content.IntegrationTests/Pair/TestPair.Timing.cs:line 60
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Content.IntegrationTests.Pair.TestPair.SyncTicks(Int32 targetDelta)
   at Content.IntegrationTests.PoolManager.GetServerClientPair(PoolSettings poolSettings) in /home/runner/work/space_station/space_station/Content.IntegrationTests/PoolManager.cs:line [248](https://github.com/xtray85/space_station/actions/runs/9217012454/job/25358304114?pr=729#step:10:249)
   at System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1.AsyncStateMachineBox`1.ExecutionContextCallback(Object s)

знал бы я как их фиксить ещё и что их вызывает
а сам РПА итак забаланшен тем, что это хайриск предмет, на который есть задание у синдиката
по сему делать пинпоинтер на батарейках малость бессмысленно

@Schrodinger71
Copy link
Collaborator

а ну всё, вроде пропали ошибки

@Us3rn4me228
Copy link
Contributor Author

а ну всё, вроде пропали ошибки

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

@KashRas2
Copy link
Collaborator

описание лучше поменять на что-то благоприятное

@Schrodinger71 Schrodinger71 merged commit 967b996 into AdventureTimeSS14:master May 26, 2024
12 of 14 checks passed
@Us3rn4me228 Us3rn4me228 deleted the anomaly_pinpointer branch May 26, 2024 19:06
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.

4 participants