2 problems with MMM-MyCommute module

[Bob]

Brief bit of background.

I have 2 offices that I go to. One is about a mile from where I live and I have 4 ways to get to there (walking, tram, train or bus). The other is about 30 miles away with a couple of alternative driving routes. Depending on which office I’m going to (and whether I want to walk or take public transport), I want the module to show me what is likely to be the best option to get to each of the specific offices.

This is where I have 2 problems I’m trying to resolve and could do with some help.

1. For driving, I never have anything but a “green” commuting time (ie, it never goes yellow or red for delays, even when adjusted to 1% and 5% delay thresholds)

2. For the closer office I can’t get the alternative transit modes to work (ie, train, tram, bus etc). If I specify anything within the transitMode option, I get Error displayed in the module. If I just use transit (with no transitmode) it works fine, although I can’t then pick which mode of transit to use.

My code is below (API is edited out, as are home & work postal codes - the rest is cut and paste)

{
	module: 'MMM-MyCommute',
	header: "Work Commute",
	position: 'bottom_right',
	config: {
		api_key: 'xxxxxxxxxxxxxxxxxxxxxxxxxxx',
		origin: 'Home Postal Code',
		showSummary: true,
		startTime: '00:00',
		endTime: '23:59',
		colorCodeTravelTime: true,
		moderateTimeThreshold: 1.01,
		poorTimeThreshold: 1.05,
		destinations: [
			{
			destination: 'Work  1 Postal Code', // Farther office, by car, alternatives on
			label: 'Office 1',
			color: '#1E90FF', //Blue
			alternatives: true,
			},
			{
			destination: 'Work  2 Postal Code', // Closer office, walking, alternatives off
			label: 'Office 2 - Walking',
			mode: 'walking',
			color: '#98FB98', // Green
			alternatives: false,
			},
			{
			destination: 'Work 2 Postal Code',
			label: 'Office 2 - Train',
			mode: 'transit',
			transitMode: 'train',
			showNextVehicleDeparture: true,
			color: '#ffff00', //Yellow
			alternatives: false,
			},
			{
			destination: 'Work 2 Postal Code,
			label: 'Office 2 - Tram',
			mode: 'transit',
			transitMode: 'tram',
			showNextVehicleDeparture: true,
			color: '#ffff00', //Yellow
			alternatives: false,
			},
			{
			destination: 'Work 2 Postal Code,
			label: 'Office 2 - Bus',
			mode: 'transit',
			transitMode: 'bus',
			showNextVehicleDeparture: true,
			color: '#ffff00', //Yellow
			alternatives: false,
			},
			]
	},
},

I’ve tried all sorts of editing, using both single and double quotes, and am using Geany to check the syntax (which I believe is correct)

Any ideas ?

[strawberry 3.141]

@Bob I can spot multiple missing quotes

[Bob]

@strawberry-3-141 Where are there quotes missing ? I could only see a couple (which I’ve corrected in my first post). I edited the code to remove identifying info, so it should be ok now. Normally when there is a code error due to syntax (such as quotes), MM won’t even run.

In this instance, I simply get a line error for the specific mode of transport.

[Bob]

This is what I get using the code above.

[strawberry 3.141]

@Bob two times you have destination: 'Work 2 Postal Code, instead of destination: 'Work 2 Postal Code',

[yawns]

{
	module: 'MMM-MyCommute',
	header: "Work Commute",
	position: 'bottom_right',
	config: {
		api_key: 'xxxxxxxxxxxxxxxxxxxxxxxxxxx',
		origin: 'Home Postal Code',
		showSummary: true,
		startTime: '00:00',
		endTime: '23:59',
		colorCodeTravelTime: true,
		moderateTimeThreshold: 1.01,
		poorTimeThreshold: 1.05,
		destinations: [
			{
			destination: 'Work  1 Postal Code', // Farther office, by car, alternatives on
			label: 'Office 1',
			color: '#1E90FF', //Blue
			alternatives: true,
			},
			{
			destination: 'Work  2 Postal Code', // Closer office, walking, alternatives off
			label: 'Office 2 - Walking',
			mode: 'walking',
			color: '#98FB98', // Green
			alternatives: false,
			},
			{
			destination: 'Work 2 Postal Code',
			label: 'Office 2 - Train',
			mode: 'transit',
			transitMode: 'train',
			showNextVehicleDeparture: true,
			color: '#ffff00', //Yellow
			alternatives: false,
			},
			{
			destination: 'Work 2 Postal Code',
			label: 'Office 2 - Tram',
			mode: 'transit',
			transitMode: 'tram',
			showNextVehicleDeparture: true,
			color: '#ffff00', //Yellow
			alternatives: false,
			},
			{
			destination: 'Work 2 Postal Code',
			label: 'Office 2 - Bus',
			mode: 'transit',
			transitMode: 'bus',
			showNextVehicleDeparture: true,
			color: '#ffff00', //Yellow
			alternatives: false,
			}
		]
	},
},

[Bob]

@strawberry-3-141 I saw that, and fixed it (just not in my post)

@yawns I cut and pasted your code (adjusted the API key and the postal codes), and still get the same error (as per previous screenshot)

[Bob]

I’m guessing no-one has any other ideas ? I wondered if there was a conflict with any other modules, so I’ve done a full rebuild of the RPi, stripped the config to just this module, and it still doesn’t work properly.

Ho hum

[j.e.f.f]

@Bob I’m the author of this module. Can you send me a private message? I’ll see if I can figure out what’s going on.

[knubbl]

I got the same problem. So im really interested what is the problem and how to fix.

Thanks

[j.e.f.f]

@Bob @knubbl The issue is that the config has api_key: "...". It should be apikey: "..." (no underscore)

[Bob]

@j.e.f.f Thanks Jeff, problem solved, and everything now works. Typical that Google allows you to use the API without a valid key, and doesn’t error out to say it’s not a valid entry :(

Oh for the good old days of 2 word *nix error messages that required a manual the size of war and peace to troubleshoot…but at least you knew there was an error ! :)

[shakir]

this helped immensely. i got my MM exactly where i want it now and can cancel CTATracker. sorry for being a nub