UI-DEN-4690-2

RESOLVED

RESOLVED

RESOLVED. Fair, not related to story

RESOLVED.

Fair, not related to story

RESOLVED

RESOLVED

RESOLVED.

RESOLVED.

Fair, not related to story. RESOLVED.

Fair, not related to story.

RESOLVED.

did

did

It is all documentations and Doxygenization. If you see otherwise is because of the some file name change and the code review take it as a new file and mark all the file content as to be reviewed. ...

It is all documentations and Doxygenization.
If you see otherwise is because of the some file name change and the code review take it as a new file and mark all the file content as to be reviewed.
Let me know if otherwise.

"*** Off the subject ***" I don't have any plans for that. It was an effort to feel the gap of not having the corresponding FW code at that moment. If it happens again will consider improving the c...

"*** Off the subject ***"
I don't have any plans for that.
It was an effort to feel the gap of not having the corresponding FW code at that moment.
If it happens again will consider improving the code otherwise we need to consolidate with our manager for priorities/planning.

"*** Off the subject ***" AGAIN, I don't understand what is the relevance of the comment to the Story and which part of the code this comment is referring to?

"*** Off the subject ***"
AGAIN,
I don't understand what is the relevance of the comment to the Story and which part of the code this comment is referring to?

This is the test code. Not the API, not the production code. And not even being used in documentation. I don't think we need to put comments for these functions as long as the name is self-descript...

This is the test code.
Not the API, not the production code.
And not even being used in documentation.
I don't think we need to put comments for these functions as long as the name is self-descriptive within context.
As I never asked you to comment on your test functions.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

Good point, I don't know. This is the way Crucible shows them.

Good point, I don't know.
This is the way Crucible shows them.

Good point, I don't know. This is the way Crucible shows them.

Good point, I don't know.
This is the way Crucible shows them.

Good point, I don't know. This is the way Crucible shows them.

Good point, I don't know.
This is the way Crucible shows them.

I kept these files noncapital since they have short one word only files and no need to be PascalCase.

I kept these files noncapital since they have short one word only files and no need to be PascalCase.

This file and all the view file has been renamed to CamelCase but seems Crucible still insist to get a review on them.

This file and all the view file has been renamed to CamelCase but seems Crucible still insist to get a review on them.

Good point, but I don't know. Seems like it's the way Crucible prefers to show the files. It has been removed and even the git log shows that. This commit is the last commit no the ApplicationContr...

Good point, but I don't know.
Seems like it's the way Crucible prefers to show the files.
It has been removed and even the git log shows that.
This commit is the last commit no the ApplicationController and there is no applicationcontroler on that commit anymore.
http://192.168.10.132:8060/changelog/application/?cs=bb74da05f81b82dad3ec844c1feb1135b949f1c2

Better to fix it now then never

Better to fix it now then never

Which of these classes will need to be in the production code? VDGROPump VDGDrainPump VDGPressures VDGReservoir VDGHeaters VDGLoadCellReadings VDGTemperatures VDGValveStates VDGOperationMode VTrea...

Which of these classes will need to be in the production code?

VDGROPump
VDGDrainPump
VDGPressures
VDGReservoir
VDGHeaters
VDGLoadCellReadings
VDGTemperatures
VDGValveStates
VDGOperationMode
VTreatmentBloodFlow
VTreatmentDialysateFlow
VTreatmentUltrafiltration
VTreatmentPressureOcclusion


Are any only for in-development testing / convenience for systems?

The functions are missing docstrings. Function descriptions and parameter types and descriptions, as well as the return type are needed

The functions are missing docstrings.

Function descriptions and parameter types and descriptions, as well as the return type are needed

Please indicate the return type as "None" if a function doesn't return anything

Please indicate the return type as "None" if a function doesn't return anything

How come these files have been renamed but they have not been deleted to show in red?

How come these files have been renamed but they have not been deleted to show in red?

Should these files be renamed following our convention?

Should these files be renamed following our convention?