fwcommon

Clone Tools

Stats

Commits this week: 2
Total Committers: 13
Last Commit: 04 Jun
Commits this week: 2

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 08 May 2023
9:06 am

Sean Nash

Is this needed? Why would a worn cartridge fail pressure test?

Is this needed? Why would a worn cartridge fail pressure test?

9:03 am

Sean Nash

Why isn't this just an else? And if else should be swapped so normal action (alarm) comes first with if and the else should be the exception (else).

Why isn't this just an else? And if else should be swapped so normal action (alarm) comes first with if and the else should be the exception (else).

9:02 am

Sean Nash

I don't think a worn cartridge should fail here (should still be dry). I think only wet cartridge should be checked here.

I don't think a worn cartridge should fail here (should still be dry). I think only wet cartridge should be checked here.

8:58 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

8:52 am

Sean Nash

Recommend being more direct in naming: TEST_CONFIG_USE_WET_CARTRIDGE TEST_CONFIG_USE_WORN_CARTRIDGE

Recommend being more direct in naming:
TEST_CONFIG_USE_WET_CARTRIDGE
TEST_CONFIG_USE_WORN_CARTRIDGE

8:48 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Thursday 04 May 2023
3:21 pm

wbracken  (deleted user)

10:39 am

Dara Navaei

Please see changes.

Please see changes.

10:35 am

Dara Navaei

Done

Done

8:29 am

Sean Nash

This looks like 104.

This looks like 104.

8:28 am

Sean Nash

Probably not needed, but for good measure can we do this in transition to GenIdle mode too?

Probably not needed, but for good measure can we do this in transition to GenIdle mode too?

8:24 am

Sean Nash

Add blank line before return statement.

Add blank line before return statement.

8:23 am

Sean Nash

This function could, in theory, be used to assign a CD2 cal table to CPo (for example).

This function could, in theory, be used to assign a CD2 cal table to CPo (for example).

Wednesday 03 May 2023
3:20 pm

wbracken  (deleted user)

Is #if temporary?

Is #if temporary?

3:18 pm

wbracken  (deleted user)

Are these #if temporary?

Are these #if temporary?

3:15 pm

wbracken  (deleted user)

Looks like merge issues not resolved.

Looks like merge issues not resolved.

3:13 pm

wbracken  (deleted user)

Is the merge correct?

Is the merge correct?

2:02 pm

Dara Navaei

HD-DEN-14892_Add Bootloader TO Existing DG Firmware
HD-DEN-14892_Add Bootloader TO Existing DG Firmware
Tuesday 02 May 2023
5:06 pm

Sean Nash

Let's discuss how test config is being used in this case. It seems like it's all over the place to me.

Let's discuss how test config is being used in this case. It seems like it's all over the place to me.

4:52 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

1:21 pm

wbracken  (deleted user)

Update header.

Update header.

12:01 pm

Dara Navaei

DG-DEN-15246_Active Reservoir Empty Post Concentrate Alarm Recovery
DG-DEN-15246_Active Reservoir Empty Post Concentrate Alarm Recovery
Monday 01 May 2023
2:59 pm

dcox  (deleted user)

11:46 am

Sean Nash

11:45 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Thursday 27 Apr 2023
4:38 pm

dcox  (deleted user)

Changed to 10 along with the max windowed timer count. DVT-005 was failing with a count of 5-7. It is passing now with max of 10.

Changed to 10 along with the max windowed timer count. DVT-005 was failing with a count of 5-7. It is passing now with max of 10.

2:20 pm

wbracken  (deleted user) and jtaylor  (deleted user)

10:02 am

Dara Navaei

HD-DEN-15199_FW DN Sprint 95 Part 2
HD-DEN-15199_FW DN Sprint 95 Part 2
Wednesday 26 Apr 2023
6:47 pm

Sean Nash

This appears to be causing problems 1) at startup and 2) after fast operations (e.g. retract, preload, seek). For #1, not sure what's happening at startup, but apparently the position reported by F...

This appears to be causing problems 1) at startup and 2) after fast operations (e.g. retract, preload, seek).
For #1, not sure what's happening at startup, but apparently the position reported by FPGA is not stable for a little while after startup.
Probably need to exempt first second (previous 1sec persistence seemed to work fine) after powerup to prevent false alarms.
For #2, I think it just takes a little while for pump to actually come to full stop after pump was running fast.
The syringe pump monitor and controller are running in priority task at 100 Hz, so a count of 5 is only 50 ms. Need to play around with this count to see how much time is needed to come to complete stop and then set count to something north of that time (at 10ms per count) for margin.

3:57 pm

Dara Navaei