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

Rats is half compatible with Fruit Trees #411

Open
ReiDaTecnologia opened this issue Sep 5, 2024 · 9 comments
Open

Rats is half compatible with Fruit Trees #411

ReiDaTecnologia opened this issue Sep 5, 2024 · 9 comments
Labels
enhancement New feature or request

Comments

@ReiDaTecnologia
Copy link

ReiDaTecnologia commented Sep 5, 2024

Description

Rats can bone meal them but can't collect them. Affect vanilla cocoa beans, all pam trees and Forestry's date and payaya (The other fruit trees from Forestry also don't work but as the fruits are in the leaves and they still drop fruits when chopped down, they aren't that bad).
To Reproduce

Steps to reproduce the behavior:

  1. Get any of the tree mentioned above
  2. Get a tamed rat to the tree (if you plant the tree's sapling near a tamed rat, it will instead break the sapling and if you grow the tree before he can do that, it will just stay/walk toward the previous location of the sapling until something else overwrite that previous command)
  3. See the Rat not collecting the tree's fruit even if it is fully growth

Modpack version v1.43.0

debug.log or Crash report

This is the debug.log of a session of me testing all Pam trees, Forestry and cocoa beans. I had to do some pauses for several reasons so it explains the time skip of it.
debug.zip

@Krutoy242
Copy link
Owner

Thank you for the explanation. I will take a look what I can do.

@ReiDaTecnologia
Copy link
Author

ReiDaTecnologia commented Sep 6, 2024

After checking Rats: RatN Edition and Pam's source code (with my basic knowledge of programming), i think it is because Pam fruit blocks don't have "BlockCrops", "BlockBush" and blocks with the set Material "GOURD" which Rats is looking for.

@Krutoy242 Krutoy242 changed the title [Bug] Rats is half compatible with Fruit Trees Rats is half compatible with Fruit Trees Nov 13, 2024
@ReiDaTecnologia
Copy link
Author

After months of work, i was able to improve the harvesting feature of rats mod. As i learned recently that you can't just upload the modified jar to curseforge, people who want to have it, can just download from the fork's release page. Any bug report and suggestions of improving the code or the harvesting feature is welcomed.

@Krutoy242
Copy link
Owner

Can you propose this changes to Rats:ROTN fork? Maybe they can just merge your PR? Your improvements looks very fitting.

@ReiDaTecnologia
Copy link
Author

Sure but i prefer to test it in a real player environment (like in my world) first.

@Krutoy242
Copy link
Owner

Krutoy242 commented Nov 23, 2024

Maybe, instead if making a separate mod, we could just make ZenScript mixin? This will make it easy to maintain and improve, since doesnt require things like dev environment, compilation and stuff like that. Also its easy to distribute.

@ReiDaTecnologia
Copy link
Author

Well if you are willing to do that, i wish you good luck!

@Krutoy242
Copy link
Owner

Yes, I will try.
Only thing that buzzed me is by the first look on your code its kinda junky. Too much code repetitions, overload of static strings, unperformant calls, and so on.

@ReiDaTecnologia
Copy link
Author

ReiDaTecnologia commented Nov 23, 2024

Yeah, i'm new to modding and not a god-like programmer. At least it works i guess... It was more of a personal work so i can use the rats to farm in my planned base's food farm area.

@Krutoy242 Krutoy242 added the enhancement New feature or request label Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants