London Bus status

[strawberry 3.141]

@soqueen there is a comma missing after limit: 5

[soqueen]

That certainly helped! The code loads now and now it’s stuck on Loading bus info… time to recheck my api codes?

[nwootton]

@strawberry-3.141 good catch!

[soqueen]

Thank you both! I have gotten it up and running! Looks GREAT!

[soqueen]

@nmootton Hi there! Thank you for being so responsive! I had another question. I actually want to load 3 of these. I tried loading the module three separate times, but it just loads 3 of the initial entry, even though the atcocodes are different for each one. Any suggestions?

[nwootton]

@soqueen I have to say I’ve never considered it. Both this and the rail version were something I built more as an experiment than anything else. I’m not even sure I’d know how to handle multiple instances of a module.

I’ll have to have a think/read and see how it could be done. Perhaps other more experienced devs may be able to chip in here?

I will say that running three of these could get expensive in terms of api calls per day… if you use the next bus capability that would be 30 points per call or a max of 33 calls per day (~ every 45 minutes), which might not be enough to make the module really useful. You could get three distinct API keys I guess, but I’m not sure how TransportAPI would view that!

I did consider some form of time range, but again it wasn’t something I needed for the experiment. Perhaps I need to find some time to reconsider that as an option!

[strawberry 3.141]

your module has a node_helper

if there are 3 instances in the config of this module, all 3 share the same node_helper and therefore get the same socketnotifications

workaround that some users have done is copy the module folder and named it modulename2 also changed the modulename.js to modulename2.js and in the js file the string from modulename to modulename2. this will create a node_helper for modulename and one for modulename2

what a dev could do is the following, have the updateinterval in the module which is sending the requesturl to the nodehelper, so every instance sends it and gets executed and then you send the result back to the module and also for which station this request was performed, then you check in the module if the result is based on the station of this instance then process the data otherwise ignore that socketnotification.

[nwootton]

@strawberry-3.141 If I get the chance I’ll look at passing an identity to the helper so it knows where to route the response back to.

[soqueen]

Hm. I tried that. I copied the module and contents to modulename2, changed the .js to 2.js - edited the 2.js to (module.register) to modulename2 .

Then I changed my config file - copied the first module to another instance and changed its name to modulename2. (then i changed the atcocode for a new bus stop). --It’s stuck on loading bus info…

[nwootton]

@soqueen I’ve just pushed a new version that hopefully allows you to have multiple instances of the module running. It’s been working happily this afternoon running three different bus stops around the area with no complaints.

You’ll need to do a git pull to update, then run the

cd MMM-UKLiveBusStopInfo
npm install 

to install the new dependencies.

Give it a try and let me know. If it goes pear-shaped I can always roll the old version back and look again later.

[soqueen]

It works! Thank you SO VERY much! I really appreciate it! Thank you!

[soqueen]

HI there! One of the newest updates took off the heading for my multiple instances. It only puts a header on the first one. I tried adding a header in the config.js file but it ignors it. Help please?

[nwootton]

@soqueen mmm weird, didn’t do that for me in testing. I’ll have a look later when I get home.

[soqueen]

updated and now even the first one is missing. :)

[nwootton]

@soqueen I’ve just hacked a fix together. Give it a try and let me know.

[nwootton]

@soqueen I’ve just noticed some weird behaviour this end while trying to fix this. Did you update your Mirror to 2.1.1? If so did you then do a new npm install afterwards?

I ask because on one box the departures now shows no header at all, while the other now shows both departures AND the bus stop name…

[soqueen]

I re-ran the npm install just in case I hadn’t done it before, and then pulled your update. Now the headers are back! Thanks!

[nwootton]

@soqueen TBH I rolled the master branch back to an earlier version that worked and branched the version with the bus stop name as the header.

Once I get the last couple of issues in that branch version sorted I’ll role it back into the master. If you want to try it and give me feedback, the branch is called HeadersIssue.