sources

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-7054: Addressed code review comments

  1. … 3 more files in changeset.
Clarify comment..."...and will a response..." Not clear on what that means.

Clarify comment..."...and will a response..." Not clear on what that means.

...cannot...

...cannot...

Note...will not (instead of won't informal language)...for more than...

Note...will not (instead of won't informal language)...for more than...

...each instruction...

...each instruction...

...instruction-based...

...instruction-based...

...that has...for each instruction-based...

...that has...for each instruction-based...

, the application

, the application

When...VM, (comma after VM)...the application

When...VM, (comma after VM)...the application

This...required...application...instruction-based.

This...required...application...instruction-based.

It..., for application...

It..., for application...

DEN-7147: Fixed a bug on arterial/venous pressures

- which was causing user to move low/high other than current default to be able to confirm for FW validation.

  1. … 1 more file in changeset.
DEN-7147: Fixed a bug on Sub-StepIndicator on bottom of the instructions

- Which was causing that to not to show correct number of the steps and titles.

- removed #include "MModel.h" in all the possible cases and include the specific model since the MModel has too much include.

- Showing the Heparin setup in the CART_INSTALL state only if we have a heparin volume setup.

- created a common template function name enumString in Gui namespace to get the enum name as string.

    • -4
    • +11
    ./gui/qml/components/StepIndicator.qml
  1. … 58 more files in changeset.
DEN-7147: preTx: Code Review 0

- The little count down progress circles become bigger.

- The completed text on the bottom of some screens with check marks become settable by code.

- Duplicate group removed from instruction file

- "Recirculate ..." text has been removed from the priming screen

- Added syringe pump state

    • -0
    • +1
    ./view/hd/data/VPreTreatmentStatesData.cpp
    • -0
    • +1
    ./view/hd/data/VPreTreatmentStatesData.h
  1. … 2 more files in changeset.
DEN-7137: preTx: Update Design Documents

- Filled all the screen with the dynamic instruction so be able to capture for SDD.

- Removed the idea of reset and clear the stack, since it is not working.

  1. … 1 more file in changeset.
UI-DEN-7135_UI Pre Treatment
UI-DEN-7135_UI Pre Treatment
Why Network being registered?

Why Network being registered?

Please take a closer look at the code. At this moment you are still in the main thread and not in the Wifi Thread.

Please take a closer look at the code.
At this moment you are still in the main thread and not in the Wifi Thread.

This is an exception that you have a view which you don't have access to its object but in general: whoever wants to use it to connect to a signal of another class should connect to that signal in ...

This is an exception that you have a view which you don't have access to its object but in general:
whoever wants to use it to connect to a signal of another class should connect to that signal in its own scop/class and do it with its private slots.
So in a case like this, these connections should be moved to the interface.
But as I said it is an exception and there no better way than what has been done here.

If you made sure your threading is fine please remove these debug which will fill up the log so fast. Please remove the other ones as well if not needed.

If you made sure your threading is fine please remove these debug which will fill up the log so fast.
Please remove the other ones as well if not needed.

This one might be on_ not do_. and should be private! A little confusing that if the signal comes from the interface how come the doAdd.... is public access? What happens if it has been called on t...

This one might be on_ not do_. and should be private!
A little confusing that if the signal comes from the interface how come the doAdd.... is public access?
What happens if it has been called on the qml side.
Could you please explain?

This screen is a child of ScreenItem and not a StackItem. How come the pop() function is working here! Which Push, pop methods is being called here? If that's the one from SettingsStack, would be m...

This screen is a child of ScreenItem and not a StackItem.
How come the pop() function is working here!
Which Push, pop methods is being called here?
If that's the one from SettingsStack, would be more cleaner to move the push and pop functions in the SettingsStack.
It's fine for now but keep that in mind for later.

Please Just make sure that the Alarm can show up and cover any dialog also.

Please Just make sure that the Alarm can show up and cover any dialog also.

Would be better to use a non-profit website for the internet test.

Would be better to use a non-profit website for the internet test.

This line caused some issues, we need to remove the log of the application output for now. please change it to by adding & and # before 2>>: $HOME/denali -u & # 2>> $HOME/filesystem.err &

This line caused some issues,
we need to remove the log of the application output for now.
please change it to by adding & and # before 2>>:
$HOME/denali -u & # 2>> $HOME/filesystem.err &

DEN-7377: preTx: Implementation [Dynamic Instruction Loading] ( 2 )

DEN-7457: Meetings & Support - General purpose

    • -36
    • +47
    ./gui/qml/compounds/InstructionView.qml
  1. … 37 more files in changeset.
DEN-7521: Ensure wifi reconnects on reboot

  1. … 5 more files in changeset.
DEN-7521: Adding rest of support for static IP assignments.

    • -0
    • +103
    ./gui/qml/dialogs/WifiJoinDisconnect.qml
    • -10
    • +28
    ./gui/qml/pages/SettingsWifi.qml
  1. … 9 more files in changeset.
DEN-7408: Code improvement for navigation

- VTreatmentCreate is cleaned up. (still needs a lot of work)

    • -1
    • +1
    ./gui/qml/pages/treatment/TreatmentStack.qml
    • -0
    • +577
    ./view/VTreatmentCreate (legacy).cpp
    • -0
    • +258
    ./view/VTreatmentCreate (legacy).h
  1. … 2 more files in changeset.