MMM-UkNationalRail - stops updating on mirror

nwootton

Still running without issue this morning, although I’m now getting emails about hitting my daily API usage allowance from the Transport API.

The only errors I’m getting are ones that the module generates when it doesn’t get the expected train info from Transport API - generally around 02:00 when no passenger trains are actually running.

Sorry to say that without being able to replicate the issue, there is very little more I can do.

randomnoise

@nwootton no worries, I’ll be home again on Sun so will try the dev console & see if anything stands out. As you say it may be electron related so I might just do a full pi + mm rebuild / reinstall.

Thanks for having a look in to it.

randomnoise

@nwootton after running since Sunday with no errors & console open, today train times finally crashed around 10:30am… then the entire mirror seems to have fallen over tonight just before I came home… the clock had stopped, all modules not updating and even the dev console was gone!

Hard reset of mirror, but of course nothing to see in pm2 logs as I didn’t start it with pm2…

unless the dev console also logs to a file somewhere then this exercise was in vain.

I think I’ll just try a full reinstall at the weekend & see if it behaves itself. I wish I’d taken a note of all the other tweaks I added to get the pi running as I liked.

randomnoise

@nwootton, Well after a full rebuild I still have never been able to get to the bottom of why it locks up. Still does it on occasion (Although it seems to be less frequent these days).

I have a new weird bug with the train times for you now though! Recently (may have been since the clocks changed, although I only noticed it the other day as I was checking time on a train I was going to catch) I seem to be showing times an hour behind! It’s bizarre as I don’t think there are any settings anywhere in the mirror for adjusting times other than the PI config settings (mine is showing correct time & UTC if I run a ‘date’ command in putty).

Any thoughts on what this could be?

randomnoise

I’ve expanded to show 8 results here as well…

randomnoise

Hmm, weirdly York station works ok.

Maybe something is up with the api data for STN.

randomnoise

It’s definitely not just that those trains being late in my earlier example either-

randomnoise

Ok, think I’ve figured this one out by manually loading the api json in my browser. The earlier train which is late is showing as being over 1 hour late (even though that’s not what it shows on the estimated departure time).

This means that the trains following are still in the list even though they have departed.

Either this has been happening every day atm, or there’s something screwy in the transport api database.

0	
mode	"train"
service	"23540003"
train_uid	"G81206"
platform	null
operator	"SR"
operator_name	"Scotrail"
aimed_departure_time	"12:56"
aimed_arrival_time	"12:56"
aimed_pass_time	null
origin_name	"Aberdeen"
source	"Network Rail"
destination_name	"Glasgow Queen Street"
category	"XX"
service_timetable	

status	"LATE"
expected_arrival_time	"12:58"
expected_departure_time	"12:58"
best_arrival_estimate_mins	-54
best_departure_estimate_mins	-54
1	
mode	"train"
service	"23541003"
train_uid	"G80279"
platform	null
operator	"SR"
operator_name	"Scotrail"
aimed_departure_time	"13:25"
aimed_arrival_time	"13:24"
aimed_pass_time	null
origin_name	"Aberdeen"
source	"Network Rail"
destination_name	"Edinburgh Waverley"
category	"XX"

nwootton

I suggest your best bet is to log an issue with TransportAPI as they might be able to identify if the issue lies with them or with the info coming through from the train operator.

randomnoise

@nwootton - Haven’t heard back from the dev support… I suspect I’m not a big enough user of the service. It’s very strange it only seems to happen for my station though (I’ve only tried a few other stations from the list admittedly).

I have been able to kind of sort it by hacking your .js though. Apologies for crudeness of code…

//With data returned
        if (this.trains.data.length > 0) {
            for (var t in this.trains.data) {
                var myTrain = this.trains.data[t];
				//Check current time vs. departure & don't show it if it's past departure
				var nowtime = new Date();
				nowtime = ("0" + nowtime.getHours()).slice(-2) + ":" +
				("0" + nowtime.getMinutes()).slice(-2) + ":" +
				("0" + nowtime.getSeconds()).slice(-2);
				if (myTrain.actualDeparture < nowtime) 
				{
					//Do nothing
				}
				else{
                //Create row for data item

Its a purely string based comparison so works while time is in 24 hour format (and will bork around midnight, but I don’t get any trains later than that…) Also as its just not displaying the earlier ones I’d need to take that in to account in the loop counter (otherwise max results is also off)… Kind of at least proves it works.

I’ll need to figure out a bit more about js programming to maybe make this work better. If I can do that & figure out how to use git more than just to clone would you be willing to have a config option for something like ‘trim departed trains from results’?