Skip to content

TOUTL new sensor - #2853

Merged
rumachan merged 5 commits into
mainfrom
toutl_newsensor
Oct 13, 2026
Merged

rumachan merged 5 commits into
mainfrom
toutl_newsensor

Conversation

@rumachan

@rumachan rumachan commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

A new sensor added after a period without one connected. Old one was removed several months ago, and recorded then.

@rumachan
rumachan requested review from elidana and gva6500 October 2, 2026 01:57
@rumachan
rumachan requested review from a team and ozym as code owners October 2, 2026 01:57
gva6500
gva6500 previously approved these changes Oct 2, 2026

@elidana elidana left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @rumachan , thanks for that. I might have things mixed up, but based on the comments from @ricardojohansson in the field logs, I wonder if something need to be changed here?

  • the serial number for the temperature sensor should be 415554?
  • the datalogger code was also updated, does it require a firmware file change to reflect this?

@rumachan

rumachan commented Oct 4, 2026

Copy link
Copy Markdown
Contributor Author

Yes you are right about the serial number @elidana . The field notes say

A new 40m 107 temperature probe was installed. Adapt number of the new sensor is 415554 and, as these sensors don't come with a serial number, the Adapt number is also to be considered the serial number.

but the equipment change information at the top of the ticket was not changed to reflect that.

There is no mention of a firmware upgrade; it is not the same as a code update. According to Delta, firmware is

Campbell Scientific,CR6,20475,Upper Tama Lake,,2023-07-24T00:09:59Z,9999-01-01T00:00:00Z
Campbell Scientific,CR6,20475,CR6.Std.12.01,2023-07-24T00:09:59Z,9999-01-01T00:00:00Z,

and according today's data file
"TOA5","20475","CR6","20475","CR6.Std.12.01","CPU:TOUTL-uppertamalake-v009.CR6","42137","Data_Table"
So there has been no change.

I will make the required change to the serial number.

@ozym
ozym force-pushed the toutl_newsensor branch from 8857029 to 5468125 Compare October 9, 2026 07:16

@elidana elidana left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks for fixing this @rumachan !

@rumachan
rumachan merged commit b921be3 into main Oct 13, 2026
18 checks passed
@rumachan
rumachan deleted the toutl_newsensor branch October 13, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants