JadedPacks / Agrarian-Skies-2

Agrarian Skies 2 modpack.
5 stars 4 forks source link

Progressive Automation Chopper Not Recognizing Rubber Trees #116

Open Kratargon opened 2 years ago

Kratargon commented 2 years ago

Bug Report

Progressive Automation choppers are capable of planting rubber tree saplings, however, they are unable to chop them.

Steps to Reproduce (for bugs)

  1. Create a chopper (I used a diamond chopper, as strong tools are needed to chop rubber trees)
  2. Put an axe and fuel in the chopper, along with a rubber tree sapling, in the correct locations, with dirt in front
  3. Bone meal the placed rubber sapling
  4. The resultant tree will not be mined, and the chopper will display 'waiting' as its status.
remclave commented 2 years ago

What tier axe is in the chopper? Rubber trees require a higher-tier axe to harvest.

Kratargon commented 2 years ago

What tier axe is in the chopper? Rubber trees require a higher-tier axe to harvest.

Obsidian, which is definitely capable of chopping rubber trees (also why I used a higher-tier chopper, so I could put the axe in). Though, you should probably know that choppers seem to go around harvesting requirements: I found by accident that a flint axe in a chopper is capable of collecting wood from higher-tier logs, notably, the Natura woods, that they are not usually able to harvest. The catch is that this process is painfully slow to the point of impracticality, especially considering how cheap it is to get a better chopper/axe. I have not extensively tested this, but it's likely a bug as well.

Loenrado commented 1 year ago

Was there ever a solution to this issue?

Corfiot commented 8 months ago

Yeah necro, I know.

Progressive automation fixed this after 1.10 so you need to make rubber logs plain logs (add logWood oredict). Add these lines to scripts/oreDict.zs and reload your world:

# Allow chopper from progressive automation to chop rubber trees
<ore:logWood>.add(<MineFactoryReloaded:rubberwood.log:1>);
Zilacon commented 8 months ago

Yeah necro, I know.

Progressive automation fixed this after 1.10 so you need to make rubber logs plain logs (add logWood oredict). Add these lines to scripts/oreDict.zs and reload your world:

# Allow chopper from progressive automation to chop rubber trees
<ore:logWood>.add(<MineFactoryReloaded:rubberwood.log:1>);

Is that all that's needed to fix this? As i don't see us backporting fixes from a MC 1.10 version of a mod.

Zilacon commented 8 months ago

I have added this to the scripts for AS2 on technic. Same version, just updated the zip.

Corfiot commented 8 months ago

Is that all that's needed to fix this? As i don't see us backporting fixes from a MC 1.10 version of a mod.

It worked for me on AS2 with no other changes. I don't know if I broke something by adding that oredict but I doubt it. If I see anything down the road as I play I'll update here.

Zilacon commented 8 months ago

Is that all that's needed to fix this? As i don't see us backporting fixes from a MC 1.10 version of a mod.

It worked for me on AS2 with no other changes. I don't know if I broke something by adding that oredict but I doubt it. If I see anything down the road as I play I'll update here.

Added the change to the repo as well, let me know if its fixed or not. Thanks. 468a67a3989b1637cd6a6a9adfbcca7465f8e266

Corfiot commented 7 months ago

I see nothing else wrong up to this point in my progress in AS2. I think it's fixed.