remoteFile in compliments module

roramirez

@jm What memory bug? I’ve don’t touch nothing there. @bhepler said about the improvement

If you can help with this the Pull Requests are welcome :)

jm

@roramirez I meant this comment:

“As an aside, I’ve noticed that the weather integration technique continually adds to the arrays. So the memory usage will grow and grow at whatever frequency you set the compliment refresh. This isn’t terrible, as straight text doesn’t take a whole lot of memory. But it may have unintended effects”

I don’t know whether to call this a bug, but you get my point ;) I’m not an experienced coder whatsoever, so I doubt I might be of help, but I am always willing to test / assist.

strawberry 3.141

@jm I can’t see that this is still happening on the current code base. Every time it tries to update an array will be generated with all possible values, after that the garbage collector will do his job and delete the array after usage.

ZtM

Here is how I got the remote file compliments to work:

The config.js settings must name the remote file with no path:

		{
			module: 'compliments',
			position: 'bottom_bar',
			config: {
				remoteFile: 'compliments.json'
			}
		},

IMO it would be great if it were possible to put a full path the the remote file so it can be stored in the config directory.

Create a compliments.json file and be sure to put double quotes around the array names like so:

{
	"morning" : [
		"You're good enough, you're smart enough and doggone it, people like you!"
	],
	"afternoon" : [
		"Yes those jeans make you look fat."
	],
	"evening" : [
		"Please put on some pants."
	]
}

Put the remote file (compliments.json) into the ~/MagicMirror/modules/default/compliments directory.
Restart MM. It will show the default compliment (usually “Hi Sexy”) for the first interval and then will start showing compliments from the external file.

Hope this helps

Lahim

Guys where can be mistake:
config.js looks like:
{ module: “compliments”, position: “lower_third”, config: { remoteFile: “compliments.json” } },
Remote file placed in ~/MagicMirror/modules/default/compliments
but it still does not work.
I made all following above written steps and nothing - still oryginal compliments are one the mirror.
I have no idea what is wrong.

Mykle1

@Lahim

Your config entry looks okay. However, the curly quotes I see will cause problems. There should not be any in your config.js or your compliments.json file. This is an acceptable quote " but this “ is not. You can post your compliments.json file here (in a code block) so we can take a look at it.

Lahim

I have “ in oryginal file so I just copied it. I also tried with ' but without luck.

The compliments.json file looks like:

{
    "anytime" : [
        "Dziala!"
    ],
    "morning" : [
        "Good morning, sunshine!",
        "Who needs coffee when you have your smile?",
        "Go get 'em, Tiger!"
    ],
    "afternoon" : [
        "dfadfafa",
        "dffffffff",
        "fsssss!"
    ],
    "evening" : [
        "2",
        "1",
        "3333"
    ]
}

In compliments.js

Module.register("compliments", {

        // Module config defaults.
        defaults: {
                compliments: {
                        anytime: [
                                "Hey there sexy!"
                        ],
                        morning: [
                                "Good morning, handsome!",
                                "Enjoy your day!",
                                "How was your sleep?"
                        ],
                        afternoon: [
                                "Hello, beauty!",
                                "You look sexy!",
                                "Looking good today!"
                        ],
                        evening: [
                                "Wow, you look hot!",
                                "You look nice!",
                                "Hi, sexy!"
                        ]
                },
                updateInterval: 30000,
                remoteFile: null,
                fadeSpeed: 4000
        },

I see the “anytime” are without "
so I also tried this in …json file but does not work.

Mykle1

@Lahim said in remoteFile in compliments module:

so I also tried this in …json file but does not work.

So, I copied your example compliments.json file from above and tested it. Here is my config.js entry:

{
    disabled: false,
    module: "compliments",
    position: "lower_third",
    config: {
        updateInterval: 30000,
        fadeSpeed: 1000,
        remoteFile: "compliments.json", // this file is in the compliments module folder
}
},

And this was the result:

So, as long as you haven’t changed anything else and your config entry is correct, and your compliments.json file is name correctly and in the compliments module folder, it should work, as it does here

Lahim

Thank you for trying and helping but I copied your setup but still the same thing.

Does compliments.json file has to be somehow written as json file? Or I can create new text file in nano and save it as compliments.json file? Does it need any special rights?
Can you attach your .json file here so I can download and place it in the /compliments/ folder?

Here is list of my files in compliments folder

Mykle1

@Lahim said in remoteFile in compliments module:

Does compliments.json file has to be somehow written as json file?

Uhh, it MUST be saved as a .json file. It’s in the README file of compliments module.

QUOTE:
“This file must be straight JSON. Note that the array names need quotes around them (“morning”, “afternoon”, “evening”, “snow”, “rain”, etc.).”

I don’t use nano if I don’t have to. I use notepad++. You don’t need the terminal to use it and it saves files in just about every file format imaginable, including JSON.

I will post my file in a moment