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.

    MMM-OpenWeatherMapForecast Doesn't display imperial units

    Scheduled Pinned Locked Moved Troubleshooting
    21 Posts 6 Posters 6.9k Views 6 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.
    • Mykle1M Offline
      Mykle1 Project Sponsor Module Developer @rslohs
      last edited by

      @rslohs

      Just a thought: At the top of your config.js file, what are your config units set to?

      Example:

      var config = {
          address: "0.0.0.0",
          port: 8080,
          ipWhitelist: [], // Set [] to allow all IP addresses
          language: "en",
          timeFormat: 12,
          units: "imperial",
      

      Create a working config
      How to add modules

      R 1 Reply Last reply Reply Quote 0
      • R Offline
        rslohs @Mykle1
        last edited by

        @mykle1 thanks for the suggestion, but that is set to imperial as well.

        S 1 Reply Last reply Reply Quote 0
        • S Offline
          sdetweil @rslohs
          last edited by sdetweil

          @rslohs its not “Units”, its “units”

          this system is case sensitive, every letter matters

          Sam

          How to add modules

          learning how to use browser developers window for css changes

          R 1 Reply Last reply Reply Quote 0
          • R Offline
            rslohs @sdetweil
            last edited by sdetweil

            @sdetweil that’s what I’m using in the config file.

            {
            module: "MMM-OpenWeatherMapForecast",
            header: "Schoolcraft, MI Weather",
            position: "top_right",
            classes: "default everyone",
            disabled: false,
            config: {
            apikey: "aaaaaaaa",
            latitude: "aaaaaaa",
            longitude: "aaaaaa",
            iconset: "4c",
            concise: false,
            forecastLayout: "table",
            units: "imperial",
            }
            },
            
            S 1 Reply Last reply Reply Quote 0
            • S Offline
              scumbelly @rslohs
              last edited by

              @rslohs

              A little late to weigh in but I can verify that imperial setting does not affect the Pop amount. It remains metric. Everything else is imperial.

              Sorry.

              1 Reply Last reply Reply Quote 0
              • S Offline
                scumbelly @rslohs
                last edited by

                @rslohs

                Guess what? I was trying to troubleshoot an issue in this module re: scale and noticed the .js file has changed wind calculations and the following…

                units: {
                accumulationRain: {
                imperial: “mm”,
                metric: “mm”,
                “”: “mm”
                },
                accumulationSnow: {
                imperial: “mm”,
                metric: “mm”,
                “”: “mm”
                },
                windSpeed: {
                imperial: “mph”,
                metric: “m/s”,
                “”: “m/s”
                }
                },

                So it looks like imperial might work for you now. I’m not going to test because I need the older .js which I had to modify to get some calculations in kph to work .

                Cheers

                1 Reply Last reply Reply Quote 0
                • J Offline
                  JohnGalt
                  last edited by

                  My instance of this module also displays rainfall in “mm”, even though I have set units = “imperial” for the MagicMirror system as a whole and also in the module configuration.

                  I see that the Open Weather site has a page specifying “API response parameters with available units of measurement” (https://openweathermap.org/weather-data). There one finds a parameter table specifying Parameter and Description - as well as columns for “Standard”, “Metric” and “Imperial”.

                  The interesting thing is this: the table specifies temperatures correctly (“Kelvin”, “Celsius”, “Fahrenheit”) but for rain it calls out “mm”, “mm”, “mm” - or other variants for different fields, e.g.: “mm/hr” and “mm/hour”.

                  As such, I am inclined to just give up and accept the module’s behavior as a quirk of how Open Weather parses what it is getting from the NWS - but I thought i would see if anyone has a better idea.

                  kayakbabeK 1 Reply Last reply Reply Quote 0
                  • kayakbabeK Offline
                    kayakbabe @JohnGalt
                    last edited by

                    @JohnGalt have you tried MMM-OpenWeatherForecast instead? It seems to work better for me adn it uses the OpenWeather Map api too.

                    J 3 Replies Last reply Reply Quote 0
                    • J Offline
                      JohnGalt @kayakbabe
                      last edited by

                      @kayakbabe - Thanks for the tip. Can you tell me which one you use? A quick web search shows several modules with the same name. These two look like top contenders:

                      https://github.com/jclarke0000/MMM-OpenWeatherForecast
                      https://github.com/themaxgoldman/MMM-OpenWeatherForecast

                      kayakbabeK 1 Reply Last reply Reply Quote 0
                      • J Offline
                        JohnGalt @kayakbabe
                        last edited by

                        @kayakbabe PS: another issue with MMM-OpenWeatherMapForecast is that it doesn’t respect the “disabled: true” configuration. I had seen this before and ignored it since the module is active, but when I went in to disable it in preparation for trying one of the others, I am reminded that I cannot just disable it. I CAN put " /* and */ " around the whole thing of course, but that is just clunky, and more importantly may signal more serious coding issues.

                        S 1 Reply Last reply Reply Quote 0
                        • 1
                        • 2
                        • 3
                        • 1 / 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