saif-ellafi / foundryvtt-mythic-gme

Mythic GM Emulator Macros and Roll Tables
Other
27 stars 15 forks source link

Fate chart and scene alteration macros fail with GURPS #3

Open jonatanhedborg opened 2 years ago

jonatanhedborg commented 2 years ago

Attached console log and screenshot of said console output

I'm running Foundry version 0.8.9.

image 127.0.0.1-1636150700422.log

saif-ellafi commented 2 years ago

Hello @jonatanhedborg thank you for taking the time to come and report this issue.

Could you share a bit of context, what were you doing at the time? - Is there any modules that could be affecting Rolling Dice or Sending Dice to the Chat that could affect this module? Also which System are you on?

jonatanhedborg commented 2 years ago

@saif-ellafi No problems, thanks for implementing this!

I was simply trying to test it out, dragged the macros to the bar and tried them.

I will try with a different system and see if it works differently.

jonatanhedborg commented 2 years ago

It appears to be working in the Savage Worlds system, so it's probably some incompatibility with GURPS (I just tried creating a new, clean world in case there was something messed up in some migration, but it did not help).

saif-ellafi commented 2 years ago

Thank you for investigating. Will flag this for when I get some time. Could be worth while to share with GURPS people?

mjeffw commented 2 years ago

I am one of the coauthors of the GURPS ruleset. I have installed this module and imported the macros, but I don't see what is wrong. I can roll on any of the included roll tables without error. How do I replicate the bug?

jonatanhedborg commented 2 years ago

@mjeffw It happens (or rather, doesn't happen) when you try to use the macros for fate chart and scene alteration.

mjeffw commented 2 years ago

Okay -- I see the problem now. The cause is that the GURPS system does not use the standard roll chat message. I could try to fix the immediate problem -- that the roll message data does not have a 'total' value -- but that may just reveal yet another problem. Instead of chasing them one by one, could you describe what this module is attempting to do?

mjeffw commented 2 years ago

Actually, I don't know what the problem is here -- it doesn't look like we're creating a custom chat message.

EDIT: I found the cause. The GURPS System overrides the standard Dice type with its own (GurpsDie) to allow for certain special abilities, and this module tries to create a roll chat message with the standard Dice type. I can try to allow both Dice types to be valid, but it might be dicey (sorry, couldn't resist).

saif-ellafi commented 2 years ago

Thank you @mjeffw for looking into this. I have of course no idea about how GURPS work or your implementation, but indeed it might help for module compatibility to have standard APIs wherever possible :) best regards!

saif-ellafi commented 2 years ago

@mjeffw Hi there! - Not sure if there is any progress on the compatibility between this module and your system? :)

saif-ellafi commented 2 years ago

image

horusofoz commented 1 year ago

@jonatanhedborg Is this still an issue with the latest version of Mythic GME Tools?