Fisheye on hdfirmware/vectorcast/xml_data

hdfirmware (DEN-15436-dpi-tubing-comes-out-of-pump-track-during-dpi-reversal-in-post-treatment): DEN-431 Finished unit testing. Ran the test suite and added a .gitignore the results folder to ignore all the VectorCAST...

Dara Navaei — Sun, 05 Jan 2020 15:50:27 -0800

Dara Navaei committed 609937a on branch master, DEN-10153_Treatment_Log_Prescribed_UF and 421 more in hdfirmware

DEN-431 Finished unit testing. Ran the test suite and added a .gitignore the results folder to ignore all the VectorCAST .html files

· /firmware/App/Controllers/RTC.c

(-34, +26)

| History

| Source

| Diff

· /firmware/App/Tasks/TaskGeneral.c

(-2, +2)

| History

| Source

| Diff

· /results/.gitignore

(-0, +1)

| History

| Source

· /results/VectorCAST.log

(-0, +10143)

| History

| Source

| Diff

· /results/cppcheck.log

(-0, +84)

| History

| Source

| Diff

· /results/cppcheckError.csv

(-0, +44)

| History

| Source

· /vectorcast/Hercules_RM46_HD_Project.vcm

(-0, +8)

| History

| Source

| Diff

· /vectorcast/Hercules_RM46_HD_Project/environment/INT_RTC/INT_RTC.cfg

(-0, +0)

| History

| Source

· /vectorcast/Hercules_RM46_HD_Project/environment/INT_RTC/INT_RTC.env

(-0, +11)

| History

| Source

· /vectorcast/Hercules_RM46_HD_Project/environment/INT_RTC/INT_RTC.mfg

(-0, +7)

| History

| Source

· /vectorcast/Hercules_RM46_HD_Project/environment/INT_RTC/INT_RTC.tst

(-0, +54)

| History

| Source

· /vectorcast/Hercules_RM46_HD_Project/environment/RTC/RTC.env

(-0, +24)

| History

| Source

| Diff

· /vectorcast/Hercules_RM46_HD_Project/environment/RTC/RTC.tst

(-18, +808)

| History

| Source

| Diff

· /vectorcast/Hercules_RM46_HD_Project/project.db

(binary)

| History

| Source

| Diff

· /vectorcast/target/clover/coverage_results_Hercules_RM46_HD_Project_combined.xml

(-203, +2443)

| History

| Source

| Diff

… 25 more files in changeset.

HD-DEN-759-1: closed by Dara Navaei

Dara Navaei — Fri, 13 Dec 2019 12:39:00 -0800

Dara Navaei Closed HD-DEN-759-1

HD-DEN-759-1: reviewers finished

Fri, 13 Dec 2019 10:24:32 -0800

lbaloa (deleted user) , Behrouz NematiPour and Dara Navaei finished reviewing HD-DEN-759-1

HD-DEN-759_BloodFlow

HD-DEN-759-1: reviewers finished

Thu, 12 Dec 2019 15:47:13 -0800

pmontazemi (deleted user) finished reviewing HD-DEN-759-1

HD-DEN-759_BloodFlow

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Thu, 12 Dec 2019 10:07:49 -0800

pmontazemi (deleted user) replied to pmontazemi (deleted user) in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Thu, 12 Dec 2019 10:05:49 -0800

pmontazemi (deleted user) replied to pmontazemi (deleted user) in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "Spaces between 0s."

pmontazemi — Wed, 11 Dec 2019 14:16:03 -0800

pmontazemi (deleted user) commented on HD-DEN-759-1

Spaces between 0s.

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Tue, 10 Dec 2019 16:44:17 -0800

pmontazemi (deleted user) replied to Sean Nash in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "In group setting, decision was made to use the ..."

pmontazemi — Tue, 10 Dec 2019 16:43:01 -0800

pmontazemi (deleted user) replied to pmontazemi (deleted user) in HD-DEN-759-1

In group setting, decision was made to use the () rules applied to [].

hdfirmware (DEN-15436-dpi-tubing-comes-out-of-pump-track-during-dpi-reversal-in-post-treatment): Ran the test suite. Added test suite results folder.

Sean Nash — Fri, 06 Dec 2019 18:38:43 -0800

Sean Nash committed 38f2656 on branch master, DEN-10153_Treatment_Log_Prescribed_UF and 425 more in hdfirmware

Ran the test suite. Added test suite results folder.

· /results/VectorCAST.log

(-0, +12306)

| History

| Source

· /results/cppcheck.log

(-0, +122)

| History

| Source

· /results/cppcheckError.log

(-0, +569)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_ALARMLAMP_execution_results_report.html

(-0, +5313)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_ALARMMGMT_execution_results_report.html

(-0, +12107)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_BLOODFLOW_execution_results_report.html

(-0, +16851)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_BUTTONS_execution_results_report.html

(-0, +8833)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_COMMBUFFERS_execution_results_report.html

(-0, +5105)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_COMM_execution_results_report.html

(-0, +3635)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_CPLD_execution_results_report.html

(-0, +2073)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_FPGA_execution_results_report.html

(-0, +13551)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_INTERRUPTS_execution_results_report.html

(-0, +3449)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_INT_ALARMLAMP_execution_results_report.html

(-0, +1735)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_INT_ALARMMGMT_execution_results_report.html

(-0, +1297)

| History

| Source

· /results/execution/VectorCAST_MinGW_C_LinuxTestSuite_INT_BLOODFLOW_execution_results_report.html

(-0, +1101)

| History

| Source

… 76 more files in changeset.

HD-DEN-759-1: comment by lbaloa "RESOLVED IN CODE WALKTHROUGH"

lbaloa — Fri, 06 Dec 2019 10:12:49 -0800

lbaloa (deleted user) replied to Sean Nash in HD-DEN-759-1

RESOLVED IN CODE WALKTHROUGH

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Fri, 06 Dec 2019 10:10:41 -0800

pmontazemi (deleted user) replied to Sean Nash in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Fri, 06 Dec 2019 10:08:14 -0800

pmontazemi (deleted user) replied to Sean Nash in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Fri, 06 Dec 2019 10:07:21 -0800

pmontazemi (deleted user) replied to Sean Nash in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Fri, 06 Dec 2019 10:06:39 -0800

pmontazemi (deleted user) replied to Sean Nash in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Fri, 06 Dec 2019 10:06:18 -0800

pmontazemi (deleted user) replied to Sean Nash in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "RESOLVED in CODE WALKTHROUGH."

pmontazemi — Fri, 06 Dec 2019 10:05:38 -0800

pmontazemi (deleted user) replied to Sean Nash in HD-DEN-759-1

RESOLVED in CODE WALKTHROUGH.

HD-DEN-759-1: comment by pmontazemi "What is the rule for arguments between [ ] ? D..."

pmontazemi — Fri, 06 Dec 2019 08:33:55 -0800

pmontazemi (deleted user) commented on HD-DEN-759-1

What is the rule for arguments between [ ] ? Does it need spacing like ( ) ?

HD-DEN-759-1: comment by Sean Nash "Done."

Sean Nash — Fri, 06 Dec 2019 07:45:41 -0800

Sean Nash replied to pmontazemi (deleted user) in HD-DEN-759-1

Done.

HD-DEN-759-1: comment by Sean Nash "Agreed."

Sean Nash — Fri, 06 Dec 2019 07:44:21 -0800

Sean Nash replied to pmontazemi (deleted user) in HD-DEN-759-1

Agreed.

HD-DEN-759-1: comment by Sean Nash "Fixed."

Sean Nash — Thu, 05 Dec 2019 08:45:23 -0800

Sean Nash replied to pmontazemi (deleted user) in HD-DEN-759-1

Fixed.

HD-DEN-759-1: comment by Sean Nash "Sorry - must have changed a different line of c..."

Sean Nash — Thu, 05 Dec 2019 08:43:43 -0800

Sean Nash replied to pmontazemi (deleted user) in HD-DEN-759-1

Sorry - must have changed a different line of code. Now done.

HD-DEN-759-1: comment by pmontazemi "Spaces between arithmetic sign and arguments, a..."

pmontazemi — Thu, 05 Dec 2019 08:42:47 -0800

pmontazemi (deleted user) commented on HD-DEN-759-1

Spaces between arithmetic sign and arguments, also did you want to add spaces between (v, l, and u)? If so, we need to do it everywhere in this code and everywhere else.

HD-DEN-759-1: comment by pmontazemi "I thought we agreed to have spaced between argu..."

pmontazemi — Thu, 05 Dec 2019 08:36:52 -0800

pmontazemi (deleted user) commented on HD-DEN-759-1

I thought we agreed to have spaced between arguments and parentheses?

HD-DEN-759-1: comment by pmontazemi "I thought we agreed to have spaced between argu..."

pmontazemi — Thu, 05 Dec 2019 08:36:41 -0800

pmontazemi (deleted user) commented on HD-DEN-759-1

I thought we agreed to have spaced between arguments and parentheses?

HD-DEN-759-1: comment by pmontazemi "I thought we agreed to have spaced between argu..."

pmontazemi — Thu, 05 Dec 2019 08:36:33 -0800

pmontazemi (deleted user) commented on HD-DEN-759-1

I thought we agreed to have spaced between arguments and parentheses?

HD-DEN-759-1: comment by Sean Nash "Done."

Sean Nash — Wed, 04 Dec 2019 15:11:44 -0800

Sean Nash replied to pmontazemi (deleted user) in HD-DEN-759-1

Done.

HD-DEN-759-1: comment by Sean Nash "Done."

Sean Nash — Wed, 04 Dec 2019 15:10:58 -0800

Sean Nash replied to pmontazemi (deleted user) in HD-DEN-759-1

Done.

HD-DEN-759-1: comment by Sean Nash "Done."

Sean Nash — Wed, 04 Dec 2019 15:09:22 -0800

Sean Nash replied to pmontazemi (deleted user) in HD-DEN-759-1

Done.

HD-DEN-759-1: comment by Sean Nash "Done. And yes, we do have MIN and MAX macros i..."

Sean Nash — Wed, 04 Dec 2019 14:57:29 -0800

Sean Nash replied to lbaloa (deleted user) in HD-DEN-759-1

Done. And yes, we do have MIN and MAX macros in common.h.