Minecraft Server Thread

[Whiteness]

yay, it wasn't my bad chunk!

[Whiteness]

The server is back up. There was a corrupted chunk and one next to it that had almost 400 entities in it, which is way over the top. For reference, please don't try to put more than about a hundred to two hundred entities per chunk if you can help it.

The bad chunks were near the big mansion that sticks out into the swamp- not sure whose that is. If there's a cave or something under it that might be where the entities were overspawning; I'm still not totally sure. However, I'll keep an eye on it.

it looks like a piece of my ranch dissappeared with two chests of stuff, one wool, one assorted items, at x: -55,. z:940

[Whiteness]

actually, it was a square from X: -63 Z: 928 to X: -49 Z: 943 that disappeared.

[Jimer Lins]

The plugin basics are done and pushed to github. I'm going to put a test version on the server later today, but I have to run some errands first.

[Bulwark]

The plugin basics are done and pushed to github. I'm going to put a test version on the server later today, but I have to run some errands first.

Code review comments

BlockEventListener line 30. Performance: Can we move 'Block placedBelow=placedBlock.getRelative(BlockFace.DOWN);' to after the if statement? we are calling this every time a block is placed, which is expensive. Putting it after the first IF will reduce an unneeded call.

BlockEventListener line 72: Possible Bug: The inventoryKey is teh sum of the item IDs in the chest? Are item IDs unique to the item type, or to the item itself? For example, if I have 3 blocks of dirt, do they have the same itemID? If so, we have a potential problem. For example: if I have 2 items in a chest with ItemIDs of 5 and 7. You have a chest with 2 items with itemIDs of 2 and 10. Both of these chests will have the same inventoryKey but have different items in them. Possible Solution: inventoryKey+=(int)stack.getType().getId() * (stack.getInventoryPosition() * 512). This will keep you from having the 5+7 = 2+10 problem. Potential downsides are future proofing against more than 512 unique item types and reaching MAXINT.

I'm wondering if there is something we could use other than onBlockPlace and onBlockDestroy. Those are probably the 2 most common events in the whole game.

[Maizan]

That bad chunk area was a very bad spawn spot. It was death at night there. Although it did produce a slime in a mine cart and spider in a mine cart.

Sent from my Windows8 phone using Tapatalk

[Jimer Lins]

Plugin Update

I've updated the plugin with the following changes:

* A few minor code fixes, based on Bulwark's code review (thank you).
* Moved the task of the actual teleport to a BukkitRunnable to be invoked on the main Minecraft thread instead of blocking the event handler.
* After much hair-pulling and digging in the code for Essentials, I added this: loc.getBlock(); to the teleport code. We don't need it but it seems to have the side effect of getting the chunk loaded or in some other way preventing the "falling through" problem that some were having.
* Adjusted transporter sound effects to play on both exit and arrival.

A Warning

Bulwark pointed out a potential flaw in the key generation using ids. I didn't like using them because they're deprecated anyway. There's a flaw in that code that could make transporters link to the wrong places.

Here's the problem: If I fix it, existing transporters will continue to work, BUT- if you want to add to any existing ones or if you change one, it will not re-link and you'll have to create both ends. People have made a ton of transporters already, so I wanted you to be aware of this.

This is what you get for being the alpha testers.

[Bulwark]

Here is a thought...
instead of using BlockPlaceEvent, can we use BlockBurnEvent ? Have it behave like the nether protal: after you place the quartz, it is not active until you click on it with a firestarter? this would reduce lag...as long as you can programatically put the fire out.

[Maizan]

Here's an thought. (although it is a little late for this)

Instead of using a chest with stuff in it. Would it be more effective to use a sign on the wall with something like [Porter] on the first line and ID key on the 2nd line and display name on line 3 and 4. ID Key would be the replacement of the items in the chest..

As the chest process is mostly working, maybe keep it in place and the required build would still the same a chest with a quartz block on top. If the chest is empty, the look for the sign.

BTW, as anyone tested for an empty chest?

Food for thought.

Maizan

[Jimer Lins]

I liked the idea of a chest because it's simple and doesn't use commands, and could be completely hidden if you did it right. I considered signs, but that seemed more complicated.

[Bulwark]

Another thought: Use playerToggleSneakEvent to trigger the portal instead of playerMoveEvent. Much less overhead.

As long as we are brainstorming, it would be fun to add vehicleMoveEvent so that minecarts and horses could teleport.

[Jimer Lins]

I've got a version that supports minecarts. I can put it on the server anytime but it requires a restart.

[Bulwark]

Bug Report: Placing one end of a portal in the Nether but not the other causes you to teleport to the correct X,Y,Z, but keeps you in the nether (usually 50 meters above a pool of lava). My guess is that walking in the nether over the X,Y,Z location of a portal in the overworld would also cause a teleport to happen.

Proposed Resolution: use the GetWorld() command when saving/using a portal. Probably requires a schema change. At this point, I'd just default all the existing portal locations to use the overworld as I don;t imagine we have any Nether or End portals up yet.

[Whiteness]

I have a teleport in the nether. Nether to nether works fine, but yeah nether to overworld doesn't.

[Cuspar]

When you guys have this figured, can you please post an "Alvis Teleporters for the Idiot" guide?

Current status: CONFUSED

[Jimer Lins]

I'm going to be unavailable for a few days this coming week. I doubt anything will need to happen but just in case it does, does anyone have experience with CentOS or Linux in general, any understanding of Minecraft servers and is willing to be a deputy server admin for a few days? If so, let me know.

[Maizan]

I am fairly comfortable with Linux and several of there flavors. Setup and managed about 12 of them over the past few years. No minecraft experience though.


Also Here is the 2 levers controlling 1 door setup. I used it for my folding staircase.





Happy building.

Maizan

[Jimer Lins]

OK, I've updated to the latest plugin version. Still no multi-world support (sorry), but you can now ride minecarts through teleporters. Let me know if you run into trouble with it.

[Bulwark]

After seeing this video, I don't think I'm going to play minecraft anymore because, really, whats the point...

[Jimer Lins]

That's insane. I love it.

[Maizan]

Wow.

Definitely Insane!

I think Bul will redo Fun Land after watching this.

[Indigo]

Bulwark, have you seen the giant steampunk tortoise with a city on its back?



I downloaded the save for it and it was pretty neat to explore.

Link if anyone wants it: http://www.planetminecraft.com/project/atropos/

[Jimer Lins]

I've pushed the latest version to Github, in the DEVELOPMENT branch of the repository. Feel free to clone it and tweak it if you want. You'll need Eclipse and a copy of Bukkit to test it locally. I use a .jardesc that simply rebuilds the plugin in the plugins folder of my local test server, then use /reload in its console. Works pretty well.

[Blackferne]

This weekend Zoe wa svery into minecraft song videos.

Two of her favorites I thought worthy of sharing:



and

[Bulwark]

in TransporterTriggerListener.java line 69, can we change it from:
if(event.getVehicle().getType() != EntityType.MINECART)
to
if ((event.getVehicle().getType() != EntityType.MINECART) && (event.getVehicle().getType() != EntityType.HORSE))

because teleporting ponies are fun!