Difference between revisions of "Talk:Artifacts"
(→Fixing some off-by-1 floor inaccuracies: new section) |
m (fix code block) |
||
| Line 5: | Line 5: | ||
The edits were reverted with no explanation, so I'd like to add more context here and get feedback from Margotbean on whether it's enough to justify making the changes. I now realize this [[Artifacts]] page also has the same inaccuracies, so if approved, I'll make the edits here along with the specific item pages. | The edits were reverted with no explanation, so I'd like to add more context here and get feedback from Margotbean on whether it's enough to justify making the changes. I now realize this [[Artifacts]] page also has the same inaccuracies, so if approved, I'll make the edits here along with the specific item pages. | ||
| − | The relevant source code is [https://github.com/veywrn/StardewValley/blob/master/StardewValley/Locations/MineShaft.cs#L2695 here] for the item with [https://stardewids.com/ item code] 96 ([[Dwarf Scroll I]]) which does not depend on floor number; and [https://github.com/veywrn/StardewValley/blob/master/StardewValley/Locations/MineShaft.cs#L2713 here] for items 97 ([[Dwarf Scroll II]]), 99 ([[Dwarf Scroll IV]]), 121 ([[Dwarvish Helm]]), 122 ([[Dwarf Gadget]]), etc. which do depend on floor number: <code>case 40:<code> is code for changing the type of item which is possible to receive upon entering floor 40. | + | The relevant source code is [https://github.com/veywrn/StardewValley/blob/master/StardewValley/Locations/MineShaft.cs#L2695 here] for the item with [https://stardewids.com/ item code] 96 ([[Dwarf Scroll I]]) which does not depend on floor number; and [https://github.com/veywrn/StardewValley/blob/master/StardewValley/Locations/MineShaft.cs#L2713 here] for items 97 ([[Dwarf Scroll II]]), 99 ([[Dwarf Scroll IV]]), 121 ([[Dwarvish Helm]]), 122 ([[Dwarf Gadget]]), etc. which do depend on floor number: <code>case 40:</code> is code for changing the type of item which is possible to receive upon entering floor 40. |
Similarly, as described in [[The_Mines#Floors]], the environment drastically changes on floors 40, 70, 80, 110, etc. so my proposed fix seems to align with the existing Mines wiki too: things are different between floor 39 and 40, rather than being different between floor 40 and 41. | Similarly, as described in [[The_Mines#Floors]], the environment drastically changes on floors 40, 70, 80, 110, etc. so my proposed fix seems to align with the existing Mines wiki too: things are different between floor 39 and 40, rather than being different between floor 40 and 41. | ||
Is this enough to justify my changes, or are you looking for something else? Thanks! [[User:Cooperka|Cooperka]] ([[User talk:Cooperka|talk]]) 15:56, 11 April 2023 (UTC) | Is this enough to justify my changes, or are you looking for something else? Thanks! [[User:Cooperka|Cooperka]] ([[User talk:Cooperka|talk]]) 15:56, 11 April 2023 (UTC) | ||
Revision as of 15:56, 11 April 2023
Fixing some off-by-1 floor inaccuracies
I recently made some edits to several specific artifact pages (such as Dwarvish Helm) to fix what I believe to be inaccuracies in the mine floors they can found on when tilling (for example, it should say floors 1-39 rather than floors 1-40, because the item cannot be found on floor 40 even if the soil were tillable).
The edits were reverted with no explanation, so I'd like to add more context here and get feedback from Margotbean on whether it's enough to justify making the changes. I now realize this Artifacts page also has the same inaccuracies, so if approved, I'll make the edits here along with the specific item pages.
The relevant source code is here for the item with item code 96 (Dwarf Scroll I) which does not depend on floor number; and here for items 97 (Dwarf Scroll II), 99 (Dwarf Scroll IV), 121 (Dwarvish Helm), 122 (Dwarf Gadget), etc. which do depend on floor number: case 40: is code for changing the type of item which is possible to receive upon entering floor 40.
Similarly, as described in The_Mines#Floors, the environment drastically changes on floors 40, 70, 80, 110, etc. so my proposed fix seems to align with the existing Mines wiki too: things are different between floor 39 and 40, rather than being different between floor 40 and 41.
Is this enough to justify my changes, or are you looking for something else? Thanks! Cooperka (talk) 15:56, 11 April 2023 (UTC)