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

Mass Engine Update #1220

Merged
merged 145 commits into from
Nov 17, 2024
Merged

Mass Engine Update #1220

merged 145 commits into from
Nov 17, 2024

Conversation

sleepyyapril
Copy link
Contributor

@sleepyyapril sleepyyapril commented Nov 14, 2024

Description

An attempt to do all engine updates in one go. Not focusing on content associated, only what's needed for it to run and any bug fixes.


TODO

  • Fix bug where unbuckling resets you to lying down.
  • Fix bug where you can no longer get up after lying down.
  • See what else I broke.

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: YML Changes any yml files labels Nov 14, 2024
@sleepyyapril

This comment was marked as resolved.

@sleepyyapril
Copy link
Contributor Author

I have run into a problem I do not know how to fix. Woe.

@OCOtheOmega
Copy link

I have run into a problem I do not know how to fix. Woe.

That why engine update for EE is pain and @VMSolidus take a lot of time for updates.

@FoxxoTrystan
Copy link
Member

OH LORD...

ENGINES UPDATES.... ITS AGONY.

ElectroJr and others added 5 commits November 14, 2024 14:29
* partial buckling refactor

* git mv test

* change test namespace

* git mv test

* Update test namespace

* Add pulling test

* Network BuckleTime

* Add two more tests

* smelly
* Replaced uses of Dirty(Component) with Dirty(Uid, Component)
Modified some systems (notably pulling-related) to use uids.

* Missed a few

* Revert changes to pulling

* No
@VMSolidus
Copy link
Member

By request, I have implemented space-wizards/space-station-14#26238
in this PR.

@VMSolidus
Copy link
Member

@DEATHB4DEFEAT DEATHB4DEFEAT removed the Status: Do Not Merge Do not merge label Nov 17, 2024
@OCOtheOmega
Copy link

Before merge i recommend wait until new version published with fix for console space-wizards/RobustToolbox#5507 and some stuff for modern HWID.

@FoxxoTrystan FoxxoTrystan requested review from FoxxoTrystan, a team, VMSolidus, DEATHB4DEFEAT and Pspritechologist and removed request for a team November 17, 2024 15:07
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Nov 17, 2024
@FoxxoTrystan FoxxoTrystan added Priority: 2-High Needs to be resolved as soon as possible Holy Shit Size: 1-Very Large For especially large issues/PRs labels Nov 17, 2024
Copy link
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

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

Alright let's fucking go.

@FoxxoTrystan FoxxoTrystan added the Status: Awaiting Merge 2 or more maintainers has approved this PR and is now awaiting merge label Nov 17, 2024
@VMSolidus VMSolidus merged commit f95774b into Simple-Station:master Nov 17, 2024
17 checks passed
@sleepyyapril sleepyyapril deleted the engine-update branch November 17, 2024 20:00
VMSolidus pushed a commit that referenced this pull request Nov 18, 2024
<!--
This is a semi-strict format, you can add/remove sections as needed but
the order/format should be kept the same
Remove these comments before submitting
-->

# Description

<!--
Explain this PR in as much detail as applicable

Some example prompts to consider:
How might this affect the game? The codebase?
What might be some alternatives to this?
How/Who does this benefit/hurt [the game/codebase]?
-->

Goal of this PR is to fix the stamina system, as was caused by #1220.

---

# Changelog

<!--
You can add an author after the `:cl:` to change the name that appears
in the changelog (ex: `:cl: Death`)
Leaving it blank will default to your GitHub display name
This includes all available types for the changelog
-->

:cl:
- fix: Fixed stamina system

Co-authored-by: sleepyyapril <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>
@sleepyyapril sleepyyapril mentioned this pull request Nov 20, 2024
VMSolidus pushed a commit that referenced this pull request Nov 22, 2024
<!--
This is a semi-strict format, you can add/remove sections as needed but
the order/format should be kept the same
Remove these comments before submitting
-->

# Description

<!--
Explain this PR in as much detail as applicable

Some example prompts to consider:
How might this affect the game? The codebase?
What might be some alternatives to this?
How/Who does this benefit/hurt [the game/codebase]?
-->

Mass bug fixing, for bugs related to #1220.
Feel free to link or send bugs.

Fix list:
- #1242 
- #1243 
- #1244
- space-wizards/space-station-14#28084
- space-wizards/space-station-14#28282
- Actually fixed PirateRadioSpawnRule heisentest (with a bandaid) (I
cancel if it's 0)
- #1263

---

# Changelog

<!--
You can add an author after the `:cl:` to change the name that appears
in the changelog (ex: `:cl: Death`)
Leaving it blank will default to your GitHub display name
This includes all available types for the changelog
-->

:cl:
- fix: Fixed chair visuals drawing depth wrongly if you sat in a
north-facing chair.
- fix: Fixed buckling doing several buckles each time you did one.
- fix: Fixed the magic mirror.
- fix: Fixed beds re-positioning you every few seconds.
- fix: Fixed E not opening containers that are in another container.
- fix: Fixed disposal systems not flushing or ejecting properly.

---------

Co-authored-by: sleepyyapril <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>
Co-authored-by: Nemanja <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Config Changes any configuration files Changes: Documentation Changes any xml or md files Changes: Localization Changes any ftl files Changes: Map Changes any yml file in the Maps directories Changes: Sprite Changes any png or json in an RSI Changes: UI Changes any XAML files Changes: YML Changes any yml files Holy Shit Priority: 2-High Needs to be resolved as soon as possible Size: 1-Very Large For especially large issues/PRs Status: Awaiting Merge 2 or more maintainers has approved this PR and is now awaiting merge Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.