[mod 0.12.30] {BB} - Belt Brush

Topics and discussion about specific mods
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

[mod 0.12.30] {BB} - Belt Brush

Post by matjojo »

Mod-Info
  • Name: Belt Brush
  • Version: 0.0.5
  • Factorio-Version: 0.12.30 (which Factorio version this was tested with)
  • Description: A brush to place belts with like concrete.
  • License: MIT
  • Release: 2016-03-19
  • Website: BeltBrush on the forum (this page)
  • Category: Helper mods
  • Tags: Utilities, Transport belts, Gameplay, Multiplayer compatible.
License
Long description
To Do
Change log
Older Versions
Attachments
BeltBrush_0.0.5.zip
BeltBrush_0.0.5 Tested with factorio 0.12.30
(2.59 KiB) Downloaded 386 times
BeltBrush_0.0.4.zip
BeltBrush_0.0.4 Tested with factorio 0.12.29 DOES NOT WORK
(2.33 KiB) Downloaded 382 times
Last edited by matjojo on Sat Apr 16, 2016 8:52 am, edited 12 times in total.
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

Reserved for future use.
Murlocking
Long Handed Inserter
Long Handed Inserter
Posts: 66
Joined: Thu Mar 10, 2016 11:41 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by Murlocking »

Could you possibly add a feature like this mod does viewtopic.php?f=92&t=14093&start=10 to your mod please?

The other guy doesn't upload very often, and having to use 1 mod instead of 2 would be great + it would be up-to-date.

Thank you!
Murlocking
Long Handed Inserter
Long Handed Inserter
Posts: 66
Joined: Thu Mar 10, 2016 11:41 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by Murlocking »

In multiplayer, game crashed with an error for my friend (He's not the Host).

He was placing belts with the "first" mode of your mod (regular) and he got kicked out of the game after an error message appeared.
He didn't screenshot it unfortunately....

Had to remove the mod from my server for now.
Hope you can find the issue!
Sorry about no error message, my friend is not really technical with PC...
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

Murlocking wrote:In multiplayer, game crashed with an error for my friend (He's not the Host).

He was placing belts with the "first" mode of your mod (regular) and he got kicked out of the game after an error message appeared.
He didn't screenshot it unfortunately....

Had to remove the mod from my server for now.
Hope you can find the issue!
Sorry about no error message, my friend is not really technical with PC...
I'm sorry! as for the message: maybe the log file can help, could you please ask him to send me (or you) the log-old and log-current files, I might learn what happened from that.
Also the server logs, if it crashed too.
Last edited by matjojo on Sun Mar 20, 2016 2:45 pm, edited 2 times in total.
sillyfly
Smart Inserter
Smart Inserter
Posts: 1101
Joined: Sun May 04, 2014 11:29 am
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by sillyfly »

I think your `on_gui_click` function should get a player index variable. This way you could also apply the logic to the player who clicked it, not always the first player.

http://lua-api.factorio.com/0.12.27/eve ... _gui_click
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

Murlocking wrote:Could you possibly add a feature like this mod does viewtopic.php?f=92&t=14093&start=10 to your mod please?

The other guy doesn't upload very often, and having to use 1 mod instead of 2 would be great + it would be up-to-date.

Thank you!
I don't know if I can do that, I sent the creator of the other mod to ask him for permission, as he might not like it. He was active on the forum yesterday, so he probably will respond quickly.
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

sillyfly wrote:I think your `on_gui_click` function should get a player index variable. This way you could also apply the logic to the player who clicked it, not always the first player.

http://lua-api.factorio.com/0.12.27/eve ... _gui_click
that should be it, I was thinking about what might cause this. Thanks for the heads up! I'll work a bit on it today, and might have an update ready tomorrow.
Murlocking
Long Handed Inserter
Long Handed Inserter
Posts: 66
Joined: Thu Mar 10, 2016 11:41 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by Murlocking »

matjojo wrote:
Murlocking wrote:In multiplayer, game crashed with an error for my friend (He's not the Host).

He was placing belts with the "first" mode of your mod (regular) and he got kicked out of the game after an error message appeared.
He didn't screenshot it unfortunately....

Had to remove the mod from my server for now.
Hope you can find the issue!
Sorry about no error message, my friend is not really technical with PC...
I'm sorry! as for the message: maybe the log file can help, could you please ask him to send me (or you) the log-old and log-current files, I might learn what happened from that.
Also the server logs, if it crashed too.
Sure. (My friend logs)

Old
https://drive.google.com/file/d/0B4JjnW ... sp=sharing

Current
https://drive.google.com/file/d/0B4JjnW ... sp=sharing

Also replied on the other mod with Desync archive folder.

Hope this can help!

The error message did state 'Belt Brush' but he doesn't recall anything other than that.
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

Murlocking wrote:Could you possibly add a feature like this mod does viewtopic.php?f=92&t=14093&start=10 to your mod please?

The other guy doesn't upload very often, and having to use 1 mod instead of 2 would be great + it would be up-to-date.

Thank you!

Twice good news, I just finished the creating and some testing of the new and improved (now multiplayer compatible) version of BB, also got green light from the mod author of that mod to integrate it's capabilities into BB implementation will wait till the next version.
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

Just uploaded version 0.0.2, it should now work in multiplayer.

Version 0.0.2 (2016-03-24): Fixed multiplayer problem.
Download in original post.
Murlocking
Long Handed Inserter
Long Handed Inserter
Posts: 66
Joined: Thu Mar 10, 2016 11:41 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by Murlocking »

matjojo wrote:Just uploaded version 0.0.2, it should now work in multiplayer.

Version 0.0.2 (2016-03-24): Fixed multiplayer problem.
Download in original post.
Sweet :) Will give it a try tonight!
Also great new about the other thing!

Can't wait for the merge!
Jierdan
Manual Inserter
Manual Inserter
Posts: 3
Joined: Mon Aug 24, 2015 5:44 am
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by Jierdan »

Got this error running version 0.0.2
Attachments
BeltBrush.png
BeltBrush.png (48.72 KiB) Viewed 8717 times
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

Jierdan wrote:Got this error running version 0.0.2

Hmmmm, that's weird, what share you doing when you got the crash?
Jierdan
Manual Inserter
Manual Inserter
Posts: 3
Joined: Mon Aug 24, 2015 5:44 am
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by Jierdan »

I think it was related to one of the other mods I have installed, but I just tried to place a belt when it gave that error.

Added the mod list I was using during the crash(Factorio 12.29).
Attachments
Modlist.txt
(1.17 KiB) Downloaded 141 times
Geisel
Manual Inserter
Manual Inserter
Posts: 4
Joined: Thu Mar 24, 2016 5:41 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by Geisel »

hey matjojo,

first of all i like your addon so much! I allways did mainlines the "Armuba" way (or Steejo who know's ;) and you make my life easier :)

But i looked at your code, and i think you can improve it and make it more clean, if you want i can show you some codingthings.
... and use github :D it's much easier for code handling ;)

best regards
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

Geisel wrote:hey matjojo,

first of all i like your addon so much! I allways did mainlines the "Armuba" way (or Steejo who know's ;) and you make my life easier :)

But i looked at your code, and i think you can improve it and make it more clean, if you want i can show you some codingthings.
... and use github :D it's much easier for code handling ;)

best regards
Thanks, I really like it when people tell me they like what I make.

I'd be really interested in learning some things to make the code a bit cleaner, I actually am doing this modding thing to learn code, I did some small things before, but nothing like this. Send me a PM and we'll figure something out.

I actually used Github before, (or any git system) but I really disliked the way the github program worked on windows, (I come from Xubuntu) and I really dislike the windows "powershell" and "cmd". I'll set it up right now actually, as I really should.
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.27] {BB} - Belt Brush

Post by matjojo »

Jierdan wrote:I think it was related to one of the other mods I have installed, but I just tried to place a belt when it gave that error.

Added the mod list I was using during the crash(Factorio 12.29).
Two more questions:
Where you playing multiplayer?

Did you ever get this problem again?
matjojo
Filter Inserter
Filter Inserter
Posts: 337
Joined: Wed Jun 17, 2015 6:08 pm
Contact:

Re: [mod 0.12.29] {BB} - Belt Brush

Post by matjojo »

Just uploaded version 0.0.3:

Version 0.0.3 (2016-03-25): Fixed some more multi-player problems and a single-player bug

please leave any feedback you want to.
Jierdan
Manual Inserter
Manual Inserter
Posts: 3
Joined: Mon Aug 24, 2015 5:44 am
Contact:

Re: [mod 0.12.29] {BB} - Belt Brush

Post by Jierdan »

So, got this from testing just version 0.0.3 in SP: Error while running the event handler: __BeltBrush__/control.lua:156: attempt to index global 'BB_glob_amount' (a nil value)
Also, the folder path wasn't correct(Zip needs the files inside a BeltBrush_0.0.3 folder).
Post Reply

Return to “Mods”