[Mod] Crafting [2.0.1] [crafting]

User avatar
Traxie21
Member
 
Posts: 753
Joined: Mon Dec 31, 2012 10:48

by Traxie21 » Wed Apr 17, 2013 18:08

Many mods seem half-complete, like mine. BlockMen's seem complete.
 

User avatar
stu
Member
 
Posts: 737
Joined: Sat Feb 02, 2013 02:51
GitHub: stujones11

by stu » Wed Apr 17, 2013 19:55

BlockMen wrote:Please give Feedback and ideas for improvement


Can you tell me how this mod differs from the existing workbench mod(s) other that the MC background?

Only I looked at the code and I really do not understand why you need to use global step for this. Perhaps I am missing something here but I suggest you look at workbench mod by darkrose.

https://github.com/darkrose/minetest_mods/tree/master/workbench
 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Wed Apr 17, 2013 20:11

stu wrote:
BlockMen wrote:Please give Feedback and ideas for improvement


Can you tell me how this mod differs from the existing workbench mod(s) other that the MC background?

Only I looked at the code and I really do not understand why you need to use global step for this. Perhaps I am missing something here but I suggest you look at workbench mod by darkrose.

https://github.com/darkrose/minetest_mods/tree/master/workbench



BlockMen wrote:Notice:
- I know there are already 2 mods with workbenches, but this one works in a different way, because it uses the default inventory and works faster (for me)


This mod uses the players inventory. so it works complete different to the existing mods. the global step is needed to make using of players inventory possible. my mod is (for me) much faster and the code is also much smaller.
 

User avatar
stu
Member
 
Posts: 737
Joined: Sat Feb 02, 2013 02:51
GitHub: stujones11

by stu » Wed Apr 17, 2013 20:42

BlockMen wrote:This mod uses the players inventory. so it works complete different to the existing mods. the global step is needed to make using of players inventory possible. my mod is (for me) much faster and the code is also much smaller.

Oh, I see, so it replenishes the crafting ingredients directly from the player's inventory? (I don't actually play minecraft)

That is fine if you are happy with it, you did ask for feedback. I still don't think you're making best use of the api though.
 

User avatar
jordan4ibanez
Member
 
Posts: 1865
Joined: Tue Sep 27, 2011 18:44
GitHub: jordan4ibanez
IRC: jordan4ibanez
In-game: jordan4ibanez

by jordan4ibanez » Wed Apr 17, 2013 23:01

stu wrote:
BlockMen wrote:This mod uses the players inventory. so it works complete different to the existing mods. the global step is needed to make using of players inventory possible. my mod is (for me) much faster and the code is also much smaller.

Oh, I see, so it replenishes the crafting ingredients directly from the player's inventory? (I don't actually play minecraft)

That is fine if you are happy with it, you did ask for feedback. I still don't think you're making best use of the api though.

That's the thing. HE IS. This mod changes the client side inventory on the fly so you don't get any server lag.
If you can think it, you can make it.
 

User avatar
stu
Member
 
Posts: 737
Joined: Sat Feb 02, 2013 02:51
GitHub: stujones11

by stu » Thu Apr 18, 2013 18:29

jordan4ibanez wrote:
stu wrote:
BlockMen wrote:This mod uses the players inventory. so it works complete different to the existing mods. the global step is needed to make using of players inventory possible. my mod is (for me) much faster and the code is also much smaller.

Oh, I see, so it replenishes the crafting ingredients directly from the player's inventory? (I don't actually play minecraft)

That is fine if you are happy with it, you did ask for feedback. I still don't think you're making best use of the api though.

That's the thing. HE IS. This mod changes the client side inventory on the fly so you don't get any server lag.

Ok now I get it, very clever then. Shame we have to resort to such tactics, but if it gets the job done...
 

User avatar
jordan4ibanez
Member
 
Posts: 1865
Joined: Tue Sep 27, 2011 18:44
GitHub: jordan4ibanez
IRC: jordan4ibanez
In-game: jordan4ibanez

by jordan4ibanez » Thu Apr 18, 2013 23:12

stu wrote:
jordan4ibanez wrote:
stu wrote:Oh, I see, so it replenishes the crafting ingredients directly from the player's inventory? (I don't actually play minecraft)

That is fine if you are happy with it, you did ask for feedback. I still don't think you're making best use of the api though.

That's the thing. HE IS. This mod changes the client side inventory on the fly so you don't get any server lag.

Ok now I get it, very clever then. Shame we have to resort to such tactics, but if it gets the job done...

That isn't resorting to tactics, since any other way the gui would be handled serverside it would be laggy, this is a good idea.

ALSO: BlockMen can you make the workbench get the player's position also? you can move around without changing your yaw.
If you can think it, you can make it.
 

User avatar
darket
Member
 
Posts: 34
Joined: Sun Nov 27, 2011 10:35

by darket » Fri Apr 19, 2013 07:18

 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Fri Apr 19, 2013 13:18



Thanks for this great review!

Could you move the download link in the description to this topic? There will be a few updates more i guess.



jordan4ibanez wrote:BlockMen can you make the workbench get the player's position also? you can move around without changing your yaw.


Good idea, thanks.
 

TheBonsai
New member
 
Posts: 6
Joined: Sun Feb 17, 2013 16:01

by TheBonsai » Fri Apr 19, 2013 18:40

jordan4ibanez wrote:ALSO: BlockMen can you make the workbench get the player's position also? you can move around without changing your yaw.


The yaw-method to detect when to swap back is a thing I didn't mention yesterday, because I didn't see any practical solution. Is this the only way (i.e." workaround" :-) )
 

User avatar
darket
Member
 
Posts: 34
Joined: Sun Nov 27, 2011 10:35

by darket » Sun Apr 21, 2013 03:40

BlockMen wrote:Thanks for this great review!

Could you move the download link in the description to this topic? There will be a few updates more i guess.


Ok, I'll do that
 

User avatar
Mito551
Member
 
Posts: 1271
Joined: Sat Jun 16, 2012 15:03

by Mito551 » Sun Apr 21, 2013 12:34

will you make support for creative? that's the only thing that this mod is bad with, at least as I see it.
 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Sun Apr 21, 2013 13:10

darket wrote:
BlockMen wrote:Thanks for this great review!

Could you move the download link in the description to this topic? There will be a few updates more i guess.


Ok, I'll do that


Thx

Mito551 wrote:will you make support for creative? that's the only thing that this mod is bad with, at least as I see it.


Yes, it will get a Minecraft like creative inventory too. But its not finished yet...
Last edited by BlockMen on Sun Apr 21, 2013 13:11, edited 1 time in total.
 

4aiman
Member
 
Posts: 1208
Joined: Mon Jul 30, 2012 05:47

by 4aiman » Mon Apr 22, 2013 16:16

BlockMen, please, check your code: http://forum.minetest.net/viewtopic.php?pid=85858#p85858
(Thanks to Mito for redirecting me here - I couldn't find this topic for some reason...)
 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Mon Apr 22, 2013 16:38

4aiman wrote:BlockMen, please, check your code: http://forum.minetest.net/viewtopic.php?pid=85858#p85858
(Thanks to Mito for redirecting me here - I couldn't find this topic for some reason...)


Well, AFIK Crafting doesnt have this issue, because there the width is always set. So please report that to jojoa1997 he has modiefied my mod when he joined it in his inventory mod. Furthermore im not involved in MiniTest...
 

4aiman
Member
 
Posts: 1208
Joined: Mon Jul 30, 2012 05:47

by 4aiman » Mon Apr 22, 2013 16:53

I know you aren't involved into Minitest, but I remembered this mod of yours and eventually end up posting here.
Anyway, thanks for help! ;)
 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Mon Apr 22, 2013 17:02

4aiman wrote:I know you aren't involved into Minitest, but I remembered this mod of yours and eventually end up posting here.
Anyway, thanks for help! ;)


Sry that i cant help you here, but i wont start to analyse jojoa1997's code ;)
Last edited by BlockMen on Mon Apr 22, 2013 17:08, edited 1 time in total.
 

User avatar
jojoa1997
Member
 
Posts: 2890
Joined: Thu Dec 13, 2012 05:11

by jojoa1997 » Mon Apr 22, 2013 17:36

BlockMen wrote:
4aiman wrote:I know you aren't involved into Minitest, but I remembered this mod of yours and eventually end up posting here.
Anyway, thanks for help! ;)


Sry that i cant help you here, but i wont start to analyse jojoa1997's code ;)
I made it. The inventory looks similar but isn't the same. Look at changelog.txt to see who did what but remember I edit a lot of the stuff.
Coding;
1X coding
3X debugging
12X tweaking to be just right
 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Tue May 21, 2013 11:38

Update: [Modpack] Craft added
(and multiplayer issues fixed)

I have decided to Pre-Release a new feature of Crafting, the creative Inventory:
Image

To install download this modpackand make sure that the current version of crafting is deinstalled. The lastest Minetestversion will produce a warning because of two mods with same name (but has no effect after all). Hopfully that will changed in future Minetest versions.

Download Pre-Release here

A Feedback to the creative inventory would be nice, because there could still be bugs.
 

User avatar
cHyper
Member
 
Posts: 587
Joined: Fri May 06, 2011 08:49
IRC: cHyper
In-game: cHyper

by cHyper » Tue May 21, 2013 14:40

awesome mod.. keep on good work...
--------------------------------------------------------
 

User avatar
jojoa1997
Member
 
Posts: 2890
Joined: Thu Dec 13, 2012 05:11

by jojoa1997 » Tue May 21, 2013 14:49

Sweet I can't wait to try this out.
Coding;
1X coding
3X debugging
12X tweaking to be just right
 

User avatar
rubenwardy
Member
 
Posts: 4500
Joined: Tue Jun 12, 2012 18:11
GitHub: rubenwardy
IRC: rubenwardy
In-game: rubenwardy

by rubenwardy » Tue May 21, 2013 15:01

Wow this is amazing!

Is the creative inventory skin-able with texture packs?
Last edited by rubenwardy on Tue May 21, 2013 15:01, edited 1 time in total.
 

User avatar
jojoa1997
Member
 
Posts: 2890
Joined: Thu Dec 13, 2012 05:11

by jojoa1997 » Tue May 21, 2013 15:27

Could someone post a video?
Coding;
1X coding
3X debugging
12X tweaking to be just right
 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Tue May 21, 2013 15:33

cHyper wrote:awesome mod.. keep on good work...

jojoa1997 wrote:Sweet I can't wait to try this out.

rubenwardy wrote:Wow this is amazing!

Thanks.

rubenwardy wrote:Is the creative inventory skin-able with texture packs?


Sure. But the ugly semi-transparent black in background is always there...
 

User avatar
jojoa1997
Member
 
Posts: 2890
Joined: Thu Dec 13, 2012 05:11

by jojoa1997 » Tue May 21, 2013 15:37

Simple. Look at the inventory mod. You set the background pos to -#, -# and set the pic size acordingly
Coding;
1X coding
3X debugging
12X tweaking to be just right
 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Tue May 21, 2013 15:41

jojoa1997 wrote:Simple. Look at the inventory mod. You set the background pos to -#, -# and set the pic size acordingly


I know how that works and this Mod uses it. But for the creative inventory it is caused by the buttons on the bottom and the gaps between the buttons
 

Nore
Member
 
Posts: 468
Joined: Wed Nov 28, 2012 11:35
GitHub: Ekdohibs

by Nore » Sat May 25, 2013 19:07

Could you check the pitch too? From the tests I made, you can't open your inventory or move or do anything if you don't move the mouse a bit. Checking the yaw and the pitch removes every problem (tested, it works).
 

User avatar
BlockMen
Member
 
Posts: 768
Joined: Fri Mar 01, 2013 17:24
GitHub: BlockMen

by BlockMen » Sun May 26, 2013 12:45

Nore wrote:Could you check the pitch too? From the tests I made, you can't open your inventory or move or do anything if you don't move the mouse a bit. Checking the yaw and the pitch removes every problem (tested, it works).


Yes, forgot to fix that in the pre-release. It will be fixed in next release.
 

User avatar
Evergreen
Member
 
Posts: 2131
Joined: Sun Jan 06, 2013 01:22
GitHub: 4Evergreen4
IRC: EvergreenTree
In-game: Evergreen

by Evergreen » Tue May 28, 2013 21:40

In the next release, could you add the inventory GUI for furnaces? It would make it more complete.
"Help! I searched for a mod but I couldn't find it!"
http://krock-works.16mb.com/MTstuff/modSearch.php
 

User avatar
rubenwardy
Member
 
Posts: 4500
Joined: Tue Jun 12, 2012 18:11
GitHub: rubenwardy
IRC: rubenwardy
In-game: rubenwardy

by rubenwardy » Wed May 29, 2013 14:54

If only you could skin the pause menu, and the main game menu....
 

PreviousNext

Return to Mod Releases

Who is online

Users browsing this forum: No registered users and 6 guests

cron