application

Clone Tools

Stats

Commits this week: 6
Total Committers: 14
Last Commit: 23 Mar
Commits this week: 6

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 01 Feb 2021
4:45 pm

Behrouz NematiPour

DEN-6621: Code Review 0

Options
    • -1
    • +1
    /sources/storage/Logger.h
4:42 pm

pmontazemi  (deleted user)

16GB and 2.4GB (units are Bytes).

16GB and 2.4GB (units are Bytes).

4:41 pm

pmontazemi  (deleted user)

RESOLVED.

RESOLVED.

4:31 pm

Behrouz NematiPour

I meant "retains" and "retain". A complete sentence would be : "if 90% retains ...." or "if 90% shall retain" I prefer to have a present verb, for input and if it was output, a past verb.

I meant "retains" and "retain".
A complete sentence would be :
"if 90% retains ...." or
"if 90% shall retain"
I prefer to have a present verb, for input and if it was output, a past verb.

4:27 pm

Behrouz NematiPour

Corrected the sentences and updated "can't" to "cannot" everywhere in the file.

Corrected the sentences and updated "can't" to "cannot" everywhere in the file.

4:27 pm

Behrouz NematiPour

Corrected the sentences and updated "can't" to "cannot" everywhere in the file.

Corrected the sentences and updated "can't" to "cannot" everywhere in the file.

4:25 pm

Behrouz NematiPour

DEN-6621: Code Review 0

Options
    • -14
    • +14
    /sources/storage/FileHandler.cpp
3:48 pm

Behrouz NematiPour

It is noted. Since this is part of the Arterial/Venous pressure feature and not the preTx_Uf, will be taking care of it as part of the code clean-up and integration I need to do later. By the way, ...

It is noted.
Since this is part of the Arterial/Venous pressure feature and not the preTx_Uf, will be taking care of it as part of the code clean-up and integration I need to do later.
By the way, it's not my solution and not even my approved one.

3:47 pm

Behrouz NematiPour

See the reply above.

See the reply above.

2:32 pm

Behrouz NematiPour

if it is a constant/read-only value that will be used mostly in qml, then better to be defined in QML unless otherwise required.

if it is a constant/read-only value that will be used mostly in qml, then better to be defined in QML unless otherwise required.

2:29 pm

Behrouz NematiPour

Removed.

Removed.

2:26 pm

Behrouz NematiPour

DEN-5759: prTx: Code Review 0

Options
    • -1
    • +0
    /sources/gui/qml/components/ConfirmTreatmentSubTable.qml
2:26 pm

pmontazemi  (deleted user)

Did you mean retained? If so, change all these to retained as well as hte name of the quint8 variable to vRetainedPercent.

Did you mean retained? If so, change all these to retained as well as hte name of the quint8 variable to vRetainedPercent.

2:25 pm

plucia  (deleted user)

Your solution requires a code change to be made and the entire denali application to have to be rebuilt every time a treatment parameter needs to be adjusted by systems. This will not serve the nee...

Your solution requires a code change to be made and the entire denali application to have to be rebuilt every time a treatment parameter needs to be adjusted by systems. This will not serve the needs of the system team. To fulfill their needs it was requested at the time to put the values in a file. You and I agreed at the time that the treatment parameter ranges should be read from a file. These two treatment parameter ranges are being read from QML which contradicts our agreed-upon decision to have them read from a file.

2:24 pm

pmontazemi  (deleted user)

"File cannot be opened for read (%1). Possible corrupted file system." 1. Rephrase and spell cannot as mentioned. 2. Add space between two sentences. 3. Add "." at end of 2nd sentence.

"File cannot be opened for read (%1). Possible corrupted file system."

1. Rephrase and spell cannot as mentioned.
2. Add space between two sentences.
3. Add "." at end of 2nd sentence.

2:20 pm

pmontazemi  (deleted user)

cannot

cannot

2:17 pm

Behrouz NematiPour

Actually, this feature (Arterial/Venous Pressure) was done before Pre-Treatment Create feature. At the moment Pre-Treatment Create has been implemented existence of these parameters has been ignore...

Actually, this feature (Arterial/Venous Pressure) was done before Pre-Treatment Create feature.
At the moment Pre-Treatment Create has been implemented existence of these parameters has been ignored and another approach has been followed.
In general, If a simple value can be defined in where it is being used the most (in this case qml) it's better to be defined there and not to call multiple setter, getter for getting a simple read only value.

2:15 pm

pmontazemi  (deleted user)

RESOLVED.

RESOLVED.

2:15 pm

pmontazemi  (deleted user)

RESOLVED.

RESOLVED.

2:14 pm

pmontazemi  (deleted user)

RESOLVED.

RESOLVED.

2:14 pm

pmontazemi  (deleted user)

RESOLVED.

RESOLVED.

2:14 pm

pmontazemi  (deleted user)

RESOLVED.

RESOLVED.

2:10 pm

Behrouz NematiPour

UI_DVT related.

UI_DVT related.

2:10 pm

Behrouz NematiPour

UI_DVT related.

UI_DVT related.

2:09 pm

Behrouz NematiPour

UI_DVT related.

UI_DVT related.

2:09 pm

Behrouz NematiPour

UI_DVT related.

UI_DVT related.

2:09 pm

Behrouz NematiPour

UI_DVT related.

UI_DVT related.

2:09 pm

Behrouz NematiPour

UI_DVT related.

UI_DVT related.

2:09 pm

Behrouz NematiPour

UI_DVT related.

UI_DVT related.

2:08 pm

Behrouz NematiPour

UI_DVT related.

UI_DVT related.