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.

    Possible Issue: MMM-GoogleAssistant with recent Google Assistant SDK update

    Scheduled Pinned Locked Moved Troubleshooting
    4 Posts 3 Posters 5.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.
    • T Offline
      teamreflect
      last edited by teamreflect

      Tried installing the MMM-GoogleAssistant today and tried to make it with the work with new google assistant sdk update… I ran in to this following error code:

      File "assistant.py", line 133, in 
        main()
      File "assistant.py", line 130, in main
        init_googleAssistant()
      File "assistant.py", line 124, in init_googleAssistant
        with Assistant(credentials) as assistant:
      TypeError: __init__() missing 1 required positional argument: 'device_model_id'
      hello from python!!
      

      any ideas on what is causing the problem?

      1 Reply Last reply Reply Quote 0
      • R Offline
        Raddy
        last edited by

        I’m having the same issue. I followed the instructions listed link text here and can get the sample assistant script to run from Google, but when I run the assistant.py script, I get the same error as above.

        1 Reply Last reply Reply Quote 0
        • R Offline
          Raddy
          last edited by Raddy

          Solved it!

          The SDK update changed the assistant initiation call. You registered a device_model_id (“Raddys-magic-mirror-device-id” for example) in the original instructions, so you have to add that to the assistant.py file. (Here are instructions on manually registering a device and creating a device instance–I don’t know if an instance is required for this to work, but it’s part of what I did trying to solve this).

          Open the assistant.py code and scroll down to line 124. The current call is this:

          with Assistant(credentials) as assistant:
          

          In the updated method found on Google’s dev site, it should read:

          with Assistant(credentials, device_model_id) as assistant:
          

          Where you replace device_model_id with whatever you registered before.

          with Assistant(credentials, "Raddys-magic-mirror-device-id") as assistant:
          

          Run it and it should work!

          gauravsaccG 1 Reply Last reply Reply Quote 0
          • gauravsaccG Offline
            gauravsacc Module Developer @Raddy
            last edited by

            @Raddy said in Possible Issue: MMM-GoogleAssistant with recent Google Assistant SDK update:

            d method f

            just an update. I’ve updated the MMM-GoogleAssistant module to work with the new SDK :)

            1 Reply Last reply Reply Quote 0

            Hello! It looks like you're interested in this conversation, but you don't have an account yet.

            Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

            With your input, this post could be even better 💗

            Register Login
            • 1 / 1
            • 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