Skip to content
  • MySensors
  • OpenHardware.io
  • Categories
  • Recent
  • Tags
  • Popular
Skins
  • Light
  • Brite
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Brand Logo
  1. Home
  2. Troubleshooting
  3. Dallas Temp failure to compile

Dallas Temp failure to compile

Scheduled Pinned Locked Moved Troubleshooting
23 Posts 11 Posters 7.9k Views 10 Watching
  • 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.
  • tbowmoT Offline
    tbowmoT Offline
    tbowmo
    Admin
    wrote on last edited by
    #21

    @tlpeter

    Ok, so you are using a newer version of the DallasTemperature library, than the one that we have supplied in our MySensorsArduinoExamples. And yes, it seems that they have made millisToWaitForConversion() a private function there. Or someone has changed the library we included in our repository at some point in time, so it is made public.

    There are 2 options, as I see it :)

    1. Use the library that we have supplied in MySensorsArduinoExamples
    2. Make a PR against the original library, found here, and make the millisToWaitForConversion() public, and get them to release a new version of the library.

    :)

    mfalkviddM 1 Reply Last reply
    0
    • tbowmoT tbowmo

      @tlpeter

      Ok, so you are using a newer version of the DallasTemperature library, than the one that we have supplied in our MySensorsArduinoExamples. And yes, it seems that they have made millisToWaitForConversion() a private function there. Or someone has changed the library we included in our repository at some point in time, so it is made public.

      There are 2 options, as I see it :)

      1. Use the library that we have supplied in MySensorsArduinoExamples
      2. Make a PR against the original library, found here, and make the millisToWaitForConversion() public, and get them to release a new version of the library.

      :)

      mfalkviddM Offline
      mfalkviddM Offline
      mfalkvidd
      Mod
      wrote on last edited by
      #22

      The millisToWaitForConversion has always been private in milesburton's library, so the answer is the latter: someone has changed the library we included in our repository at some point in time.

      I found the answer in git:

      commit e47e596075282b122ac5d266ec6ad5c60ca8c978
      Author: Robo Print <roboprint@users.noreply.github.com>
      Date:   Sat Jun 27 23:00:37 2015 +0200
      
          Make DallasTemperature::millisToWaitForConversion() publicly available
      

      I agree with tbowmo's options.

      1 Reply Last reply
      0
      • R Offline
        R Offline
        robosensor
        wrote on last edited by
        #23

        It's my code :scream:

        I changed this method from private to public, this function is used to determine wait time needed for right non-blocking wait call. Seems like I should make PR to original library instead of modifying MYS copy.

        1 Reply Last reply
        4
        Reply
        • Reply as topic
        Log in to reply
        • Oldest to Newest
        • Newest to Oldest
        • Most Votes


        13

        Online

        11.7k

        Users

        11.2k

        Topics

        113.1k

        Posts


        Copyright 2025 TBD   |   Forum Guidelines   |   Privacy Policy   |   Terms of Service
        • Login

        • Don't have an account? Register

        • Login or register to search.
        • First post
          Last post
        0
        • MySensors
        • OpenHardware.io
        • Categories
        • Recent
        • Tags
        • Popular