-
Notifications
You must be signed in to change notification settings - Fork 193
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
Benos #413
Benos #413
Conversation
Added aliens
Now stage 5 spawns larva inside of the host with an ability to gib it's host and turn to a normal larva
Added nests and fixed funny bug
Added infected icons
New hunter ability - leap
Added Plasma counter Added lusty xenomorpgh maiden added Lamarr
Alien egg spawner
https://drive.google.com/uc?export=download&id=1bRX8p-w7OSSD9QRHwISNcarjIr4O55O9 Showcase video link |
RSI Diff Bot; head commit 5b85df8 merging into c62f777 Resources/Textures/Interface/Alerts/plasma_counter.rsi
Resources/Textures/Interface/Alien/screen_alien.rsi
Resources/Textures/Interface/Misc/infected.rsi
Resources/Textures/Mobs/Aliens/Xenos/alien.rsi
Resources/Textures/Mobs/Aliens/Xenos/alienqueen.rsi
Resources/Textures/Mobs/Species/Xeno/organs.rsi
Resources/Textures/Structures/Walls/resin.rsi
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice surprise PR; I'm assuming you're still working on this so this is all I'll review
@@ -477,3 +477,25 @@ | |||
minimumPlayers: 20 | |||
weight: 5 | |||
- type: MobReplacementRule | |||
|
|||
- type: entity | |||
id: AlienInfestation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resources/Prototypes/DeltaV/GameRules/events.yml
XenoVentsRule
This needs to remove / replace / alter XenoVentsRule.
Now alien infestation replaces xeno vents gamerule
.rsi is the only valid test fail, others are related to Microsoft |
Can you please say, what is actually wrong with rsi. I am new to contributing... |
https://github.com/Simple-Station/Einstein-Engines/actions/runs/9147685738/job/25149458256?pr=413 It's all in here, it doesn't like how some of the licenses are typed in and some of the meta.jsons are missing states I think. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
null isn’t valid for license and version and yeah missing states.
did you also actually make the sprites for the plasma power display or are they ported from SS13 like the other CM stuff? As you’ll need to reference the commit it’s taken from. RSI edit is good for this.
Yuh, I'm also curious as to where these Sprites are from |
Background for them I've got from tgstation (license is written in screen-alien.rsi), and paitned numbers on it, cuz I haven't found any way to display actual numbers on status icon |
So I'll preface this that we aren't really going to be accepting sprites ported from /tg/station, when far better sprites can be found on Baystation and its downstreams, such as Paradise. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR lack a ton of documentation and TG Sprite/Gameplay seem, off...
I feel theres better ways to do this in SS14 then just following what SS13 Did.
namespace Content.Client.Aliens; | ||
|
||
/// <summary> | ||
/// This is used for... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary needed
namespace Content.Client.Aliens.Systems; | ||
|
||
/// <summary> | ||
/// This handles... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary Needed
SubscribeLocalEvent<EyeComponent, ToggleLightingAlienActionEvent>(OnToggleLighting); | ||
} | ||
|
||
private void OnStartup(EntityUid uid, AlienComponent component, ComponentStartup args) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function is not used at all, is it needed?
namespace Content.Client.Aliens.Systems; | ||
|
||
/// <summary> | ||
/// This handles... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary Needed
Also is this... System... used for anything?
namespace Content.Client.Overlays; | ||
|
||
/// <summary> | ||
/// This handles... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary Needed
namespace Content.Shared.Aliens.Systems; | ||
|
||
/// <summary> | ||
/// This handles... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary Needed
@@ -90,10 +90,12 @@ public enum ChatChannel : ushort | |||
/// </summary> | |||
Telepathic = 1 << 15, | |||
|
|||
XenoHivemind = 2 << 2, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be 16
/// </summary> | ||
Telepathic = ChatChannel.Telepathic, | ||
|
||
XenoHivemind = ChatChannel.XenoHivemind, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary Needed
namespace Content.Shared.Medical; | ||
|
||
/// <summary> | ||
/// This handles... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary Needed
namespace Content.Shared.Medical; | ||
|
||
/// <summary> | ||
/// This is used for... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary Needed
Closed as derelict; reopen if you come back to this :) |
Description
Alien infestation event from ss13. Also some of the facehugger bugs and missing localizing, shown in video are fixed by the time of publishing this PR.
TODO
-[] Add vent crawling, when this system will be implemented in future.
Media
Changelog
:cl Kai5: