Skip to content

Commit 2b16035

Browse files
HammadTheOneHammadTheOne
authored
Update dependencies and webpack config (#722)
* Update dependencies and webpack config * Change cryptography version * Test with different resource class * Upgrade resource class for all jobs * Added import * Remove environment marker * Skip speck tests * Remove test file * Linting * Updated webpack config and package.json for ES checks * Fix parmed version * Running tests * Update ideogram test wait timer * Add environment marker * Removing environment marker * Re-running tests
1 parent ac0be91 commit 2b16035

File tree

127 files changed

+7060
-8265
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

127 files changed

+7060
-8265
lines changed

Diff for: ‎.circleci/config.yml

+105-102
Original file line numberDiff line numberDiff line change
@@ -5,87 +5,89 @@ jobs:
55
docker:
66
- image: percyio/agent
77
auth:
8-
username: dashautomation
9-
password: $DASH_PAT_DOCKERHUB
8+
username: dashautomation
9+
password: $DASH_PAT_DOCKERHUB
1010
steps:
1111
- run: percy finalize --all
1212

1313
node:
14+
resource_class: medium+
1415
docker:
1516
- image: circleci/python:3.7.5-stretch-node
1617
auth:
17-
username: dashautomation
18-
password: $DASH_PAT_DOCKERHUB
18+
username: dashautomation
19+
password: $DASH_PAT_DOCKERHUB
1920
steps:
2021
- checkout
2122
- run: echo $PYTHON_VERSION > ver.txt
2223
- restore_cache:
23-
key: dep-{{ checksum ".circleci/config.yml" }}-{{ checksum "ver.txt" }}-{{ checksum "dev-requirements.txt" }}
24+
key: dep-{{ checksum ".circleci/config.yml" }}-{{ checksum "ver.txt" }}-{{ checksum "dev-requirements.txt" }}
2425
- run:
25-
name: 🐍 pip dev requirements
26-
command: |
27-
sudo pip install virtualenv --upgrade
28-
python -m venv venv || virtualenv venv && . venv/bin/activate
29-
python -m pip install --upgrade pip
30-
pip install --progress-bar off --no-cache-dir -r dev-requirements.txt
26+
name: 🐍 pip dev requirements
27+
command: |
28+
sudo pip install virtualenv --upgrade
29+
python -m venv venv || virtualenv venv && . venv/bin/activate
30+
python -m pip install --upgrade pip
31+
pip install --progress-bar off --no-cache-dir -r dev-requirements.txt
3132
- save_cache:
32-
key: dep-{{ checksum ".circleci/config.yml" }}-{{ checksum "ver.txt" }}-{{ checksum "dev-requirements.txt" }}
33-
paths:
34-
- venv
33+
key: dep-{{ checksum ".circleci/config.yml" }}-{{ checksum "ver.txt" }}-{{ checksum "dev-requirements.txt" }}
34+
paths:
35+
- venv
3536
- run:
36-
name: Check current version of node
37-
command: node -v
37+
name: Check current version of node
38+
command: node -v
3839
- restore_cache:
39-
key: deps1-{{ .Branch }}-{{ checksum "package-lock.json" }}
40+
key: deps1-{{ .Branch }}-{{ checksum "package-lock.json" }}
4041

4142
- run:
42-
name: Install package.json
43-
command: npm ci
43+
name: Install package.json
44+
command: npm ci
4445

4546
- save_cache:
46-
key: deps1-{{ .Branch }}-{{ checksum "package-lock.json" }}
47-
paths:
48-
- node_modules
47+
key: deps1-{{ .Branch }}-{{ checksum "package-lock.json" }}
48+
paths:
49+
- node_modules
4950

5051
- run:
51-
name: Run eslint
52-
command: ./node_modules/.bin/eslint src
53-
when: always
52+
name: Run eslint
53+
command: ./node_modules/.bin/eslint src
54+
when: always
5455

5556
- run:
56-
name: Install Dash for package build and build package
57-
command: |
58-
python -m venv venv
59-
. venv/bin/activate
60-
pip install --upgrade pip
61-
git clone --depth 1 https://github.com/plotly/dash.git dash-main
62-
cd dash-main && pip install -e .[dev,testing] --progress-bar off && renderer build && cd ../
63-
npm run build
57+
name: Install Dash for package build and build package
58+
command: |
59+
python -m venv venv
60+
. venv/bin/activate
61+
pip install --upgrade pip
62+
git clone --depth 1 https://github.com/plotly/dash.git dash-main
63+
cd dash-main && pip install -e .[dev,testing] --progress-bar off && renderer build && cd ../
64+
npm run build
6465
6566
- run:
66-
name: Install Dash Bio for using dash_bio.utils
67-
command: |
68-
. venv/bin/activate
69-
python setup.py sdist
70-
cd dist
71-
find . -name "*.gz" | xargs pip install --progress-bar off --no-cache-dir --ignore-installed --quiet && cd ..
72-
no_output_timeout: 20m
67+
name: Install Dash Bio for using dash_bio.utils
68+
command: |
69+
. venv/bin/activate
70+
python setup.py sdist
71+
cd dist
72+
find . -name "*.gz" | xargs pip install --progress-bar off --no-cache-dir --ignore-installed --quiet && cd ..
73+
no_output_timeout: 20m
7374

7475
- run:
75-
name: Run unit tests
76-
command: |
77-
. venv/bin/activate
78-
cd tests/unit
79-
python unit_test_data_setup.py
80-
cd ../../
81-
npm run test
76+
name: Run unit tests
77+
command: |
78+
. venv/bin/activate
79+
cd tests/unit
80+
python unit_test_data_setup.py
81+
cd ../../
82+
npm run test
8283
8384
python-3.6: &test-template
85+
resource_class: medium+
8486
docker:
8587
- image: circleci/python:3.6.9-stretch-node-browsers
8688
auth:
87-
username: dashautomation
88-
password: $DASH_PAT_DOCKERHUB
89+
username: dashautomation
90+
password: $DASH_PAT_DOCKERHUB
8991
environment:
9092
PYTHON_VERSION: py36
9193
PERCY_ENABLE: 0
@@ -95,85 +97,86 @@ jobs:
9597
- run: echo $PYVERSION > ver.txt
9698

9799
- restore_cache:
98-
key: deps1-{{ .Branch }}-{{ checksum "dev-requirements.txt" }}-{{ checksum "tests/requirements.txt" }}-{{ checksum "ver.txt" }}-{{ checksum ".circleci/config.yml" }}
100+
key: deps1-{{ .Branch }}-{{ checksum "dev-requirements.txt" }}-{{ checksum "tests/requirements.txt" }}-{{ checksum "ver.txt" }}-{{ checksum ".circleci/config.yml" }}
99101

100102
- run:
101-
name: Create virtualenv
102-
command: |
103-
python3 -m venv venv
103+
name: Create virtualenv
104+
command: |
105+
python3 -m venv venv
104106
105107
- run:
106-
name: Install requirements
107-
command: |
108-
. venv/bin/activate
109-
pip install --upgrade pip
110-
pip install --progress-bar off -r tests/requirements.txt --quiet
111-
no_output_timeout: 20m
108+
name: Install requirements
109+
command: |
110+
. venv/bin/activate
111+
pip install --upgrade pip
112+
pip install --progress-bar off -r tests/requirements.txt --quiet
113+
no_output_timeout: 20m
112114

113115
- save_cache:
114-
key: deps1-{{ .Branch }}-{{ checksum "dev-requirements.txt" }}-{{ checksum "tests/requirements.txt" }}-{{ checksum "ver.txt" }}-{{ checksum ".circleci/config.yml" }}
115-
paths:
116-
- venv
116+
key: deps1-{{ .Branch }}-{{ checksum "dev-requirements.txt" }}-{{ checksum "tests/requirements.txt" }}-{{ checksum "ver.txt" }}-{{ checksum ".circleci/config.yml" }}
117+
paths:
118+
- venv
117119

118120
- run:
119-
name: Install Dash
120-
command: |
121-
. venv/bin/activate
122-
git clone --depth 1 https://github.com/plotly/dash.git dash-main
123-
cd dash-main && pip install -e .[dev,testing] --progress-bar off && renderer build && cd ../
121+
name: Install Dash
122+
command: |
123+
. venv/bin/activate
124+
git clone --depth 1 https://github.com/plotly/dash.git dash-main
125+
cd dash-main && pip install -e .[dev,testing] --progress-bar off && renderer build && cd ../
124126
125127
- run:
126-
name: Run pylint
127-
command: |
128-
. venv/bin/activate
129-
pylint tests/ dash_bio/component_factory/ dash_bio/utils/
130-
when: always
128+
name: Run pylint
129+
command: |
130+
. venv/bin/activate
131+
pylint tests/ dash_bio/component_factory/ dash_bio/utils/
132+
when: always
131133

132134
- run:
133-
name: Run flake8
134-
command: |
135-
. venv/bin/activate
136-
flake8 --max-line-length=100 tests/ dash_bio/component_factory/ dash_bio/utils/
137-
when: always
135+
name: Run flake8
136+
command: |
137+
. venv/bin/activate
138+
flake8 --max-line-length=100 tests/ dash_bio/component_factory/ dash_bio/utils/
139+
when: always
138140

139141
- run:
140-
name: Run integration tests
141-
command: |
142-
. venv/bin/activate
143-
npm ci
144-
npm run build
145-
python setup.py sdist
146-
cd dist
147-
find . -name "*.gz" | xargs pip install --progress-bar off --no-cache-dir --ignore-installed --quiet && cd ..
148-
pytest --nopercyfinalize tests/integration
149-
no_output_timeout: 20m
142+
name: Run integration tests
143+
command: |
144+
. venv/bin/activate
145+
npm ci
146+
npm run build
147+
python setup.py sdist
148+
cd dist
149+
find . -name "*.gz" | xargs pip install --progress-bar off --no-cache-dir --ignore-installed --quiet && cd ..
150+
pytest --nopercyfinalize tests/integration
151+
no_output_timeout: 20m
150152

151153
- run:
152-
name: Run unit tests
153-
command: |
154-
python -m venv venv
155-
. venv/bin/activate
156-
pytest tests/unit
154+
name: Run unit tests
155+
command: |
156+
python -m venv venv
157+
. venv/bin/activate
158+
pytest tests/unit
157159
158160
python-3.7:
159161
<<: *test-template
162+
resource_class: medium+
160163
docker:
161164
- image: circleci/python:3.7.5-stretch-node-browsers
162165
auth:
163-
username: dashautomation
164-
password: $DASH_PAT_DOCKERHUB
166+
username: dashautomation
167+
password: $DASH_PAT_DOCKERHUB
165168
environment:
166169
PYTHON_VERSION: py37
167170
PERCY_ENABLE: 1
168171
PERCY_PARALLEL_TOTAL: -1
169172

170173
workflows:
171-
version: 2
172-
build:
173-
jobs:
174-
- python-3.6
175-
- python-3.7
176-
- node
177-
- percy-finalize:
178-
requires:
174+
version: 2
175+
build:
176+
jobs:
177+
- python-3.6
179178
- python-3.7
179+
- node
180+
- percy-finalize:
181+
requires:
182+
- python-3.7

Diff for: ‎.eslintrc

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
#"flowVersion": "0.53" // Flow version
1414
}
1515
},
16-
"parser": "babel-eslint",
16+
"parser": "@babel/eslint-parser",
1717
"parserOptions": {
1818
"ecmaVersion": 6,
1919
"sourceType": "module",
@@ -109,7 +109,7 @@
109109
"react/no-did-update-set-state": ["error"],
110110
"react/no-direct-mutation-state": ["error"],
111111
"react/no-is-mounted": ["error"],
112-
"react/no-unknown-property": ["error"],
112+
"react/no-unknown-property": ["error", { "ignore": ["eventDatum", "eventdatum", "config", "layout", "tracks"] }],
113113
"react/prefer-es6-class": ["error", "always"],
114114
"react/prop-types": "error",
115115
"valid-jsdoc": ["off"],

Diff for: ‎NAMESPACE

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ export(dashbioOncoPrint)
1414
export(dashbioPileup)
1515
export(dashbioSequenceViewer)
1616
export(dashbioSpeck)
17-
export(dashbioVolcano)
1817
export(dashbioManhattan)
18+
export(dashbioVolcano)
1919
import(manhattanly)
2020

2121
import(dash)

Diff for: ‎R/dashbioAlignmentChart.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioAlignmentChart <- function(id=NULL, colorscale=NULL, conservationcolor=NULL, conservationcolorscale=NULL, conservationmethod=NULL, conservationopacity=NULL, correctgap=NULL, data=NULL, eventDatum=NULL, extension=NULL, gapcolor=NULL, gapcolorscale=NULL, gapopacity=NULL, groupbars=NULL, height=NULL, loading_state=NULL, numtiles=NULL, opacity=NULL, overview=NULL, scrollskip=NULL, sequenceIds=NULL, showconsensus=NULL, showconservation=NULL, showgap=NULL, showid=NULL, showlabel=NULL, textcolor=NULL, textsize=NULL, tickstart=NULL, ticksteps=NULL, tileheight=NULL, tilewidth=NULL, width=NULL) {
45

56
props <- list(id=id, colorscale=colorscale, conservationcolor=conservationcolor, conservationcolorscale=conservationcolorscale, conservationmethod=conservationmethod, conservationopacity=conservationopacity, correctgap=correctgap, data=data, eventDatum=eventDatum, extension=extension, gapcolor=gapcolor, gapcolorscale=gapcolorscale, gapopacity=gapopacity, groupbars=groupbars, height=height, loading_state=loading_state, numtiles=numtiles, opacity=opacity, overview=overview, scrollskip=scrollskip, sequenceIds=sequenceIds, showconsensus=showconsensus, showconservation=showconservation, showgap=showgap, showid=showid, showlabel=showlabel, textcolor=textcolor, textsize=textsize, tickstart=tickstart, ticksteps=ticksteps, tileheight=tileheight, tilewidth=tilewidth, width=width)

Diff for: ‎R/dashbioCircos.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioCircos <- function(id=NULL, config=NULL, enableDownloadSVG=NULL, enableZoomPan=NULL, eventDatum=NULL, layout=NULL, loading_state=NULL, selectEvent=NULL, size=NULL, style=NULL, tracks=NULL) {
45

56
props <- list(id=id, config=config, enableDownloadSVG=enableDownloadSVG, enableZoomPan=enableZoomPan, eventDatum=eventDatum, layout=layout, loading_state=loading_state, selectEvent=selectEvent, size=size, style=style, tracks=tracks)

Diff for: ‎R/dashbioFornaContainer.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioFornaContainer <- function(id=NULL, allowPanningAndZooming=NULL, colorScheme=NULL, customColors=NULL, height=NULL, hoverPattern=NULL, loading_state=NULL, nodeFillColor=NULL, sequences=NULL, width=NULL) {
45

56
props <- list(id=id, allowPanningAndZooming=allowPanningAndZooming, colorScheme=colorScheme, customColors=customColors, height=height, hoverPattern=hoverPattern, loading_state=loading_state, nodeFillColor=nodeFillColor, sequences=sequences, width=width)

Diff for: ‎R/dashbioIdeogram.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioIdeogram <- function(id=NULL, accessToken=NULL, ancestors=NULL, annotationHeight=NULL, annotationTracks=NULL, annotations=NULL, annotationsColor=NULL, annotationsData=NULL, annotationsLayout=NULL, annotationsPath=NULL, assembly=NULL, barWidth=NULL, brush=NULL, brushData=NULL, chrHeight=NULL, chrMargin=NULL, chrWidth=NULL, chromosomeScale=NULL, chromosomes=NULL, className=NULL, container=NULL, dataDir=NULL, demarcateCollinearChromosomes=NULL, filterable=NULL, fullChromosomeLabels=NULL, geometry=NULL, heatmaps=NULL, histogramScaling=NULL, homology=NULL, legend=NULL, loading_state=NULL, organism=NULL, orientation=NULL, perspective=NULL, ploidy=NULL, ploidyDesc=NULL, rangeSet=NULL, resolution=NULL, rotatable=NULL, rotated=NULL, rows=NULL, sex=NULL, showAnnotTooltip=NULL, showBandLabels=NULL, showChromosomeLabels=NULL, showFullyBanded=NULL, showNonNuclearChromosomes=NULL, style=NULL) {
45

56
props <- list(id=id, accessToken=accessToken, ancestors=ancestors, annotationHeight=annotationHeight, annotationTracks=annotationTracks, annotations=annotations, annotationsColor=annotationsColor, annotationsData=annotationsData, annotationsLayout=annotationsLayout, annotationsPath=annotationsPath, assembly=assembly, barWidth=barWidth, brush=brush, brushData=brushData, chrHeight=chrHeight, chrMargin=chrMargin, chrWidth=chrWidth, chromosomeScale=chromosomeScale, chromosomes=chromosomes, className=className, container=container, dataDir=dataDir, demarcateCollinearChromosomes=demarcateCollinearChromosomes, filterable=filterable, fullChromosomeLabels=fullChromosomeLabels, geometry=geometry, heatmaps=heatmaps, histogramScaling=histogramScaling, homology=homology, legend=legend, loading_state=loading_state, organism=organism, orientation=orientation, perspective=perspective, ploidy=ploidy, ploidyDesc=ploidyDesc, rangeSet=rangeSet, resolution=resolution, rotatable=rotatable, rotated=rotated, rows=rows, sex=sex, showAnnotTooltip=showAnnotTooltip, showBandLabels=showBandLabels, showChromosomeLabels=showChromosomeLabels, showFullyBanded=showFullyBanded, showNonNuclearChromosomes=showNonNuclearChromosomes, style=style)

Diff for: ‎R/dashbioIgv.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioIgv <- function(id=NULL, className=NULL, genome=NULL, loading_state=NULL, locus=NULL, minimumBases=NULL, reference=NULL, style=NULL, tracks=NULL) {
45

56
props <- list(id=id, className=className, genome=genome, loading_state=loading_state, locus=locus, minimumBases=minimumBases, reference=reference, style=style, tracks=tracks)

Diff for: ‎R/dashbioJsme.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioJsme <- function(id=NULL, eventSmiles=NULL, height=NULL, loading_state=NULL, options=NULL, smiles=NULL, style=NULL, width=NULL) {
45

56
props <- list(id=id, eventSmiles=eventSmiles, height=height, loading_state=loading_state, options=options, smiles=smiles, style=style, width=width)

Diff for: ‎R/dashbioMolecule2dViewer.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioMolecule2dViewer <- function(id=NULL, height=NULL, loading_state=NULL, modelData=NULL, scrollZoom=NULL, selectedAtomIds=NULL, width=NULL) {
45

56
props <- list(id=id, height=height, loading_state=loading_state, modelData=modelData, scrollZoom=scrollZoom, selectedAtomIds=selectedAtomIds, width=width)

Diff for: ‎R/dashbioMolecule3dViewer.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioMolecule3dViewer <- function(id=NULL, atomLabelsShown=NULL, backgroundColor=NULL, backgroundOpacity=NULL, height=NULL, labels=NULL, loading_state=NULL, modelData=NULL, onChangeSelection=NULL, onRenderNewData=NULL, orbital=NULL, selectedAtomIds=NULL, selectionType=NULL, shapes=NULL, style=NULL, styles=NULL, width=NULL, zoom=NULL, zoomTo=NULL) {
45

56
props <- list(id=id, atomLabelsShown=atomLabelsShown, backgroundColor=backgroundColor, backgroundOpacity=backgroundOpacity, height=height, labels=labels, loading_state=loading_state, modelData=modelData, onChangeSelection=onChangeSelection, onRenderNewData=onRenderNewData, orbital=orbital, selectedAtomIds=selectedAtomIds, selectionType=selectionType, shapes=shapes, style=style, styles=styles, width=width, zoom=zoom, zoomTo=zoomTo)

Diff for: ‎R/dashbioNeedlePlot.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioNeedlePlot <- function(id=NULL, clickData=NULL, domainStyle=NULL, height=NULL, loading_state=NULL, margin=NULL, mutationData=NULL, needleStyle=NULL, rangeSlider=NULL, width=NULL, xlabel=NULL, ylabel=NULL) {
45

56
props <- list(id=id, clickData=clickData, domainStyle=domainStyle, height=height, loading_state=loading_state, margin=margin, mutationData=mutationData, needleStyle=needleStyle, rangeSlider=rangeSlider, width=width, xlabel=xlabel, ylabel=ylabel)

Diff for: ‎R/dashbioNglMoleculeViewer.R

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# AUTO GENERATED FILE - DO NOT EDIT
22

3+
#' @export
34
dashbioNglMoleculeViewer <- function(id=NULL, data=NULL, downloadImage=NULL, height=NULL, imageParameters=NULL, loading_state=NULL, molStyles=NULL, pdbString=NULL, stageParameters=NULL, width=NULL) {
45

56
props <- list(id=id, data=data, downloadImage=downloadImage, height=height, imageParameters=imageParameters, loading_state=loading_state, molStyles=molStyles, pdbString=pdbString, stageParameters=stageParameters, width=width)

0 commit comments

Comments
 (0)