hdfirmware / firmware

App

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
Last Commit: 19 Nov
Commits this week: 0

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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 30 Jun 2022
2:47 pm

Darren Cox

No. Removed.

No. Removed.

2:47 pm

Darren Cox

updated.

updated.

2:46 pm

Darren Cox

Added index variable and removed duplication of code.

Added index variable and removed duplication of code.

2:44 pm

Darren Cox

No. Removed.

No. Removed.

2:44 pm

Darren Cox

Update

Update

2:43 pm

Darren Cox

Updated

Updated

2:18 pm

Darren Cox

Per conversation with Sean: It is a DG Fault generated that the HD is identifying.

Per conversation with Sean: It is a DG Fault generated that the HD is identifying.

2:13 pm

Darren Cox

Leave as is per conversation w Sean. Reported by HD, but actually a DG generated problem.

Leave as is per conversation w Sean. Reported by HD, but actually a DG generated problem.

12:45 pm

wbracken  (deleted user)

General comment: Should floating point values be of the format N.NF? What is the coding standard? My experience has been that it's best to always specify the "F".

General comment: Should floating point values be of the format N.NF? What is the coding standard? My experience has been that it's best to always specify the "F".

10:09 am

Dara Navaei

10:02 am

Sean Nash

This has the rank of a fault and I think it should be an HD fault - so set the T/F properties same as the alarm above (FPGA clock speed check).

This has the rank of a fault and I think it should be an HD fault - so set the T/F properties same as the alarm above (FPGA clock speed check).

9:44 am

Sean Nash

Maybe change param name to final (i.e. final or start).

Maybe change param name to final (i.e. final or start).

9:42 am

Sean Nash

I think we can handle both reservoirs with same code (no need for else). Probably just need to add a macro somewhere that converts a reservoir ID and primary/backup selection into a loadcell ID.

I think we can handle both reservoirs with same code (no need for else). Probably just need to add a macro somewhere that converts a reservoir ID and primary/backup selection into a loadcell ID.

9:39 am

Sean Nash

I think this whole condition should be removed. If we are taking baseline reading for a reservoir, it is not time to do the drift check yet. Just need to save the readings here.

I think this whole condition should be removed. If we are taking baseline reading for a reservoir, it is not time to do the drift check yet. Just need to save the readings here.

9:37 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:37 am

Sean Nash

Remove blank line.

Remove blank line.

9:37 am

Sean Nash

I think this alarm should be an HD fault for now (no recovery).

I think this alarm should be an HD fault for now (no recovery).

9:34 am

Sean Nash

Do an explicit condition here. i.e. if ( TRUE == reservoirFull )

Do an explicit condition here. i.e. if ( TRUE == reservoirFull )

9:27 am

Sean Nash

It's not so much empty and full - neither is empty - it's more like start and final weights. Consider renaming these. Also, consider moving these declarations to top of module. Nothing wrong with t...

It's not so much empty and full - neither is empty - it's more like start and final weights. Consider renaming these.
Also, consider moving these declarations to top of module. Nothing wrong with them being here, but the style for this code base has been to declare everything at top of module.

9:19 am

Sean Nash

Use FALSE, not false.

Use FALSE, not false.

9:18 am

Sean Nash

Use TRUE (not true).

Use TRUE (not true).

9:13 am

Sean Nash

Is fabs() appropriate here?

Is fabs() appropriate here?

9:12 am

Sean Nash

Is fabs() appropriate here?

Is fabs() appropriate here?

Tuesday 28 Jun 2022
10:09 am

Michael Garthwaite

This #define should be at the top of the file.

This #define should be at the top of the file.

Friday 24 Jun 2022
11:39 am

Dara Navaei

HD-DEN-12847_SW Dev Sprint 71 Darren
HD-DEN-12847_SW Dev Sprint 71 Darren
Thursday 16 Jun 2022
5:34 pm

Dara Navaei

5:08 pm

Dara Navaei and Michael Garthwaite

Tuesday 14 Jun 2022
2:23 pm

Dara Navaei

1:30 pm

Michael Garthwaite and Sean Nash

11:09 am

Michael Garthwaite

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.