MagicMirror Forum
    • Recent
    • Tags
    • Unsolved
    • Solved
    • MagicMirror² Repository
    • Documentation
    • 3rd-Party-Modules
    • Donate
    • Discord
    • Register
    • Login
    A New Chapter for MagicMirror: The Community Takes the Lead
    Read the statement by Michael Teeuw here.

    London Bus status

    Scheduled Pinned Locked Moved Requests
    27 Posts 4 Posters 16.5k Views 3 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • nwoottonN Offline
      nwootton Module Developer
      last edited by nwootton

      @Porkiddo

      Just Had a look at the Bus stop module again. When it’s used with certain other modules, it appears to error when trying to render the DOM.

      I managed to get it to fall over when used in conjunction with the commute module. I can see the data is coming in, I can see the params are correct, but an ‘appendChild’ error is being thrown when it tries to draw it on screen.

      I’ll investigate more tomorrow and see if I can figure it out.

      Nope I’m an idiot… the issue was a wrong location in my config “bottom_middle” instead of “bottom_center”

      N

      1 Reply Last reply Reply Quote 0
      • S Offline
        soqueen
        last edited by soqueen

        HI there! Thanks so much for tackling this. I seem to be a mess and can’t get it to load. There must be some mistake in my config file as I get the error upon loading. I have checked off all the boxes in the troubleshooting - as in I have an app_id, app_key and I know the atcocode - I have verified all against the live api docs and it runs without error. I will try to load my config entry and see if you guys can help me spot what I entered wrong. Thanks in advance!!0_1491816537498_config-file.jpg

        nwoottonN strawberry 3.141S 2 Replies Last reply Reply Quote 0
        • nwoottonN Offline
          nwootton Module Developer @soqueen
          last edited by

          @soqueen I’d normally copy and paste the code into either my config or into jsonlint.com to check it’s validity. But, as you’ve posted a picture not a snippet, I can’t do that, so if I type it, it will not be the same as yours!

          However, first glance you’ve got a ‘//’ on your header row (far left), nothing else stands out as being wrong.

          Try pasting the entire config.js into jsonlint.com. That will tell you if the file is properly formed and highlight any errors.

          If that comes back ok, try removing everything except the bus stop module and see if it still fails. If it does paste the reduced config and we’ll see what we can do next.

          1 Reply Last reply Reply Quote 0
          • strawberry 3.141S Offline
            strawberry 3.141 Project Sponsor Module Developer @soqueen
            last edited by

            @soqueen there is a comma missing after limit: 5

            Please create a github issue if you need help, so I can keep track

            nwoottonN 1 Reply Last reply Reply Quote 1
            • S Offline
              soqueen
              last edited by

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

              1 Reply Last reply Reply Quote 0
              • nwoottonN Offline
                nwootton Module Developer @strawberry 3.141
                last edited by

                @strawberry-3.141 good catch!

                1 Reply Last reply Reply Quote 0
                • S Offline
                  soqueen
                  last edited by

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

                  1 Reply Last reply Reply Quote 0
                  • S Offline
                    soqueen
                    last edited by

                    @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?

                    nwoottonN 1 Reply Last reply Reply Quote 0
                    • nwoottonN Offline
                      nwootton Module Developer @soqueen
                      last edited by 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!

                      1 Reply Last reply Reply Quote 0
                      • strawberry 3.141S Offline
                        strawberry 3.141 Project Sponsor Module Developer
                        last edited by

                        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.

                        Please create a github issue if you need help, so I can keep track

                        nwoottonN 1 Reply Last reply Reply Quote 0
                        • 1
                        • 2
                        • 3
                        • 2 / 3
                        • First post
                          Last post
                        Enjoying MagicMirror? Please consider a donation!
                        MagicMirror created by Michael Teeuw.
                        Forum managed by Sam, technical setup by Karsten.
                        This forum is using NodeBB as its core | Contributors
                        Contact | Privacy Policy