td-firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch LDT-1903-air-trap-level-control---td---0 into develop

LDT-1903 Air Trap updates

Bamboo Commit: Updated TDCommon.h with build versions from Bamboo.

Merge branch 'develop' of bitbucket.org:diality-cloud/td-firmware into develop

Merge branch LDT-1903-air-trap-level-control---td---0 into develop

LDT-1903 adjusting lowering delay timer after fill complete

Bamboo Commit: Updated TDCommon.h with build versions from Bamboo.

Merge branch LDT-1903-air-trap-level-control---td---0 into develop

    • -13
    • +12
    /firmware/App/Controllers/AirPump.c
LDT-1903 addressed the review comments

LDT-1903 addressed review comments

LDT-1903 addressed the review comments

    • -13
    • +12
    /firmware/App/Controllers/AirPump.c
I don't see why we need the extra precision here. I suspect a 64-bit division is expensive (s/w library implementation instead of co-processor) so I don't like to use doubles unless it is truly nec...

I don't see why we need the extra precision here. I suspect a 64-bit division is expensive (s/w library implementation instead of co-processor) so I don't like to use doubles unless it is truly necessary.

Add blank line before return.

Add blank line before return.

0.0F

0.0F

Use "F64" macro instead of double.

Use "F64" macro instead of double.

Yes , right now its void , so that's why i didn't change that . I will push new code

Yes , right now its void , so that's why i didn't change that . I will push new code

LDT-2004 Airpump RPM implementation

setAirPumpState could return a BOOL (FALSE if range check fails or state invalid) that you can set result to.

setAirPumpState could return a BOOL (FALSE if range check fails or state invalid) that you can set result to.

Merge branch LDT-1903-air-trap-level-control---td---0 into develop

    • -2
    • +11
    /firmware/App/Controllers/AirPump.c
Inputs is none or FPGA.

Inputs is none or FPGA.

LDT-473 added place holder for param range checks with institutional records

done

done

done , but I need to check it here

done , but I need to check it here

done

done

LDT-1903 Adressed the review comments

Yes, without checking the changes in this file, I pushed it. It is OK when I merge the code. There is no option to correct since my local branch shows 28.its auto generated and it will take next ve...

Yes, without checking the changes in this file, I pushed it. It is OK when I merge the code. There is no option to correct since my local branch shows 28.its auto generated and it will take next version during the merge with staging.

LDT-1903 Adressed the new changes for air pump PWM duty cycle for lowering and review comments

If I don't check it here, Dialin get a true for even invalid range. I need to pass the result immediately to user so that user can take the action.

If I don't check it here, Dialin get a true for even invalid range. I need to pass the result immediately to user so that user can take the action.