your-land/bugtracker
your-land
/
bugtracker
44
20
Code Issues 2.5k Projects 2 Wiki

Petz #6129

New Issue
Closed
opened 2024-01-29 02:10:42 +00:00 by acmgit · 12 comments
acmgit commented 2024-01-29 02:10:42 +00:00
Copy Link
2024-01-29 03:06:55: ERROR[Main]: ServerError: AsyncErr: Lua: Runtime error from mod 'petz' in callback luaentity_Step(): ...alltech_Game/mods/mp_mobs/petz/petz/brains/bh_attack.lua:89: attempt to perform arithmetic on field 'height' (a nil value)
2024-01-29 03:06:55: ERROR[Main]: stack traceback:
2024-01-29 03:06:55: ERROR[Main]: 	...alltech_Game/mods/mp_mobs/petz/petz/brains/bh_attack.lua:89: in function 'func'
2024-01-29 03:06:55: ERROR[Main]: 	...t/games/Smalltech_Game/mods/mp_mobs/petz/kitz/engine.lua:675: in function 'execute_queues'
2024-01-29 03:06:55: ERROR[Main]: 	...t/games/Smalltech_Game/mods/mp_mobs/petz/kitz/engine.lua:867: in function 'stepfunc'
2024-01-29 03:06:55: ERROR[Main]: 	...tech_Game/mods/mp_mobs/petz/petz/petz/chicken_mobkit.lua:150: in function 'func'
2024-01-29 03:06:55: ERROR[Main]: 	/usr/share/minetest/builtin/profiler/instrumentation.lua:108: in function </usr/share/minetest/builtin/profiler/instrumentation.lua:101>

From Server Zeitsprung.

``` 2024-01-29 03:06:55: ERROR[Main]: ServerError: AsyncErr: Lua: Runtime error from mod 'petz' in callback luaentity_Step(): ...alltech_Game/mods/mp_mobs/petz/petz/brains/bh_attack.lua:89: attempt to perform arithmetic on field 'height' (a nil value) 2024-01-29 03:06:55: ERROR[Main]: stack traceback: 2024-01-29 03:06:55: ERROR[Main]: ...alltech_Game/mods/mp_mobs/petz/petz/brains/bh_attack.lua:89: in function 'func' 2024-01-29 03:06:55: ERROR[Main]: ...t/games/Smalltech_Game/mods/mp_mobs/petz/kitz/engine.lua:675: in function 'execute_queues' 2024-01-29 03:06:55: ERROR[Main]: ...t/games/Smalltech_Game/mods/mp_mobs/petz/kitz/engine.lua:867: in function 'stepfunc' 2024-01-29 03:06:55: ERROR[Main]: ...tech_Game/mods/mp_mobs/petz/petz/petz/chicken_mobkit.lua:150: in function 'func' 2024-01-29 03:06:55: ERROR[Main]: /usr/share/minetest/builtin/profiler/instrumentation.lua:108: in function </usr/share/minetest/builtin/profiler/instrumentation.lua:101> ``` From Server Zeitsprung.
flux added the
1. kind/invalid
5. result/wontfix
 labels 2024-01-29 23:31:55 +00:00
flux commented 2024-01-29 23:36:13 +00:00
Member
Copy Link

this is not the place to report petz bugs that don't specifically affect the your-land server. please report this upstream: https://github.com/runsy/petz/issues

this is not the place to report petz bugs that don't specifically affect the your-land server. please report this upstream: https://github.com/runsy/petz/issues
Bla commented 2024-01-30 00:06:31 +00:00
Copy Link

this is not the place to report petz bugs that don't specifically affect the your-land server. please report this upstream: https://github.com/runsy/petz/issues

probably because of https://forum.minetest.net/viewtopic.php?p=431721#p431721 and the following post

> this is not the place to report petz bugs that don't specifically affect the your-land server. please report this upstream: https://github.com/runsy/petz/issues > probably because of https://forum.minetest.net/viewtopic.php?p=431721#p431721 and the following post
flux commented 2024-01-30 02:04:36 +00:00
Member
Copy Link

probably because of https://forum.minetest.net/viewtopic.php?p=431721#p431721 and the following post

runs has certainly started at least working on the mod somewhat again, though i fear either i or your-land will have to adopt the mod more formally. but that hasn't happened yet.

> probably because of https://forum.minetest.net/viewtopic.php?p=431721#p431721 and the following post runs has certainly started at least working on the mod somewhat again, though i fear either i or your-land will have to adopt the mod more formally. but that hasn't happened yet.
acmgit commented 2024-01-30 02:12:04 +00:00
Author
Copy Link

Right and after i have talked with AliasAlreadyTaken.

Right and after i have talked with AliasAlreadyTaken.
acmgit commented 2024-01-30 02:15:03 +00:00
Author
Copy Link

Sorry.
i didn't know you had such a bureaucracy.
Bye.

Sorry. i didn't know you had such a bureaucracy. Bye.
flux commented 2024-01-30 02:33:44 +00:00
Member
Copy Link

Sorry.
i didn't know you had such a bureaucracy.
Bye.

apologies, i didn't mean to offend. i admire your work, cucina_vegana in particular. we just don't have time to fix all of our own issues (over 2000 are open), let alone those of others. i don't understand the petz code well enough to know what sort of entity it expects to have a "height" parameter or why. only runs knows why that code is the way it is. i could fix the crash with a nil check, but that'd be a PR to upstream, and i know you can do the same yourself.

> Sorry. > i didn't know you had such a bureaucracy. > Bye. apologies, i didn't mean to offend. i admire your work, cucina_vegana in particular. we just don't have time to fix all of our own issues (over 2000 are open), let alone those of others. i don't understand the petz code well enough to know what sort of entity it expects to have a "height" parameter or why. only runs knows why that code is the way it is. i could fix the crash with a nil check, but that'd be a PR to upstream, and i know you can do the same yourself.
flux commented 2024-01-30 02:35:02 +00:00
Member
Copy Link

and i really don't want to encourage the whole minetest community to come here and report all of their problems in the hope that i'll fix them. this is just the your-land bugtracker.

and i *really* don't want to encourage the whole minetest community to come here and report all of their problems in the hope that i'll fix them. this is just the your-land bugtracker.
flux closed this issue 2024-01-30 02:35:02 +00:00
AliasAlreadyTaken commented 2024-01-30 02:35:39 +00:00
Owner
Copy Link

Then let's see whether runs keeps going and if not, let's adopt it formally.

Then let's see whether runs keeps going and if not, let's adopt it formally.
whosit commented 2024-01-30 06:31:23 +00:00
Member
Copy Link

One issue with adopting, is that petz repo has only b3d models. I'm not sure how to edit them keeping the animations, since blender plugin does not support animation import yet... And Runs can be using blender to just export to b3d. If he does, then hopefully he will post original blender files too...

One issue with adopting, is that petz repo has only b3d models. I'm not sure how to edit them keeping the animations, since blender plugin does not support animation import yet... And Runs can be using blender to just export to b3d. If he does, then hopefully he will post original blender files too...
AliasAlreadyTaken commented 2024-01-30 08:20:45 +00:00
Owner
Copy Link

One issue with adopting, is that petz repo has only b3d models. I'm not sure how to edit them keeping the animations, since blender plugin does not support animation import yet... And Runs can be using blender to just export to b3d. If he does, then hopefully he will post original blender files too...

That's where we have a slightly secret weapon ... I believe the original blender files are here: https://github.com/runsy/petz_raw

And on top of that, there's someone who can read them and work with them and already pushed a couple of fixes upstream.

> One issue with adopting, is that petz repo has only b3d models. I'm not sure how to edit them keeping the animations, since blender plugin does not support animation import yet... And Runs can be using blender to just export to b3d. If he does, then hopefully he will post original blender files too... That's where we have a slightly secret weapon ... I believe the original blender files are here: https://github.com/runsy/petz_raw And on top of that, there's someone who can read them and work with them and already pushed a couple of fixes upstream.
acmgit commented 2024-01-30 14:24:36 +00:00
Author
Copy Link

All ok, flux, nothing has happened.
I understand, when you say, you have more than enough own issues to fix.

Runsy has always fixed the issue and this were never a problem with this, but as i read, he don't want to work again on petz,
so i was asking AliasAlreadyTaken.

As long runsy will take care about the issues, i will post him. After them, i don't know, as long petz is running fine, i will have it on my servers, but then, chi sera ;-).

Greetings, Clyde.

All ok, flux, nothing has happened. I understand, when you say, you have more than enough own issues to fix. Runsy has always fixed the issue and this were never a problem with this, but as i read, he don't want to work again on petz, so i was asking AliasAlreadyTaken. As long runsy will take care about the issues, i will post him. After them, i don't know, as long petz is running fine, i will have it on my servers, but then, chi sera ;-). Greetings, Clyde.
👍 1
AliasAlreadyTaken commented 2024-02-26 11:00:44 +00:00
Owner
Copy Link

This is fixed upstream

This is fixed upstream
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
1. kind/balancing

Depends on balancing

  
1. kind/breaking

something that alters or disables an existing game mechanic

  
1. kind/bug

Something is not working

  
1. kind/construction

A build needs fixing

  
1. kind/documentation

Needs proper documentation

  
1. kind/enhancement

New feature

  
1. kind/griefing

players despoiling builds or nature

  
1. kind/invalid

Not a valid report, e.g. created by mistake with no content.

  
1. kind/meme

issue is mostly a joke

  
1. kind/node limit

something which works for or against the node limit

  
1. kind/other

Something not within the other kinds

  
1. kind/protocol

something concerning how staff should react to player requests, or how players should treat each other

  
2. prio/controversial

not likely to happen w/out a lot of discussion

  
2. prio/critical

Very important

  
2. prio/elevated

something players are eager to have fixed

  
2. prio/good first issue

the resolution to this is could be done by a new minetest contributor, who doesn't yet understand the mod ecosystem

  
2. prio/interesting

one of us has real interest in getting this implemented or fixed. however, more research might be needed.

  
2. prio/low

Not so important

  
3. source/art

art needs to be created or found for this to exist. includes sound, 3d modelling, and in-game building.

  
3. source/client

Issue is with the minetest client, or with player client configuration or the technical details of their setup

  
3. source/engine

Issue is due to a bug or feature of the minetest engine

  
3. source/ingame

Problem and solution exist in-game, e.g. builds or player moderation

  
3. source/integration

Two or more nonbuggy mechanics conflict, or our attempts to rectify conflicting mechanics have created a new bug

  
3. source/lag

A problem which is caused or made worse by lag, due to server processing issues, client processing issues, or network issues

  
3. source/license

A licensing problem is involved

  
3. source/mod upstream

Problem is upstream (not engine/client)

  
3. source/unknown

   
3. source/website

Affects the website

  
4. step/approved

Alias has approved this feature, someone should implement it

  
4. step/at work

Being worked on

  
4. step/blocked

we plan to do this, but other work has to be finished first

  
4. step/discussion

issue is under discussion, a solution has not been dictated or agreed upon

  
4. step/help wanted

Need some help

  
4. step/needs confirmation

   
4. step/partially fixed

this is mitigated or partially solved, but more work needs to be done

  
4. step/question

More information is needed

  
4. step/ready to deploy

has been QA tested by another person. has no outstanding issues. might be installed on prod, but needs final testing

  
4. step/ready to QA test

the code is supposedly fixed, but needs to be tested.

  
4. step/want approval

Someone can quickly implement this if Alias approves the change/feature

  
5. result/cannot reproduce

   
5. result/duplicate

This issue or pull request already exists

  
5. result/fixed

problem is fixed

  
5. result/maybe

not planned currently, but maybe some day

  
5. result/wontfix

the reported issue won't be changed, whether it's intentional or not.

  
ugh/petz

yet another petz bugz.

  
ugh/QA main

That's something we can only wait and see whether it hapens again: On the main server

  
ugh/QA NOK

Fix didn't yield the expected results

  
ugh/QA OK

tested

No Label
1. kind/balancing
1. kind/breaking
1. kind/bug
1. kind/construction
1. kind/documentation
1. kind/enhancement
1. kind/griefing
1. kind/invalid
1. kind/meme
1. kind/node limit
1. kind/other
1. kind/protocol
2. prio/controversial
2. prio/critical
2. prio/elevated
2. prio/good first issue
2. prio/interesting
2. prio/low
3. source/art
3. source/client
3. source/engine
3. source/ingame
3. source/integration
3. source/lag
3. source/license
3. source/mod upstream
3. source/unknown
3. source/website
4. step/approved
4. step/at work
4. step/blocked
4. step/discussion
4. step/help wanted
4. step/needs confirmation
4. step/partially fixed
4. step/question
4. step/ready to deploy
4. step/ready to QA test
4. step/want approval
5. result/cannot reproduce
5. result/duplicate
5. result/fixed
5. result/maybe
5. result/wontfix
ugh/petz
ugh/QA main
ugh/QA NOK
ugh/QA OK
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
5 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: your-land/bugtracker#6129
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?