Navigation

    Netgate Discussion Forum
    • Register
    • Login
    • Search
    • Categories
    • Recent
    • Tags
    • Popular
    • Users
    • Search

    Temperature Fahrenheit /Celsius Option and Alarms

    Bounties
    13
    25
    9279
    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
      TonyStone last edited by

      So there is a couple features I would like to see.  In my opinion they should be pretty easy to implement.  I want the option to display the CPU temps in either fahrenheit or celsius.  I would also like to have it trigger a couple types of alarms.  One alarm option should be to send an email and the other should be to start playing a tone through the PC speaker if installed.

      I also noticed that some of the features people have implemented have not been included with upgrades to future versions of pfSense.  I want to have some assurance that this option will not need to be reimplemented on each upgrade or I guess I would want this to be designed as an extra installable package.  I am willing to spend about $75.00 $125.00 on this as of right now.  I am willing to negotiate.

      1 Reply Last reply Reply Quote 0
      • A
        arthurbrownleeiv last edited by

        Bump.

        1 Reply Last reply Reply Quote 0
        • luckman212
          luckman212 LAYER 8 last edited by

          I'd kick in $50 to this too… that brings it up to 175??

          1 Reply Last reply Reply Quote 0
          • F
            Finger79 last edited by

            I'll kick in $20 just for the Fahrenheit option.

            [Edited to Add:  $20 USD sent via PayPal on 12/22/17 after code was successfully merged into pfSense master branch.]

            I looked at the code, but I'm a noob with PHP (looks similar to JavaScript though) and a noob with building pfSense.  The conversion from C to F is simple.  I think what looks challenging (to me, as a noob) is to update the GUI System Preferences (maybe in Advanced?  where would the Celsius/Fahrenheit setting look best?) as well as updating the Dashboard with the new F temperature(s) and all the Temp widgets.  Plus that colored meter goes from 0 - 100 degrees C = 0 - 100% so it'd have to be tuned to Fahrenheit as well….so 32 - 212 F?

            1 Reply Last reply Reply Quote 0
            • D
              doktornotor Banned last edited by

              1 Reply Last reply Reply Quote 0
              • L
                loonylion last edited by

                Is this still wanted? I've put something together but cant find a way to test the package installing. Don't want it released for public consumption without more testing.




                1 Reply Last reply Reply Quote 0
                • T
                  Teken last edited by

                  Very interested in this will this even happen??

                  1 Reply Last reply Reply Quote 0
                  • L
                    loonylion last edited by

                    The OP has never responded to me. The work is done but I've still not found any way to test the install process.

                    1 Reply Last reply Reply Quote 0
                    • F
                      Finger79 last edited by

                      If Fahrenheit is selected, I'd like to only see Fahrenheit in the UI, not both.

                      1 Reply Last reply Reply Quote 0
                      • L
                        loonylion last edited by

                        @Finger79:

                        If Fahrenheit is selected, I'd like to only see Fahrenheit in the UI, not both.

                        That's a heck of a lot more complicated to achieve because the system uses Celsius internally. This is a very minor modification to the UI files that does the conversion live and displays the result

                        1 Reply Last reply Reply Quote 0
                        • M
                          miken32 last edited by

                          https://github.com/pfsense/pfsense/pull/3891

                          This patch adds a preference to the thermal sensors widget, which is also read by the system information widget.

                          Notification is the job of a network monitoring system, not pfSense. (Your pfSense can't email you about high temperatures if its CPU is a puddle of molten slag!)

                          1 Reply Last reply Reply Quote 0
                          • F
                            Finger79 last edited by

                            @miken32:

                            https://github.com/pfsense/pfsense/pull/3891

                            This patch adds a preference to the thermal sensors widget, which is also read by the system information widget.

                            Notification is the job of a network monitoring system, not pfSense. (Your pfSense can't email you about high temperatures if its CPU is a puddle of molten slag!)

                            Looks awesome, sir!  Hope they merge the PR.

                            Was curious, maybe a quick Find & Replace:

                            • getCelciusValue() –> getCelsiusValue()
                            • celc --> cels
                            1 Reply Last reply Reply Quote 0
                            • M
                              miken32 last edited by

                              @Finger79:

                              @miken32:

                              https://github.com/pfsense/pfsense/pull/3891

                              This patch adds a preference to the thermal sensors widget, which is also read by the system information widget.

                              Notification is the job of a network monitoring system, not pfSense. (Your pfSense can't email you about high temperatures if its CPU is a puddle of molten slag!)

                              Looks awesome, sir!  Hope they merge the PR.

                              Was curious, maybe a quick Find & Replace:

                              • getCelciusValue() –> getCelsiusValue()
                              • celc --> cels

                              LOL I was paying too much attention to spelling Fahrenheit correctly that I messed up the easy one!

                              1 Reply Last reply Reply Quote 0
                              • M
                                miken32 last edited by

                                And since this is the bounty forum, if any of the original requesters want to step up, it would be accepted gratefully.

                                https://www.paypal.me/miken32

                                or if you're Canadian, email money transfer works better; my username at eire.ca.

                                1 Reply Last reply Reply Quote 0
                                • M
                                  miken32 last edited by

                                  @Finger79:

                                  Looks awesome, sir!  Hope they merge the PR.

                                  Until then, here's how to apply it. The changed files are saved with an "orig" suffix:

                                  curl https://patch-diff.githubusercontent.com/raw/pfsense/pfsense/pull/3891.diff | patch -b -d / -p 2 -u
                                  
                                  1 Reply Last reply Reply Quote 0
                                  • L
                                    loonylion last edited by

                                    @miken32:

                                    And since this is the bounty forum, if any of the original requesters want to step up, it would be accepted gratefully.

                                    https://www.paypal.me/miken32

                                    or if you're Canadian, email money transfer works better; my username at eire.ca.

                                    I did the work before you arrived and nobody ever responded to me. I suspect the less than friendly responses to the original request put people off.

                                    1 Reply Last reply Reply Quote 0
                                    • M
                                      miken32 last edited by

                                      @loonylion:

                                      @miken32:

                                      And since this is the bounty forum, if any of the original requesters want to step up, it would be accepted gratefully.

                                      https://www.paypal.me/miken32

                                      or if you're Canadian, email money transfer works better; my username at eire.ca.

                                      I did the work before you arrived and nobody ever responded to me. I suspect the less than friendly responses to the original request put people off.

                                      I wouldn't call it "less than friendly" just someone pointing out the likely reason that this work wasn't done as part of the original product (i.e. nobody uses Fahrenheit except Americans; though it's also used in Belize and Bahamas as well as a few other small island nations.)

                                      I think it is a fairly common (though certainly not universal) pattern on this board. Random people show up and make requests, someone does the work and then the OP is never heard from again. I'm not in it for the money anyway, else I wouldn't have posted my code on Github. Just wanted to contribute to the project that helps me earn a living. If someone finds enough value in it to reward me personally, so much the better!

                                      F 1 Reply Last reply Reply Quote 0
                                      • V
                                        Visseroth last edited by

                                        Just noticed this thread.
                                        I'm game. I'd be interested in throwing in another $20 to see this implemented. It's critical to know that your equipment isn't cooking.

                                        1 Reply Last reply Reply Quote 0
                                        • F
                                          Finger79 last edited by

                                          @Visseroth:

                                          Just noticed this thread.
                                          I'm game. I'd be interested in throwing in another $20 to see this implemented. It's critical to know that your equipment isn't cooking.

                                          The pull request got merged back in December 2017.  I'm seeing the new code in 2.4.3 Development branch.  I assume this will also hit 2.4.3 Release once it comes out.

                                          1 Reply Last reply Reply Quote 0
                                          • V
                                            Visseroth last edited by

                                            Nice! Well thank you for the update. I think it'll be a handy addition to PfSense. Getting alerts your stuff is hot is just as important as getting alerts that your stuff is crashing.

                                            1 Reply Last reply Reply Quote 0
                                            • M
                                              miken32 last edited by

                                              @Visseroth:

                                              Nice! Well thank you for the update. I think it'll be a handy addition to PfSense. Getting alerts your stuff is hot is just as important as getting alerts that your stuff is crashing.

                                              My patch only adds the option for changing the unit of measurement displayed. Adding alerting is beyond the scope of what a firewall is supposed to do; that's what network monitoring systems are for!

                                              1 Reply Last reply Reply Quote 0
                                              • F
                                                fibrewire @miken32 last edited by fibrewire

                                                After reading GitHub, I read that the option is changed on the Dashboard under "Thermal Sensors" widgit while before reading this I was wasting time looking at "System Information" widgit. Thank YOU!!!

                                                @miken32 Here's a different map to look at when deciding to use Fahrenheit.

                                                alt text

                                                1 Reply Last reply Reply Quote 3
                                                • P
                                                  pinknemo13 last edited by pinknemo13

                                                  Bump.

                                                  1 Reply Last reply Reply Quote 0
                                                  • J
                                                    jclemons7 last edited by

                                                    Bump

                                                    1 Reply Last reply Reply Quote 0
                                                    • A
                                                      apnsettings Banned last edited by

                                                      This post is deleted!
                                                      1 Reply Last reply Reply Quote 0
                                                      • First post
                                                        Last post