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

/vg/ multitool menu port & more buildable atmos equipment #12

Closed
wants to merge 35 commits into from

Conversation

PJB3005
Copy link
Contributor

@PJB3005 PJB3005 commented Nov 30, 2014

ports /vg/ multitool menus, makes a lot more atmospherics equipment buildable & configurable with multitool menus (vents, air injectors, sensors, DP vents, Dvalves, etc...)

AAC: scripting atmos

PJB3005 added 2 commits November 30, 2014 13:46
ports VG multitool menus, and moves Tcomms to them

((think of multitool menus as the menu you get when you use a multitool
on Tcomms equipment))
@PJB3005
Copy link
Contributor Author

PJB3005 commented Nov 30, 2014

vents done, you can now change frequency and ID tag on them

@PJB3005
Copy link
Contributor Author

PJB3005 commented Nov 30, 2014

scrubbers done

@PJB3005
Copy link
Contributor Author

PJB3005 commented Nov 30, 2014

gas sensors, done, working on consoles that can use 'em

@PJB3005
Copy link
Contributor Author

PJB3005 commented Nov 30, 2014

air injectors

@PJB3005 PJB3005 changed the title VG multitool menu port VG multitool menu port & more buildable atmos equipment Nov 30, 2014
@PJB3005
Copy link
Contributor Author

PJB3005 commented Nov 30, 2014

Gas sensors and Air injectors buildable & multitool menu done

@mvanalphen
Copy link
Contributor

Looking mighty delicious 🌚 💯

@PJB3005
Copy link
Contributor Author

PJB3005 commented Nov 30, 2014

Thanks Mark

couple of atmos consoles, using this PR you should be able to rebuild the big atmos tanks

@nicky89107
Copy link
Contributor

Will this also allow you to create new airlocks?

@PJB3005
Copy link
Contributor Author

PJB3005 commented Nov 30, 2014

cycling ones? yes, once I get to porting that over, currently focusing more on atmos equipment

@PJB3005
Copy link
Contributor Author

PJB3005 commented Nov 30, 2014

Atmos automations console, A.K.A. scripting atmos
still buggy, probably

My completely not rushed super air conditioner for atmos, once the temp goes above 350°K, it opens up a vent connected to a cooled N2 canister, closes it when the room temperature reaches 290°K

also I didn't copy the sprite yet

@fj45
Copy link
Contributor

fj45 commented Nov 30, 2014

This looks very fun, can't wait to mess around with it. Having more automation would be really useful.

@ghost
Copy link

ghost commented Dec 2, 2014

Does it work with the shuttle controller we run?

@PJB3005
Copy link
Contributor Author

PJB3005 commented Dec 2, 2014

Sam, I didn't get to the cycling airlocks yet, I'll figure something out for that part, as /vg/ has no cycling airlocks prebuild on the map, and no support to use them for shuttles, not to mention /vg/ doesn't have things like shuttle "docking"

seriously why be a snowflake attackby()
@mvanalphen
Copy link
Contributor

You'll have to let Fox know when it's ready for merging.

@PJB3005
Copy link
Contributor Author

PJB3005 commented Dec 6, 2014

yeah, I know, been kinda procrastinating and stuff, gonna have to fix that damn merge conflict too

Fox-McCloud added a commit that referenced this pull request Dec 9, 2014
seriously some VG code was even broken
@PJB3005
Copy link
Contributor Author

PJB3005 commented Dec 10, 2014

So apperently this PR went from multitools to JSON code upgrades, okay then

GITHUB STOP BEING SHIT
@PJB3005
Copy link
Contributor Author

PJB3005 commented Jan 31, 2015

*goes insane about merge conflicts

@ZomgPonies
Copy link
Contributor

Fix the merge conflicts, make sure it's fox.dm and not Fox.dm in the dme for consistency and I'll make sure it's merged promptly this time.

@PJB3005
Copy link
Contributor Author

PJB3005 commented Feb 7, 2015

done!

@Fox-McCloud
Copy link
Member

@PJB3005

We still need to discuss this at length; could you please notify me (here or on the forums) of what day would be a good time?

@PJB3005
Copy link
Contributor Author

PJB3005 commented Feb 10, 2015

I'm not the kind of person that's good with specific moments, it's probably just best if we ping eachother on IRC when it's convenient

@ZomgPonies ZomgPonies added Feature This PR is a new addition to the game Do Not Merge This PR must not be merged or closed Controversial This PR has left people very conflicted labels Feb 11, 2015
PJB3005 added 8 commits February 13, 2015 22:34
…o vg-masterrace

Conflicts:
	code/game/machinery/machinery.dm
	code/modules/nano/_JSON.dm
	code/setup.dm
…ting) in multitool menu code, which I recently discovered.
past me, why the snowflaking
…act with vents or scrubbers, the vents or scrubbers in question need to have a setting switched in their multitool menu, now you can't nuke every vent and scrubber on the station to syphon with them anymore
…o vg-masterrace

Conflicts:
	code/game/machinery/atmoalter/meter.dm
fixes an oversight that can cause multitools to not work with multitool menus anymore (runtime error related)

sanity checks added to AAC
@mvanalphen
Copy link
Contributor

We've discussed the matter, and we've come to the conclusion that we'd rather not implement this feature on the server. The feature effectively allows you to do anything you want with the atmospherics system, as long as you have the board and know how to code. However, this knowledge is limited to a select number of people, and will probably not be used much, while those who do know how to use it will have a major advantage over others. While there's certainly a lot of "difficult" features in the game, we'd rather avoid adding another feature that requires extensive OOC knowledge.

If you'd like a more in-depth explanation, feel free to contact me/Fox/Ponies personally.

@mvanalphen mvanalphen closed this Feb 21, 2015
@PJB3005
Copy link
Contributor Author

PJB3005 commented Feb 24, 2015

really? rip 2500 lines of code ;_;

@PJB3005
Copy link
Contributor Author

PJB3005 commented Feb 24, 2015

would it still be unbalanced if I removed the AAC?

Also programming the AAC is easier than programming a lego mindstorms, i'm not a genious with atmos but even I can comeup with cool sit for the AAC, if this is unbalaned because it's an "OOC knowledg" thing then we can get ridof atmos, NTSL and toxins while we're at it

@tigercat2000
Copy link
Contributor

@PJB3005 for what it's worth, I would love to see this again now that we have LINDA

dunno how much that is worth tho

@PJB3005
Copy link
Contributor Author

PJB3005 commented May 1, 2015

go ahead and reopen it, but I'm not coding anything for it anymore.

if you want to make a PR to my fork or something go ahead.

@PJB3005 PJB3005 changed the title VG multitool menu port & more buildable atmos equipment /vg/ multitool menu port & more buildable atmos equipment May 14, 2015
Tastyfish pushed a commit to Tastyfish/Paradise that referenced this pull request May 31, 2016
Tilk1 pushed a commit to Tilk1/Paradise that referenced this pull request Dec 27, 2022
Update 29/11/22 [MDB IGNORE] [IDB IGNORE]
mattgroy pushed a commit to mattgroy/Paradise that referenced this pull request Nov 10, 2023
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## What Does This PR Do
<!-- Include a small to medium description of what your PR changes. -->
<!-- Document all changes, as not doing this may delay reviews or even
discourage maintainers from merging your PR! -->
<!-- If your PR fixes an issue, add "Fixes ParadiseSS13#1234" somewhere in the PR
description. This will automatically close the bug upon PR submission.
-->

* Changes default keybindings to SS220
* Adds translation to keybindings

## Why It's Good For The Game
<!-- Add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

Easier transition from old para

## Changelog
:cl:
add: Default Keybindings from SS220
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->
<!-- If a PR has no impact on players (i.e. a code refactor that does
not change functionality) then the entire Changelog heading and contents
can be removed. -->

---------

Co-authored-by: Furior <68264134+Furrior@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Controversial This PR has left people very conflicted Do Not Merge This PR must not be merged or closed Feature This PR is a new addition to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants