search

DuelMonster / MinersAdvantage

MinersAdvantage is a reimagening of my previous mod 'SuperMiner'. It is a rewrite from the ground up that aims to solve all previous issues and to add additional features.
GNU Lesser General Public License v3.0
21 stars 1 forks source link

Illumination puts torch on every digged block regardless of light level #6

Closed Knito58 closed 6 years ago

Knito58 commented 6 years ago

Issue description:

When digging a shaft a torch is placed at every step

What happens:

What I said :). The light level is not taken into account.

What you expected to happen:

Torch is placed at confgured "lowest light level".

Does this issue occur on a server or single player world?

Steps to reproduce:

  1. Illumination on
  2. Have torches
  3. Dig a shaft

Versions (make sure you are on the latest version before reporting):

Crash log:

No crash no log :)

DuelMonster commented 6 years ago

Could you please retry after updating forge to the current recommended version 14.23.3.2655

Thanks

Knito58 commented 6 years ago

Forge Version 14.23.3.2666: Same error. Even in the brightest place, when I hit a block then a torch is placed. A shaft = a line full of torches placed one after the other.

DuelMonster commented 6 years ago

OK I'll take a look at it tonight. Thnx

DuelMonster commented 6 years ago

@Knito58 I'm now wondering whether this issue may be related to GravelMiner also... Would u mind removing/disabling GravelMiner and trying again? Thanks

I have just installed GravelMiner on my single player instance running forge 14.23.3.2655 and found that all is working as expected, so maybe it isn't GravelMiner... Please down grade to the recommended version of forge. Some of the latest versions can be flaky. 14.23.3.2657, for example, caused an awful lot of mods to crash .

Knito58 commented 6 years ago

Sorry. Now I have the recommended Forge 14.23.3.2655 and the error persists. Maybe I configured st wrongly in the Illumination section?

Illu enabled = true use block light level = false lowest light level = 4

Edit: Setting "use block light level" to TRUE fixes bug.

DuelMonster commented 6 years ago

AH, yes, without the "use block light level" being true MA uses the Sky light level.