henokemp / lit-feelingthepast

A chapter for the Lives in Transit game!
Other
1 stars 0 forks source link

Validator errors in my forked version #10

Open martindusinberre opened 4 years ago

martindusinberre commented 4 years ago

@henokemp A final thing: I ran the validator in my forked version just now. Hopefully @interrogator has already set you up with the validator and you may have picked these errors up already, but if not:

OK, that's it from me for the time being! Sorry to be slow in doing all this, but hope it gives you some stuff to think about in advance of our next stand-up meeting on Tuesday. All best, Martin

henokemp commented 4 years ago

@interrogator Already informed me about the png vs jpg errors and I've fixed them.

No worries about the speed. I'll make the changes as described in the other issues and mainly think about dialog. As I already mentioned, I was able to play up until 1.5.1 sometime past week. This means I was already able to fix some issues. I'm looking forward to the next import to the text-system. :)

henokemp commented 4 years ago

Regarding the validator: @interrogator did not personally set me up, rather I just downloaded it myself. I am able to use it, but the output is a little confusing, to be honest, and might be the reason why the system is crashing as soon as I push new changes.

When I run maruval lit-feelingthepast the out put looks like this:

Validating content at /home/henrik/lit-feelingthepast
107 JSON files found.

====================================================================================================
Problem #1 -- Content error in /home/henrik/lit-feelingthepast/chapter1/1.0.0/imageComponent1.json
----------------------------------------------------------------------------------------------------

/home/henrik/lit-feelingthepast/chapter1/1.0.0/imageComponent1.json has key 'images', but resources/1.0.0_garden.jpg not found.

====================================================================================================
Problem #2 -- Content error in /home/henrik/lit-feelingthepast/chapter1/1.0.0/pageTransition1.json
----------------------------------------------------------------------------------------------------

/home/henrik/lit-feelingthepast/chapter1/1.0.0/pageTransition1.json has key 'to', but chapter1/1.0.1/page.json not found.

====================================================================================================
Problem #3 -- Content error in /home/henrik/lit-feelingthepast/chapter1/1.0.0/pageTransition2.json
----------------------------------------------------------------------------------------------------

/home/henrik/lit-feelingthepast/chapter1/1.0.0/pageTransition2.json has key 'to', but chapter1/1.0.1/page.json not found.

====================================================================================================
Problem #4 -- Content error in /home/henrik/lit-feelingthepast/chapter1/1.0.1/character1.json
----------------------------------------------------------------------------------------------------

/home/henrik/lit-feelingthepast/chapter1/1.0.1/character1.json has key 'image', but resources/character1.jpg not found.

====================================================================================================
Problem #5 -- Content error in /home/henrik/lit-feelingthepast/chapter1/1.0.1/imageComponent1.json
----------------------------------------------------------------------------------------------------

/home/henrik/lit-feelingthepast/chapter1/1.0.1/imageComponent1.json has key 'images', but resources/1.0.0_garden.jpg not found.

====================================================================================================
Problem #6 -- Content error in /home/henrik/lit-feelingthepast/chapter1/1.0.1/mail1.json
----------------------------------------------------------------------------------------------------

/home/henrik/lit-feelingthepast/chapter1/1.0.1/mail1.json has key 'from', but chapter1/1.0.1/character1.json not found.

In total 67 content errors looking similar to this, I assume for every pageTransition and image one. I'm a little lost how I can fix this, maybe @interrogator can help.

interrogator commented 4 years ago

be sure to cd into the the root of lit-feelingthepast and run maruval ., rather than being outside the game directory and doing maruval lit-feelingthepast. Currently i get no errors for master branch doing it that way.

and be sure you have the most up-to-date validator running:

pip uninstall marugoto-validator
pip install git+https://github.com/uzh/marugoto-validator

This is the most recent version of the validator, pip install marugoto-validator may be a bit out of date, my bad.

interrogator commented 4 years ago

@henokemp I just put your game back online on the test system. Not sure why it was down, nature of the beast unfortunately.

interrogator commented 4 years ago

@martindusinberre those errors are fixed now, and may have been at the time you ran the validation. Going forward, make sure you stay on master branch, git pull and have the latest content before validating this game. (obviously, different workflow from PL where you are the sole contributor)