Guild icon
Gamemode 4 Labs
chairs
Avatar
BluePsychoRanger 14-Feb-19 03:32 PM
Dennis: Module: throw a saddle onto a stair to turn it into a chair (edited)
Pinned a message.
Avatar
Dennis 14-Feb-19 03:51 PM
It should be updated now on github, with the 2 advancements
So if there's any feedback let me know
Avatar
iGalaxy 15-Feb-19 01:29 AM
💺
gm4_thumbsup 2
Avatar
razorsharp 15-Feb-19 02:50 AM
here's an idea for an advancement name: What Are You Supposed to do With It?
get it because minecraft characters dont sit down
Avatar
Texture Bacon 15-Feb-19 02:52 AM
My vote is a mild regen buff when in the chair
Avatar
razorsharp 15-Feb-19 02:53 AM
oh god what is sitting i cant comprehend it oh god my square virtual flesh body is so rigid what does it mean to sit
Avatar
Claw 15-Feb-19 04:20 AM
I have no knees
Avatar
ShadowSlam 15-Feb-19 12:42 PM
But... but they sit in minecarts, boats, and on horses and pigs... (edited)
Avatar
razorsharp 18-Feb-19 09:55 PM
No they just kinda
Split in half slightly
Avatar
Dennis 18-Feb-19 10:11 PM
makes sense
Avatar
Sparks 28-Feb-19 10:57 AM
Like the module, left feedback 😃
I just noticed the couch potato advancement. That's great
Avatar
Dennis 28-Feb-19 11:00 AM
https://media.giphy.com/media/3diGTnVKA26rhT1MEb/giphy.gif this was before some changes, but it's basically this
it's now a bit slower to run on the gm4 clock
couch potato was BPR's suggestion, couldn't ignore that one
also, same issue as bat grenades where the pig is not invisible for 1 tick
Avatar
Sparks 28-Feb-19 11:06 AM
Ooh!
Sorry, I didn't check to see if there was a visible saddle before giving feedback as I don't have time to test rn
I really like
Avatar
Dennis 28-Feb-19 11:08 AM
no problem! don't think I even mentioned it anywhere
Avatar
Sparks 28-Feb-19 11:23 AM
I think this needs a whoopie-cushion expansion pack.
Avatar
Dennis 28-Feb-19 11:25 AM
Definitely!
just need some good sound effects then 🤔
Avatar
Sparks 28-Feb-19 12:01 PM
I just realised that a) ghast bags are the pefect whoopie-cushion item.and b) loot tables can be included with datapacks now so we can totally do ghast bags from 4D
Avatar
Lue 28-Feb-19 12:04 PM
yea but we don't have aether ghasts
do ghast bags really make sense to drop from nether ghasts
Avatar
Sparks 28-Feb-19 12:07 PM
in 4D no. In Minecraft, why not?
I do think their levitation effect would be a bit OP outside of 4D aether though
Avatar
Dennis 28-Feb-19 12:13 PM
only allow it in the end if that's possible? (edited)
Avatar
Kroppeb 28-Feb-19 12:28 PM
Mmh how do ghasts float though?
Avatar
Dennis 28-Feb-19 12:36 PM
They have ghast bags inside them 👀
Avatar
Kroppeb 28-Feb-19 12:40 PM
But how do they float
Avatar
kyrkis 28-Feb-19 01:05 PM
They are lighter than the thin air in Aether
Avatar
Compsogbrickus 28-Feb-19 01:16 PM
Since ghasts are giant hot air bags, the ghast bags would just contain air, but maybe also contain some sort of heating element
Avatar
Kroppeb 28-Feb-19 02:20 PM
But the nether is hot already
Avatar
Sparks 28-Feb-19 03:57 PM
For a hot air balloon to be bouyant it needs to be hotter than the surrounding air
So actually the air inside a Nether ghast would have to be far hotter than an aether ghast to make it fly (assuming both types of ghast weigh the same)
Avatar
Kroppeb 28-Feb-19 04:02 PM
Sulfur dioxide is 2.2* as heavy as normal air (edited)
Avatar
Bloo 28-Feb-19 05:03 PM
thats only true if the gas inside is the same as the gas in the surroundings, and if the buoyancy of the structure itself is tiny
Avatar
Giovata 28-Feb-19 11:41 PM
I really want to get the [Couch Potato] advancement (edited)
Avatar
BluePsychoRanger 04-Apr-19 04:25 PM
This module has been approved!
Pinned a message.
Avatar
BluePsychoRanger 04-Apr-19 05:16 PM
@Dennis In the final version, is the player still able to move the pig in any way or was that bug fixed?
Avatar
Dennis 04-Apr-19 05:18 PM
It's included as an option
More like a correction when the player is near the chair it resets its position/rotation
But I haven't found any way to stop them from moving when holding the thing
Avatar
BluePsychoRanger 04-Apr-19 05:19 PM
Okay, gonna add some info to the Technical Details
Avatar
Dennis 04-Apr-19 05:20 PM
And it's not that bad if players can move the saddle, only concern with it is that someone could go in my neatly organised room with chairs and rotate them all
Avatar
Dennis 04-Apr-19 05:49 PM
great, wiki just needs the icon now
Avatar
ToffeeMax 04-Apr-19 05:49 PM
gm4.co has icon, but wiki is being a little crap bag and not wanting to allow uploads atm
Avatar
Sparks 04-Apr-19 05:50 PM
Don't forget to credit duck for the icon art
Avatar
ToffeeMax 04-Apr-19 05:50 PM
On Wiki?
Avatar
Sparks 04-Apr-19 06:26 PM
yes
Avatar
ToffeeMax 05-Apr-19 04:50 PM
Cool
Avatar
Dennis 27-Apr-19 12:04 PM
I've noticed that if the pig is moved a pixel back, then it allows you to place blocks in front of it because its hitbox it not there anymore (edited)
Does this still look good enough?
Avatar
Misode 27-Apr-19 12:08 PM
I would make the pig invisible
But yeah that looks good
Avatar
Dennis 27-Apr-19 12:09 PM
Thought it would be a nice chair extension
but I'll remove their heads for you
Avatar
Qbert 27-Apr-19 02:16 PM
lol
Avatar
Dennis 04-May-19 10:41 AM
I'm not too sure myself which one is better.. C: Current B: Very small adjustment for the new texture A: Hitbox is inside stair, can place blocks in front of the stairs without breaking the chair first (I think saddle is too small with old textures)
I like a bigger saddle but not having that hitbox is nice as well
Avatar
Misode 04-May-19 10:43 AM
A is definitely an advantage, eg. you can place a carpet in front of the stair
Avatar
Andante 04-May-19 10:43 AM
I prefer A, at least with the new textures (edited)
Avatar
Misode 04-May-19 10:43 AM
I'd say keep the 1.13 version how it is and change it for 1.14
Avatar
Sparks 04-May-19 10:44 AM
^
Avatar
Dennis 04-May-19 10:44 AM
so change to A for 1.14
and next question, to update on github, do i edit it in my own fork, then pull request?
Avatar
Misode 04-May-19 10:46 AM
Yep
And to make sure you have the latest changes, you create a branch from master
Avatar
Dennis 04-May-19 11:18 AM
and how do i combine multiple commits into 1 pull request?
Avatar
Andante 04-May-19 11:20 AM
I don't think you can, but the admins can squash and merge the pull into one commit (edited)
Avatar
Misode 04-May-19 11:20 AM
A PR is based on a branch, so it can always have multiple commits
You just commit to the same branch
Avatar
Dennis 04-May-19 11:23 AM
just added 2 pull requests needs to be merged then
Avatar
Misode 04-May-19 11:23 AM
Uh why two pull requests?
They are the same as far as I can tell
Avatar
Dennis 04-May-19 11:24 AM
2 different files, but the same changes basically
Avatar
Misode 04-May-19 11:24 AM
Oh
You could've made those on the same branch
You can select the current branch you're working on here
Avatar
Dennis 04-May-19 11:26 AM
one day I will understand this!
Avatar
Sparks 04-May-19 11:26 AM
Github is SO useful but there is a learning curve :/ I'm still a bit lost when using it tbh
Avatar
Misode 04-May-19 11:27 AM
Yeah it takes a while to get how it works
Avatar
Dennis 04-May-19 02:09 PM
Another small change is coming, the saddle item now just spawns normally but thought it would be nicer to give it a random motion
Avatar
Sparks 04-May-19 02:09 PM
motion?
Avatar
Dennis 04-May-19 02:11 PM
like when you break a block and the item entity goes in any direction
Avatar
Sparks 04-May-19 02:13 PM
Ohhh gotcha
Avatar
Dennis 04-May-19 02:44 PM
Made a new pull request will all changes it
Avatar
Lue 04-May-19 03:03 PM
@Dennis in 1.14 you can use a loot table to easily get random motion
just a loot table with a single saddle as the drop, loot spawn ~ ~ ~ loot chairs:drop_saddle (edited)
Avatar
Dennis 04-May-19 03:04 PM
oh never heard of that command before
Avatar
Misode 04-May-19 03:05 PM
It's a very useful command
Avatar
Dennis 04-May-19 03:09 PM
that's a lot better indeed
just need to figure out how to add the loot table
Avatar
Lue 04-May-19 03:11 PM
datapacks/gm4_chairs/data/chairs/loot_tables/drop_saddle.json
{ "type": "minecraft:generic", "pools": [ { "rolls": 1, "entries": [ { "type": "minecraft:item", "name": "minecraft:saddle" } ] } ] }
Avatar
Dennis 04-May-19 03:12 PM
thanks, that works much much better!
Avatar
Dennis 04-May-19 03:19 PM
added it to the pull request as well, I think I'm starting to understand this thing
Avatar
Sparks 05-May-19 09:08 AM
I think there's been talk of saddle shamirs so at some point this module should be updated to not accept saddles with shamirs on them or they will be lost
Avatar
Dennis 05-May-19 09:30 AM
Would it be possible to copy over that bit of nbt onto the pig so it would still work and return the shamir saddle?
Avatar
Andante 05-May-19 09:54 AM
I'm just on my dev server, and whenever I get out of a chair I seem to fall through the block?
Also, you can kill the pig and get infinite porkchops (edited)
Avatar
Dennis 05-May-19 10:02 AM
blame 1.14
in 1.13 it would just put the player on top of the block below the stair but somehow in 1.14 it changed so the player is slightly lower and sinks through the ground
I tried to tp the player up if it was inside the stair block but the clock isn't fast enough for that
Avatar
Sparks 05-May-19 11:10 AM
We should give the pig the minecraft:empty loot table?
Avatar
Misode 05-May-19 11:11 AM
Oh yeah that's a good idea
Avatar
Dennis 05-May-19 11:25 AM
I wish I would've known about these loottables before I started this :p
Avatar
Dennis 05-May-19 11:47 AM
how did you get porkchops btw @Andante? did you kill it in creative mode?
Avatar
Andante 05-May-19 11:47 AM
Oh yeah, I was in creative
didn't realise that
Avatar
Dennis 05-May-19 11:47 AM
creative mode can kill invulnerable entities
but removing their drops won't do any harm
Avatar
Dennis 05-May-19 12:06 PM
removed droptable in the PR, also removed a command where the pigs where added to teams as that's now done in the summon command
Avatar
kyrkis 10-May-19 11:10 PM
Our chairs turned into no AI pigmen from lightning @Dennis
Avatar
Dennis 10-May-19 11:10 PM
nice
can you ride them?
Avatar
kyrkis 10-May-19 11:10 PM
no lol
please fix xD
I mean we can kill them, I think
but like i dont want this happening
Avatar
Dennis 10-May-19 11:11 PM
I think that would need a fix on the side of lightning rods, not sure if I can prevent that
Avatar
kyrkis 10-May-19 11:11 PM
it was natural lightning lol
Avatar
Dennis 10-May-19 11:11 PM
are they still NoAI btw?
oh
Avatar
kyrkis 10-May-19 11:11 PM
yep still no AI
a lightning struck me and georgia sitting in the bench
it targets entities and there were 4
Avatar
Dennis 10-May-19 11:13 PM
One option I can think of is check for pigmen with the chair tag (assuming they still have that) and turn them back
but that's interesting, hadn't thought of that option yet
Avatar
Misode 10-May-19 11:15 PM
I think it's similar to drowneds, where almost no nbt is transferred
Avatar
Dennis 10-May-19 11:21 PM
yep, looks like NoAI is the only nbt that is kept
Avatar
Dennis 10-May-19 11:37 PM
Don't really know the best way this could be fixed tbh
althought I found the InLove tag which could be used to prevent the players from feeding carrots to their chairs
Avatar
Sparks 10-May-19 11:45 PM
Haha oh dear!
I didn't see that one coming...
Avatar
Chopper2112 10-May-19 11:56 PM
Breeding chairs think
Avatar
Qbert 11-May-19 12:59 AM
lol
Avatar
EnderSlime_123 11-May-19 08:50 PM
XD
Exported 145 message(s)