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

[MIRROR] General maintenance for frames #2147

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1155
Original PR: tgstation/tgstation#81473

About The Pull Request

Builds upon the changes in tgstation/tgstation#81477

1. Qol

  • Adds detailed examines & screentips for building & deconstructing both machine & computer frames
  • Adding a circuitboard from a rped to a computer frame will automatically screw it in place like before

2. Code Improvements

  • Merged procs like update_path_names() , get_requested_amt() etc into their required places to reduce proc calls
  • Autodocs procs like add_cable(), add_glass() etc. makes them private
  • Moved code for machine frame into its own file

Changelog

🆑 SyncIt21
qol: adds examines & screentips for building & deconstructing both machine & computer frames.
qol: Adding a circuitboard from a rped to n computer frame will automatically screw it in place like before.
code: merged procs for computer & machine frames. autodocs them where possible.
code: moved code for machine frame into its own file.
/:cl:

* General maintenance for frames (#81473)

## About The Pull Request
Builds upon the changes in
tgstation/tgstation#81477

**1. Qol**
- Adds detailed examines & screentips for building & deconstructing both
machine & computer frames
- Adding a circuitboard from a rped to a computer frame will
automatically screw it in place like before

**2. Code Improvements**
- Merged procs like `update_path_names()` , `get_requested_amt()` etc
into their required places to reduce proc calls
- Autodocs procs like `add_cable()`, `add_glass()` etc. makes them
private
- Moved code for machine frame into its own file

* General maintenance for frames

---------

Co-authored-by: SyncIt21 <[email protected]>
@Iajret Iajret merged commit 5ec5eb8 into master Feb 29, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1155 branch February 29, 2024 18:19
AnywayFarus added a commit that referenced this pull request Feb 29, 2024
Iajret pushed a commit that referenced this pull request Apr 23, 2024
* Plumbing & chem reaction chamber patches (#82781)

## About The Pull Request
1. Both Plumbing & Chem reaction chamber will use the beakers
`heat_capacity()` proc instead of the formulae `SPECIFIC_HEAT_DEFAULT *
reagents.total_volume`. This yields the exact same results as before but
will mean in the future if individual reagents heat capacities are
changed then the heating effects will be accurately reflected here. It
also uses that amount of power for heating
2. Plumbing rection chamber will not use energy when its either
'emptying' or when no reagents are present in its internal buffer
3. You can hit the chem reaction chamber with items like beakers,
screwdrivers, crowbars and what not

## Changelog
:cl:
fix: plumbing & chem reaction chamber heating effects accurately reflect
the beakers heat capacity & power usage
fix: plumbing reaction will not use power when emptying or when there
are no reagents to heat
fix: you can hit the chem reaction chamber with items like beakers,
screwdrivers, crowbars and what not
/:cl:

* Plumbing & chem reaction chamber patches

---------

Co-authored-by: SyncIt21 <[email protected]>
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.

3 participants