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

Console spam when looking at a barrel with The One Probe #111

Open
SimplyLinn opened this issue Mar 5, 2017 · 4 comments
Open

Console spam when looking at a barrel with The One Probe #111

SimplyLinn opened this issue Mar 5, 2017 · 4 comments

Comments

@SimplyLinn
Copy link

in #7b3cc90 you accidentally commited a debug-console output to master. It spams my console with "test" whenever I look at a barrel.

src/main/java/exnihiloadscensio/blocks/BlockBarrel.java:161

Method addProbeInfo

Also, for some reason the TOP integration seems to not work that well with Ex Nihilo at all, as almost all blocks display an error when I look at them, but the main issue is the console spam.

Screenshot of the error displayed in game. (It's currently of a sieve, but it doesn't matter, it's the same for every block from Ex Nihilo Adscensio)
image

Line in console output: [exnihiloadscensio.blocks.BlockBarrel:addProbeInfo:161]: test

@SimplyLinn
Copy link
Author

Apparently there was a PR for this... #110

@MikeLydeamore
Copy link
Owner

So the console spam is fixed, but I'm curious about the error.
I've been testing this out and have yet to get one block to error. Can you confirm your TOP version?

@SimplyLinn
Copy link
Author

SimplyLinn commented Mar 6, 2017

The One Probe 1.1x-1.4.4
If you want, I can give you the exported zip from Cursed Client which will recreate the exact environment I have on your end.

UrmitFactory-1.10.2-1.6.zip

There's roughly 200 mods in that thing and I'm surprised it even starts up :P

It's basically a combination of HermitPack and SkyFactory 3, which I just mashed together in the cursed client and hoped stuff would work.

@MikeLydeamore
Copy link
Owner

Just FYI, this is waiting upon McJty/compatlayer#9

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

No branches or pull requests

2 participants