Time Nick Message 13:30 TenPlus1 Hi folks 13:31 TenPlus1 am using latest 5.9 dev and every sign in the area now returns this error: "WARNING[Main]: generateImagePart(): Skipping "signs_lib_color_15px_n.png" as it's out-of-bounds (314,16) for [combine" any idea why ?? 13:39 TenPlus1 What causes it to be out of bounds ? the api.txt doesn't tell us this 13:40 MTDiscord It sounds like it means that some mod is asking the [combine modifier to add a layer to a texture that would be entirely clipped out, thus wasting time and memory. 13:41 MTDiscord In my experience the [combine modifier is actually quite strict, and IIRC it has refused to add layers that are even partially clipped, like drawing a 16x16 texture at -8,-8. 13:42 MTDiscord returns this error: "WARNING[Main]: -- is it actually an error, or is it (as the text suggests) a warning? 13:42 TenPlus1 The fill_line function in the signs_lib api.lua file seems to be the issue according to appguru but using a print statement to show values I cannot find or see the out of bounds texture 13:50 sfan5 just to make sure: your build is less than 7 days old, right? 13:56 MTDiscord If my mod code is causing MT to do something that's just wasting time and/or resources (such as creating de facto duplicate image cache entries) and is entirely preventable by me, then I am thankful when MT produces warning messages that alert me to this and give me the option to act. Of course, if I choose not to act on it, to the detriment of downstream users, it's still good for the engine to inform them. 14:24 sfan5 by the way "some mod is asking the [combine modifier to add a layer to a texture that would be entirely clipped out" is indeed correct 14:24 sfan5 originally to my surprise, partially clipped layers are however allowed 14:24 sfan5 the warning is only printed if you request to put something that is entirely out-of-bounds 14:51 MTDiscord "partially clipped layers are however allowed" and actually work, or just don't generate a warning? Being able to use [combine to crop an image is really useful, since otherwise MT really doesn't offer a [crop operator (things like [sheet are too limited to work in many cases). 14:52 MTDiscord I just know that around the 0.4/5.0 era, I tried using [combine to crop, since it seemed like a pretty obvious use case to me, and it just produced only fully transparent images. I ended up giving up, and hadn't heard about it changing since then so I haven't retried. 14:53 MTDiscord If it DOES work, then that's actually really useful to me because I'm using a mask hack to produce "dig particles" (the new "node" option for particles post-5.6 is nice, but I don't think it works on non-node items) but cropping with [combine would be better. 14:56 sfan5 it works and doesn't generate a warning 15:22 rubenwardy merging #14075 in 10 15:22 ShadowBot https://github.com/minetest/minetest/issues/14075 -- Allow toggling touchscreen mode at runtime by okias 15:22 rubenwardy #13616 also has two approvals, any reason it hasn't been merged? 15:22 ShadowBot https://github.com/minetest/minetest/issues/13616 -- [NO SQUASH] Meshgen tests by numberZero 15:41 sfan5 no, feel free to 15:45 rubenwardy done 22:53 [MTMatrix] Any chance to see v-rob's new GUI/HUD system in 5.9 as experimental? 23:30 v-rob I have a few minor fixes/changes I need to push (and I'll probably split sizers out into another PR), but I still think that it's feasible to have this merged as experimental. 23:32 MTDiscord I would like to see it in 5.9. I can probably find the time to review part of it in about a month.