install_to_venv.sh

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DIALIN-DENBUG-280_Issues With HD Broadcast P1
DIALIN-DENBUG-280_Issues With HD Broadcast P1
DIALIN-DENBUG-280_Issues With HD Broadcast P1B
DIALIN-DENBUG-280_Issues With HD Broadcast P1B
This CR should be closed due to bad branching. Should have been branched out of p1_staging.

This CR should be closed due to bad branching. Should have been branched out of p1_staging.

DIALIN-DIAL-299_Air Trap Observer NO Longer Pub P1
DIALIN-DIAL-299_Air Trap Observer NO Longer Pub P1
RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

The version of crucible we're using has an issue where duplicate files show in the file viewer on the left. If you look at the more recent hemodialysis_device.py file (see the files viewer on the ...

The version of crucible we're using has an issue where duplicate files show in the file viewer on the left.

If you look at the more recent hemodialysis_device.py file (see the files viewer on the left), this message id is removed: http://dvm-linux02:8060/cru/DIALIN-DEN-4856-1#CFR-19219

It is also removed from both msg_defs.py: http://dvm-linux02:8060/cru/DIALIN-DEN-4856-1#CFR-19027 and http://dvm-linux02:8060/cru/DIALIN-DEN-4856-1#CFR-19203

Done

Done

Yes, if it's already there no reason to remove it. I've added it back

Yes, if it's already there no reason to remove it. I've added it back

This message id is not being used.

This message id is not being used.

The file should be renamed to match this.

The file should be renamed to match this.

should we keep the copyright?

should we keep the copyright?

DIALIN-DEN-4856_Sprint_27
DIALIN-DEN-4856_Sprint_27
Good, So please consider cleaning up used libraries periodically. RESOLVED.

Good,
So please consider cleaning up used libraries periodically.

RESOLVED.

RESOLVED

RESOLVED

1 - pip uses requirements.txt to install dependencies. See tools/setup_environment.sh for more detail. 2 - What you're asking about is the offline install capability. I have created a ticket in Jir...

1 - pip uses requirements.txt to install dependencies. See tools/setup_environment.sh for more detail.
2 - What you're asking about is the offline install capability. I have created a ticket in Jira http://dvm-linux02:8080/browse/DIAL-34 to track this new feature request.
3 - The offline install capability addresses this situation, in the (very) unlikely event that pypi hosting can't be relied upon.
4 - No. My understanding is that we do not need to validate these libraries because Dialin is NPSW.
5 - It's fine to have all of these libraries. For example, some of them are dependencies of python-can. Periodically this requirements.txt will be cleaned if a library is no longer being used. For this change-set, I added flake8 so we can run static code analysis. Flake8 is installed along with its dependencies.

I have updated it so it says Dialin now.

I have updated it so it says Dialin now.

I have some questions regarding these libraries. 1 - What tool/code is checking for these requirements? 2 - What if the requirement is not met? (not a correct version or lib not exists) 3 - If the ...

I have some questions regarding these libraries.
1 - What tool/code is checking for these requirements?
2 - What if the requirement is not met? (not a correct version or lib not exists)
3 - If the lib does not exist is it going to be installed or thrown an error?
4 - Don't we need to validate all these libraries?
5 - What is our process of adding more libraries to the list? On the other way is it fine to have all these libs? are all of them being used?

Why this code review starts with "UI_" it's all Dialin. Shouldn't it be started with Dialin?

Why this code review starts with "UI_" it's all Dialin.
Shouldn't it be started with Dialin?