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

feat(content, mods/aftershock): Return vehicle freezers/fridges to aftershock #4505

Merged
merged 2 commits into from
May 7, 2024

Conversation

KheirFerrum
Copy link
Collaborator

@KheirFerrum KheirFerrum commented Apr 21, 2024

Purpose of change

Describe the solution

  • Adds back the full size vehicle fridges/freezers to Aftershock. Adds the ability to install the glass variants as well (though benefit is minimal, except that light on those tiles will now radiate out.) though the epower cost is 20 points higher (to reflect the energy inefficiency of glass fridges/freezers both in real life and in vanilla.
  • Removed an unused construction group while I was at it.

Describe alternatives you've considered

Testing

  • Start a world with Aftershock, make a vehicle and install the fridge/freezer into them.
    • Check that each installation uses the appropriate materials (glass fridges use glass fridges not normal fridges)
  • Check that items placed into them once turned on has the appropriate timer displayed.

Additional context

It's really odd to me that this was only noticed recently.

One odd quirk of the system is that the "turn on freezer/fridge" vehicle command doesn't allow you to selectively turn them on, they will turn on all corresponding vehicle parts. (Luckily it checks for flags and not IDs so the glass freezer/fridge both will be appropriately... turned on)

Checklist

@github-actions github-actions bot added JSON related to game datas in JSON format. mods PR changes related to mods. labels Apr 21, 2024
@KheirFerrum KheirFerrum changed the title feat(content, mods/Aftershock): Return vehicle freezers/fridges to aftershock feat(content, mods/aftershock): Return vehicle freezers/fridges to aftershock Apr 21, 2024
@VissValdyr
Copy link
Contributor

To be perfectly honest, full size freezers can be in a vehicle too. Especially RVs can benefit from them. https://images.app.goo.gl/JohSQgGzArRuWykK9

@scarf005
Copy link
Member

if so, should full-size fridges be mainlined too?

@VissValdyr
Copy link
Contributor

if so, should full-size fridges be mainlined too?

I would say yes. We can fit a forge into a car, so why not a regular sized fridge. Make the full sized fridge even spawn into RVs (regular and deluxe). Mini fridges will still be part of limousines, so they still spawn.

I would support the full size fridge into mainline.

@RoyalFox2140
Copy link
Collaborator

In that PR I knew I was removing full-sized vehicle freezers but figured if it was necessary to have them they needed to be mainlined content. I simply wanted to remove redundancy from aftershock in as quick as possible. Adding them back to Aftershock would put it back in obscurity.

@Relgar
Copy link
Contributor

Relgar commented Apr 22, 2024

if so, should full-size fridges be mainlined too?

I would say yes. We can fit a forge into a car, so why not a regular sized fridge. Make the full sized fridge even spawn into RVs (regular and deluxe). Mini fridges will still be part of limousines, so they still spawn.

I would support the full size fridge into mainline.

This would be great - I just took a look at chest freezers and apparently those can store a whopping SEVEN HUNDRED LITERS compared to the upright freezers of 300L (current standard), so this could also be a future addition as well.

@KheirFerrum
Copy link
Collaborator Author

if so, should full-size fridges be mainlined too?

I would say yes. We can fit a forge into a car, so why not a regular sized fridge. Make the full sized fridge even spawn into RVs (regular and deluxe). Mini fridges will still be part of limousines, so they still spawn.

I would support the full size fridge into mainline.

My issue with mainlining it is that there's no discernible drawback between minifridges/freezers vs their full size variants. If for instance we made minifreezers/fridges more compatible with other vehicle parts, or full size versions less compatible, I'd be more persuaded. That said. I'd rather that be a different PR discussion entirely.

@VissValdyr
Copy link
Contributor

VissValdyr commented Apr 22, 2024

if so, should full-size fridges be mainlined too?

I would say yes. We can fit a forge into a car, so why not a regular sized fridge. Make the full sized fridge even spawn into RVs (regular and deluxe). Mini fridges will still be part of limousines, so they still spawn.
I would support the full size fridge into mainline.

My issue with mainlining it is that there's no discernible drawback between minifridges/freezers vs their full size variants. If for instance we made minifreezers/fridges more compatible with other vehicle parts, or full size versions less compatible, I'd be more persuaded. That said. I'd rather that be a different PR discussion entirely.

Draw backs should be: no see through, more weight, more energy consumption, no mountable (passable), not board able. Enough to consider.

Also those are special fridges, home fridges can't be used for that. So only found ones can be installed

Copy link
Collaborator

@RoyalFox2140 RoyalFox2140 left a comment

Choose a reason for hiding this comment

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

@chaosvolt @scarf005 This has been good to merge back in. Not sure what the hold up was, but it's fine to add it back after I took it out without supplying an alternative.

Copy link
Member

@chaosvolt chaosvolt left a comment

Choose a reason for hiding this comment

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

¯\(ツ)/¯.

@scarf005 scarf005 merged commit 8dbe849 into cataclysmbnteam:main May 7, 2024
10 checks passed
@KheirFerrum KheirFerrum deleted the AFS-Freezers branch May 8, 2024 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JSON related to game datas in JSON format. mods PR changes related to mods.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants