Skip to content

Fix CO2 Compactor descriptions#257

Merged
ChefZander merged 3 commits intoChefZander:masterfrom
AbbyRead:description-fixes
Mar 2, 2026
Merged

Fix CO2 Compactor descriptions#257
ChefZander merged 3 commits intoChefZander:masterfrom
AbbyRead:description-fixes

Conversation

@AbbyRead
Copy link
Copy Markdown
Contributor

Partly this is to give a subscript to the chemical formula and to fix a typo ("equivilent"). But mostly it's to fix a misleading bit of mouseover text. I left CO2 non-subscripted in places where someone might try searching for it like the quest book and the recipe encyclopedia.

I'm still not sure why you can pass through CO2 Compactors, but it's obvious from the code that it was intended that you can. If anyone can tell me why they need to be pass-through, I'm very interested because I'm drawing a blank.

Partly this is to give a subscript to the chemical formula and to fix a typo.  But mostly it's to fix a misleading bit of mouseover text.  I left CO2 non-subscripted in places where someone might try searching for it like the quest book and the recipe encyclopedia.
Copy link
Copy Markdown
Contributor

@TheEt1234 TheEt1234 left a comment

Choose a reason for hiding this comment

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

unsure if this is better (does it look better in-game)

Comment thread mods/sbz_bio/habitat.lua
core.register_node('sbz_bio:co2_compactor', {
description = 'CO2 Compactor',
info_extra = "Stores 30 co2. Habitat regulator doesn't consider it a wall, similar to how airlock works",
info_extra = "Stores 30 CO₂. Habitat regulator doesn't consider it a wall, but it can be passed through",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice

Comment thread mods/sbz_bio/habitat.lua Outdated
'CO2 supply: ',
'CO₂ supply: ',
math.max(co2_supply, co2_supply_temp),
'\nCO2 demand: ',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

CO₂ demand

also what other places does CO2 show up

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh, good catch. I was searching whole word and didn't consider that it would think printf style escape sequences were part of the word.

This is all I found from file search, but I'll do another without whole-word on. I think it's still good to leave it as CO2 in some places for searchability though. Just not necessary to do so beyond that, imo. If there was a way to tag these things on the back-end that would be even better, but I haven't looked into that.

@AbbyRead
Copy link
Copy Markdown
Contributor Author

I think that's everything. This is the regex expression I used to trim out as many false positives as possible:
(?<!_|local |return |\+ |- )(CO2|co2)(?!_| =| \+| -)

We maybe could use CO₂ in the code itself, but I think that would be harder to maintain if we did. So I just left anything that's a variable or a string used for code the way it was.

@AbbyRead
Copy link
Copy Markdown
Contributor Author

unsure if this is better (does it look better in-game)

I think it does. It's easier to recognize it as a chemical formula and not just some weird acronym. More sophisticated.

@TheEt1234
Copy link
Copy Markdown
Contributor

ok

@ChefZander ChefZander merged commit 1d2f62d into ChefZander:master Mar 2, 2026
@AbbyRead AbbyRead deleted the description-fixes branch March 2, 2026 18:17
AbbyRead pushed a commit to AbbyRead/skyblock_zero that referenced this pull request Mar 9, 2026
AbbyRead pushed a commit to AbbyRead/skyblock_zero that referenced this pull request Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants