• Recent
  • Tags
  • Unsolved
  • Solved
  • MagicMirror² Repository
  • Documentation
  • 3rd-Party-Modules
  • Donate
  • Discord
  • Register
  • Login
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 15.2k 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.
  • N Offline
    nwootton Module Developer
    last edited by nwootton Mar 17, 2017, 11:46 AM Mar 16, 2017, 10:39 PM

    @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 Apr 10, 2017, 9:29 AM Apr 10, 2017, 9:23 AM

      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

      N S 2 Replies Last reply Apr 10, 2017, 4:23 PM Reply Quote 0
      • N Offline
        nwootton Module Developer @soqueen
        last edited by Apr 10, 2017, 4:23 PM

        @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
        • S Offline
          strawberry 3.141 Project Sponsor Module Developer @soqueen
          last edited by Apr 10, 2017, 6:07 PM

          @soqueen there is a comma missing after limit: 5

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

          N 1 Reply Last reply Apr 10, 2017, 6:46 PM Reply Quote 1
          • S Offline
            soqueen
            last edited by Apr 10, 2017, 6:18 PM

            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
            • N Offline
              nwootton Module Developer @strawberry 3.141
              last edited by Apr 10, 2017, 6:46 PM

              @strawberry-3.141 good catch!

              1 Reply Last reply Reply Quote 0
              • S Offline
                soqueen
                last edited by Apr 10, 2017, 6:46 PM

                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 Apr 10, 2017, 6:54 PM

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

                  N 1 Reply Last reply Apr 10, 2017, 7:04 PM Reply Quote 0
                  • N Offline
                    nwootton Module Developer @soqueen
                    last edited by nwootton Apr 10, 2017, 7:14 PM Apr 10, 2017, 7:04 PM

                    @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
                    • S Offline
                      strawberry 3.141 Project Sponsor Module Developer
                      last edited by Apr 10, 2017, 7:16 PM

                      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

                      N 1 Reply Last reply Apr 10, 2017, 7:40 PM 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