psdi-data-conversion 0.0.35__tar.gz → 0.0.37__tar.gz
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/CONTRIBUTING.md +27 -3
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/PKG-INFO +90 -51
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/README.md +88 -49
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/app.py +110 -10
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/constants.py +2 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/converter.py +16 -6
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/converters/atomsk.py +3 -1
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/converters/base.py +99 -39
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/converters/c2x.py +3 -1
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/converters/openbabel.py +40 -1
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/database.py +5 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/main.py +18 -10
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/accessibility.htm +5 -5
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/convert.htm +18 -13
- psdi_data_conversion-0.0.37/psdi_data_conversion/static/content/convertato.htm +87 -0
- psdi_data_conversion-0.0.37/psdi_data_conversion/static/content/convertc2x.htm +88 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/documentation.htm +4 -4
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/download.htm +26 -10
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/feedback.htm +4 -4
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/index-versions/psdi-common-header.html +1 -1
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/psdi-common-header.html +1 -1
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/report.htm +9 -7
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/common.js +20 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/convert.js +1 -2
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/convert_common.js +80 -7
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/convertato.js +1 -2
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/convertc2x.js +1 -2
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/format.js +12 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/report.js +6 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/styles/format.css +0 -6
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/styles/psdi-common.css +10 -6
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/templates/index.htm +10 -2
- psdi_data_conversion-0.0.37/psdi_data_conversion/testing/constants.py +15 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/testing/conversion_callbacks.py +7 -6
- psdi_data_conversion-0.0.37/psdi_data_conversion/testing/conversion_test_specs.py +388 -0
- psdi_data_conversion-0.0.37/psdi_data_conversion/testing/gui.py +366 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/testing/utils.py +108 -51
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/pyproject.toml +17 -8
- psdi_data_conversion-0.0.37/tests/gui/gui_test.py +109 -0
- {psdi_data_conversion-0.0.35/tests → psdi_data_conversion-0.0.37/tests/python}/cli_test.py +20 -84
- psdi_data_conversion-0.0.37/tests/python/converter_test.py +78 -0
- {psdi_data_conversion-0.0.35/tests → psdi_data_conversion-0.0.37/tests/python}/file_io_test.py +4 -2
- {psdi_data_conversion-0.0.35/tests → psdi_data_conversion-0.0.37/tests/python}/security_test.py +2 -2
- psdi_data_conversion-0.0.35/.github/workflows/ci-cron.yml +0 -51
- psdi_data_conversion-0.0.35/.github/workflows/ci-deploy-production.yml +0 -61
- psdi_data_conversion-0.0.35/.github/workflows/ci-feature.yml +0 -16
- psdi_data_conversion-0.0.35/.github/workflows/ci-main.yml +0 -34
- psdi_data_conversion-0.0.35/.github/workflows/ci-pr.yml +0 -26
- psdi_data_conversion-0.0.35/.github/workflows/ci-rc-hotfix.yml +0 -18
- psdi_data_conversion-0.0.35/.github/workflows/ci-release.yml +0 -130
- psdi_data_conversion-0.0.35/.github/workflows/job-anchore-scan.yml +0 -52
- psdi_data_conversion-0.0.35/.github/workflows/job-container-push.yml +0 -44
- psdi_data_conversion-0.0.35/.github/workflows/job-deploy-k8s.yml +0 -67
- psdi_data_conversion-0.0.35/.github/workflows/job-open-pull-request.yml +0 -56
- psdi_data_conversion-0.0.35/.github/workflows/job-publish-doc.yml +0 -52
- psdi_data_conversion-0.0.35/.github/workflows/job-tag.yml +0 -50
- psdi_data_conversion-0.0.35/.github/workflows/job-test-gui.yml +0 -36
- psdi_data_conversion-0.0.35/.github/workflows/job-test-python.yml +0 -51
- psdi_data_conversion-0.0.35/psdi_data_conversion/static/content/convertato.htm +0 -80
- psdi_data_conversion-0.0.35/psdi_data_conversion/static/content/convertc2x.htm +0 -81
- psdi_data_conversion-0.0.35/psdi_data_conversion/testing/constants.py +0 -12
- psdi_data_conversion-0.0.35/psdi_data_conversion/testing/conversion_test_specs.py +0 -208
- psdi_data_conversion-0.0.35/run_local.sh +0 -31
- psdi_data_conversion-0.0.35/test_data/output/caffeine-no-flags.smi +0 -1
- psdi_data_conversion-0.0.35/test_data/output/caffeine_a_in.smi +0 -1
- psdi_data_conversion-0.0.35/test_data/output/caffeine_a_in_kx_f4_l5_out.smi +0 -1
- psdi_data_conversion-0.0.35/test_data/output/caffeine_a_in_kx_f4_out.smi +0 -1
- psdi_data_conversion-0.0.35/test_data/output/caffeine_a_in_kx_out.smi +0 -1
- psdi_data_conversion-0.0.35/test_data/output/caffeine_a_in_x_out.smi +0 -1
- psdi_data_conversion-0.0.35/test_data/output/error_Fapatite_ins_to_cml_OB +0 -33
- psdi_data_conversion-0.0.35/test_data/output/error_cyclopropane_mol_to_xyz_c2x +0 -33
- psdi_data_conversion-0.0.35/test_data/output/error_nacl_mol_to_xyz_atomsk +0 -33
- psdi_data_conversion-0.0.35/tests/converter_test.py +0 -143
- psdi_data_conversion-0.0.35/tests/selenium/files/standard_test.cdxml +0 -3
- psdi_data_conversion-0.0.35/tests/selenium/run.sh +0 -5
- psdi_data_conversion-0.0.35/tests/selenium/selenium_tests.py +0 -140
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/.gitignore +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/CHANGELOG.md +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/LICENSE +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/__init__.py +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/bin/LICENSE_ATOMSK +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/bin/LICENSE_C2X +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/bin/linux/atomsk +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/bin/linux/c2x +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/bin/mac/atomsk +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/bin/mac/c2x +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/converters/__init__.py +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/dist.py +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/file_io.py +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/log_utility.py +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/scripts/atomsk.sh +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/scripts/c2x.sh +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/security.py +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/header-links.html +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/index-versions/header-links.html +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/index-versions/psdi-common-footer.html +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/content/psdi-common-footer.html +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/data/data.json +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/colormode-toggle-dm.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/colormode-toggle-lm.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/psdi-icon-dark.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/psdi-icon-light.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/psdi-logo-darktext-simple.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/psdi-logo-darktext.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/psdi-logo-lighttext-simple.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/psdi-logo-lighttext.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-bluesky-black.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-bluesky-white.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-instagram-black.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-instagram-white.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-linkedin-black.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-linkedin-white.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-mastodon-black.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-mastodon-white.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-x-black.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-x-white.svg +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-youtube-black.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/social-logo-youtube-white.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/ukri-epsr-logo-darktext.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/ukri-epsr-logo-lighttext.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/ukri-logo-darktext.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/img/ukri-logo-lighttext.png +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/accessibility.js +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/data.js +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/load_accessibility.js +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/static/javascript/psdi-common.js +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/psdi_data_conversion/testing/__init__.py +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/scripts/setup_bin.py +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/1ARJ.mmcif +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/1NE6.mmcif +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/5a9z-assembly1.cif +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/Fapatite.ins +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/aceticacid.mol +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/benzyne.molden +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/caffeine-smi.tar +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/caffeine-smi.tar.gz +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/caffeine-smi.zip +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/caffeine.inchi +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/ch3cl-esp.cub +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/cyclopropane_err.mol +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/ethanol.xyz +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/fullRhinovirus.pdb +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/hemoglobin.pdb +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/nacl.cif +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/nacl.mol +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/aceticacid.log.txt +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/aceticacid.mol2 +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/caffeine-2D-fastest.xyz +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/caffeine-3D-best.xyz +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/caffeine.smi +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/caffeine.xyz +0 -0
- /psdi_data_conversion-0.0.35/test_data/output/caffeine-ia.smi → /psdi_data_conversion-0.0.37/test_data/output/caffeine_a_in.smi +0 -0
- /psdi_data_conversion-0.0.35/test_data/output/caffeine-ia-okx-oof4l5.smi → /psdi_data_conversion-0.0.37/test_data/output/caffeine_a_in_kx_f4_l5_out.smi +0 -0
- /psdi_data_conversion-0.0.35/test_data/output/caffeine-ia-okx-oof4.smi → /psdi_data_conversion-0.0.37/test_data/output/caffeine_a_in_kx_f4_out.smi +0 -0
- /psdi_data_conversion-0.0.35/test_data/output/caffeine-ia-okx.smi → /psdi_data_conversion-0.0.37/test_data/output/caffeine_a_in_kx_out.smi +0 -0
- /psdi_data_conversion-0.0.35/test_data/output/caffeine-ia-ox.smi → /psdi_data_conversion-0.0.37/test_data/output/caffeine_a_in_x_out.smi +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/hemoglobin_Atomsk.xyz +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/hemoglobin_c2x.xyz +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/nacl.log +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/nacl.mol +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/quartz_OB.cif +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/quartz_OB.log.txt +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/quartz_atomsk.cif +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/quartz_atomsk.log.txt +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/standard_test.inchi +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/xyz_files-mol.zip +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/output/xyz_files.log.txt +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/periodic_dmol3.outmol +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/quartz.xyz +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/quartz_err.xyz +0 -0
- {psdi_data_conversion-0.0.35 → psdi_data_conversion-0.0.37}/test_data/standard_test.cdxml +0 -0
- {psdi_data_conversion-0.0.35/tests → psdi_data_conversion-0.0.37/tests/python}/database_test.py +0 -0
- {psdi_data_conversion-0.0.35/tests → psdi_data_conversion-0.0.37/tests/python}/dist_test.py +0 -0
- {psdi_data_conversion-0.0.35/tests → psdi_data_conversion-0.0.37/tests/python}/logging_test.py +0 -0
@@ -297,6 +297,22 @@ For debugging python issues, it's recommended to install the package in editable
|
|
297
297
|
pip install --editable .'[gui-test]'
|
298
298
|
```
|
299
299
|
|
300
|
+
### Running Unit Tests
|
301
|
+
|
302
|
+
The GUI unit tests require a web driver to be installed, and will install one if one isn't. To avoid repeated unnecessary API calls for the installation, you can set the environmental variable `DRIVER` when running tests to the location of your driver. This can be set to be automatically picked up by many IDEs by setting this in a `.env` file in the root of your project, e.g.:
|
303
|
+
|
304
|
+
```
|
305
|
+
DRIVER=~/.wdm/drivers/geckodriver/linux64/v0.36.0/geckodriver
|
306
|
+
```
|
307
|
+
|
308
|
+
To get the address of your driver, you can run the following in a Python terminal from within this projects virtual environment (after installing the `[gui-test]` optional dependencies):
|
309
|
+
|
310
|
+
```python
|
311
|
+
from webdriver_manager.firefox import GeckoDriverManager
|
312
|
+
driver_path = GeckoDriverManager().install()
|
313
|
+
print(f"Gecko driver installed to: {driver_path}")
|
314
|
+
```
|
315
|
+
|
300
316
|
## Continuous Integration
|
301
317
|
|
302
318
|
This project uses various GitHub workflows to perform Continuous Integration tasks. These can be found in the `.github/workflows` folder. The files which start with "ci-" are the directly-triggered workflows, and the files which start with "job-" are reusable workflows called by the former. These workflows handle the following tasks:
|
@@ -310,7 +326,7 @@ See the commons within the files for further details.
|
|
310
326
|
|
311
327
|
## Publishing
|
312
328
|
|
313
|
-
The Python library, CLA, and local GUI are
|
329
|
+
The Python library, CLA, and local GUI are published as a Python package via PyPI. This section describes how the package is set up and how it's published.
|
314
330
|
|
315
331
|
### Package Setup
|
316
332
|
|
@@ -318,11 +334,11 @@ The package's setup is defined in the `pyproject.toml` file. This defines the pr
|
|
318
334
|
|
319
335
|
The package uses [Hatch](https://hatch.pypa.io/latest/) for its build backend, as it is simpler to configure than the classic [Setuptools](https://setuptools.pypa.io/en/latest/userguide/) and provides some useful extensibility.
|
320
336
|
|
321
|
-
The version of the package is set to be determined from the version control system, meaning on the release branch, the version will always match the latest tag.
|
337
|
+
The version of the package is set to be determined from the version control system, meaning on the release branch, the version will always match the latest tag. This alleviates us of having to manually maintain the version for the package to keep it correct, but does result in some quirks. It's a bit fussier to set up (though that's done now), and it makes the user take an extra step if they want to install from source but haven't cloned the repository - this is noted in the installation instructions in the README.
|
322
338
|
|
323
339
|
### Initial Publication
|
324
340
|
|
325
|
-
This section details the
|
341
|
+
This section details the proceduce for the initial publication of this package - now that this is complete, this section is left in for reference in case of future need.
|
326
342
|
|
327
343
|
First, it's necessary to install a couple required packages in order to build a Python package: `build` to build it and `twine` to upload it. These can be installed with pip via:
|
328
344
|
|
@@ -350,6 +366,14 @@ To upload, follow [this tutorial](https://packaging.python.org/en/latest/tutoria
|
|
350
366
|
|
351
367
|
The `ci-release.yml` workflow is planned to publish any new releases to PyPI after the initial publication. This can be set up by uncommenting the relevant lines in this file, possibly updating the `job-publish-pypi.yml` to enable this (testing will be necessary), and setting up Trusted Publishing for the project on PyPI (see [guide](https://docs.github.com/en/actions/security-for-github-actions/security-hardening-your-deployments/configuring-openid-connect-in-pypi))
|
352
368
|
|
369
|
+
### Project Management
|
370
|
+
|
371
|
+
This project is published on PyPI at https://pypi.org/project/psdi-data-conversion/ and on TestPyPI at https://test.pypi.org/project/psdi-data-conversion/. Maintainers can manage the project through the "Manage" link on that page or from their own projects page.
|
372
|
+
|
373
|
+
The most important setting to be aware of here is Publishing -> Trusted Publisher Management. This is the system used to allow automatic publishing of releases from GitHub. It's set up so that the current project, organisation, environment, and workflow for publishing are approved. If any of these change, this will need to be updated by adding a new trusted publisher with the new settings (on both PyPI and TestPyPI) and removing the old one.
|
374
|
+
|
375
|
+
The management page can also be used to add or remove collaborators through the Collaborators tab. Generally the project on these sites doesn't require much maintenance, but at least a few active collaborators should be on it at all times to avoid getting locked out if someone is suddenly unavailable.
|
376
|
+
|
353
377
|
## Deployment
|
354
378
|
|
355
379
|
Deployment is handled by the `job-deploy-k8s.yml` reusable workflow, which is triggered from `ci-main.yml` to deploy to dev on each push to `main` and `ci-release.yml` to deploy to staging on each push to `release`. When a push to production is desired, it must be triggered manually by calling the `ci-deploy-production.yml` workflow on the `release` branch (it's set up to fail if run on any other branch).
|
@@ -1,6 +1,6 @@
|
|
1
|
-
Metadata-Version: 2.
|
1
|
+
Metadata-Version: 2.4
|
2
2
|
Name: psdi_data_conversion
|
3
|
-
Version: 0.0.
|
3
|
+
Version: 0.0.37
|
4
4
|
Summary: Chemistry file format conversion service, provided by PSDI
|
5
5
|
Project-URL: Homepage, https://data-conversion.psdi.ac.uk/
|
6
6
|
Project-URL: Documentation, https://psdi-uk.github.io/psdi-data-conversion/
|
@@ -278,7 +278,10 @@ This is the repository for the PSDI PF2 Chemistry File Format Conversion project
|
|
278
278
|
- [Installation and Setup](#installation-and-setup)
|
279
279
|
- [Running the App](#running-the-app)
|
280
280
|
- [Testing](#testing)
|
281
|
-
- [
|
281
|
+
- [Troubleshooting](#troubleshooting)
|
282
|
+
- [OSError: [Errno 24] Too many open files](#oserror-errno-24-too-many-open-files)
|
283
|
+
- [Errors running c2x or Atomsk converters](#errors-running-c2x-or-atomsk-converters)
|
284
|
+
- [Licensing](#licensing)
|
282
285
|
- [Contributors](#contributors)
|
283
286
|
- [Funding](#funding)
|
284
287
|
|
@@ -312,15 +315,17 @@ This is the repository for the PSDI PF2 Chemistry File Format Conversion project
|
|
312
315
|
- `test_data`
|
313
316
|
- (Files used for testing the project)
|
314
317
|
- `tests`
|
315
|
-
-
|
318
|
+
- `gui`
|
319
|
+
- (Unit tests for the GUI, aka the local version of the web app)
|
320
|
+
- `python`
|
321
|
+
- (Unit tests for the Python library and command-line application)
|
316
322
|
- `CHANGELOG.md` (Updates since initial public release)
|
317
323
|
- `CONTRIBUTING.md` (Guidelines and information for contributors to the project)
|
318
324
|
- `DOCKERFILE` (Dockerfile for image containerising PSDI's data conversion service)
|
319
|
-
- `LICENSE` (Apache
|
325
|
+
- `LICENSE` (Apache License version 2.0)
|
320
326
|
- `pyproject.toml` (Python project metadata and settings)
|
321
327
|
- `README.md` (This file)
|
322
328
|
- `requirements.txt` (Requirements for the web app deployment of this project)
|
323
|
-
- `run_local.sh` (Helper script to run the web app locally)
|
324
329
|
|
325
330
|
## Requirements
|
326
331
|
|
@@ -403,13 +408,21 @@ In addition to the dependencies listed above, this project uses the assets made
|
|
403
408
|
|
404
409
|
### Installation
|
405
410
|
|
406
|
-
The CLA and Python library are installed together. This
|
411
|
+
The CLA and Python library are installed together. This project is available on PyPI, and so can be installed via pip with:
|
412
|
+
|
413
|
+
```bash
|
414
|
+
pip install psdi-data-conversion
|
415
|
+
```
|
416
|
+
|
417
|
+
If you wish to install from source, this can be done most easily by cloning the project and then executing:
|
407
418
|
|
408
419
|
```bash
|
409
420
|
pip install .
|
410
421
|
```
|
411
422
|
|
412
|
-
|
423
|
+
from this project's directory. You can also replace the '.' in this command with the path to this project's directory to install it from elsewhere.
|
424
|
+
|
425
|
+
**Note:** This project uses git to determine the version number. If you clone the repository, you won't have to do anything special here, but if you get the source e.g. by extracting a release archive, you'll have to do one additional step before running the command above. If you have git installed, simply run `git init` in the project directory and it will be able to install. Otherwise, edit the project's `pyproject.toml` file to uncomment the line that sets a fixed version, and comment out the lines that set it up to determine the version from git - these are pointed out in the comments there.
|
413
426
|
|
414
427
|
Depending on your system, it may not be possible to install packages in this manner without creating a virtual environment to do so in. You can do this by first installing the `venv` module for Python3 with e.g.:
|
415
428
|
|
@@ -572,51 +585,25 @@ Enter https://data-conversion.psdi.ac.uk/ in a browser. Guidance on usage is giv
|
|
572
585
|
|
573
586
|
### Installation and Setup
|
574
587
|
|
575
|
-
|
588
|
+
This project is available on PyPI, and so can be installed via pip, including the necessary dependencies for the GUI, with:
|
576
589
|
|
577
590
|
```bash
|
578
|
-
pip install
|
591
|
+
pip install psdi-data-conversion'[gui]'
|
579
592
|
```
|
580
593
|
|
581
|
-
If
|
582
|
-
|
583
|
-
If you've cloned this repository, you can use the `run_local.sh` bash script to run the application. Otherwise (e.g. if you've installed from a wheel or PyPI), copy and paste the following into a script:
|
594
|
+
If you wish to install the project locally from source, this can be done most easily by cloning the project and then executing:
|
584
595
|
|
585
596
|
```bash
|
586
|
-
|
587
|
-
|
588
|
-
# The envvar MAX_FILESIZE can be used to set the maximum allowed filesize in MB - 0 indicates no maximum
|
589
|
-
if [ -z $MAX_FILESIZE ]; then
|
590
|
-
export MAX_FILESIZE=0
|
591
|
-
fi
|
592
|
-
|
593
|
-
# Logging control - "full" sets server-style logging, which is necessary to produce the output logs with the expected
|
594
|
-
# names. This should not be changed, or else errors will occur
|
595
|
-
export LOG_MODE=full
|
596
|
-
|
597
|
-
# The level to log at. Leave blank for defaults, which logs at INFO level for user output and ERROR level for the server
|
598
|
-
# log and stdout. If set to a different value here (e.g. DEBUG), all these channels will be set to that level
|
599
|
-
export LOG_LEVEL=
|
600
|
-
|
601
|
-
# The envvar SERVICE_MODE can be set to "true" to make this behave as if it's running as the public web service -
|
602
|
-
# uncomment the following line to enable that
|
603
|
-
# export SERVICE_MODE=true
|
604
|
-
|
605
|
-
# Uncomment the following line to enable debug mode
|
606
|
-
# export FLASK_ENV=development
|
607
|
-
|
608
|
-
# Execute a local run of the application from the proper path
|
609
|
-
|
610
|
-
PACKAGE_PATH=`python -c "import psdi_data_conversion; print(psdi_data_conversion.__path__[0])"`
|
611
|
-
cd $PACKAGE_PATH/..
|
612
|
-
python -m flask --app psdi_data_conversion/app.py run
|
597
|
+
pip install .'[gui]'
|
613
598
|
```
|
614
599
|
|
615
|
-
If
|
600
|
+
**Note:** This project uses git to determine the version number. If you clone the repository, you won't have to do anything special here, but if you get the source e.g. by extracting a release archive, you'll have to do one additional step before running the command above. If you have git installed, simply run `git init` in the project directory and it will be able to install. Otherwise, edit the project's `pyproject.toml` file to uncomment the line that sets a fixed version, and comment out the lines that set it up to determine the version from git - these are pointed out in the comments there.
|
601
|
+
|
602
|
+
If your system does not allow installation in this manner, it may be necessary to set up a virtual environment. See the instructions in the [command-line application installation](#installation) section above for how to do that, and then try to install again once you've set one up and activated it.
|
616
603
|
|
617
604
|
### Running the App
|
618
605
|
|
619
|
-
|
606
|
+
Once installed, the command-line script `psdi-data-convert-gui` will be made available, which can be called to start the server. You can then access the website by going to <http://127.0.0.1:5000> in a browser (this will also be printed in the terminal, and you can CTRL+click it there to open it in your default browser). Guidance for using the app is given on each page of it. When you're finished with the app, key CTRL+C in the terminal where you called the script to shut down the server, or, if the process was backgrounded, kill the appropriate process.
|
620
607
|
|
621
608
|
In case of problems when using Chrome, try opening Chrome from the command line:
|
622
609
|
open -a "Google Chrome.app" --args --allow-file-access-from-files
|
@@ -639,28 +626,80 @@ To test the CLA and Python library, install the optional testing requirements lo
|
|
639
626
|
|
640
627
|
```bash
|
641
628
|
pip install .'[test]'
|
642
|
-
pytest
|
629
|
+
pytest tests/python
|
643
630
|
```
|
644
631
|
|
645
632
|
To test the local version of the web app, install the GUI testing requirements locally (which also include the standard GUI requirements and standard testing requirements), start the server, and test by executing the GUI test script:
|
646
633
|
|
647
634
|
```bash
|
648
635
|
pip install .'[gui-test]'
|
649
|
-
|
650
|
-
|
651
|
-
|
652
|
-
|
636
|
+
pytest tests/gui
|
637
|
+
```
|
638
|
+
|
639
|
+
Both of these sets of tests can also be run together if desired through:
|
640
|
+
|
641
|
+
```bash
|
642
|
+
pip install .'[gui-test]'
|
643
|
+
pytest
|
644
|
+
```
|
645
|
+
|
646
|
+
## Troubleshooting
|
647
|
+
|
648
|
+
This section presents solutions for commonly-encountered issues.
|
649
|
+
|
650
|
+
### OSError: [Errno 24] Too many open files
|
651
|
+
|
652
|
+
You may see the error:
|
653
|
+
|
654
|
+
```
|
655
|
+
OSError: [Errno 24] Too many open files
|
656
|
+
```
|
657
|
+
|
658
|
+
while running the command-line application, using the Python library, or running tests This error is caused by a program hitting the limit of the number of open filehandles allowed by the OS. This limit is typically set to 1024 on Linux systems and 256 on MacOS systems, and thus this issue occurs much more often on the latter. You can see what your current limit is by running the command:
|
659
|
+
|
660
|
+
```bash
|
661
|
+
ulimit -a | grep "open files"
|
662
|
+
```
|
663
|
+
|
664
|
+
This limit can be temporarily increased for the current terminal session by running the command:
|
665
|
+
|
666
|
+
```bash
|
667
|
+
ulimit -n 1024 # Or another, higher number
|
668
|
+
```
|
669
|
+
|
670
|
+
First, try increasing the limit and then redo the operation which caused this error to see if this resolves it. If this does, you can make this change permanent in a few ways, the easiest of which is to add this command to your `.bashrc` file so it will be set for every new terminal session.
|
671
|
+
|
672
|
+
If you see this error when the filehandle limit is already set to a high value such as 1024, this may indicate the presence of a bug in the project which causes a leaking of filehandles, so please open an issue about it, pasting the error you get and the details of your system, particularly including your current filehandle limit.
|
673
|
+
|
674
|
+
### Errors running c2x or Atomsk converters
|
675
|
+
|
676
|
+
We provide support for the c2x and Atomsk converters by packing binaries which support common Linux and MacOS platforms with this project, but we cannot guarantee universal support for these binaries. In particular, they may rely on dynamically-linked libraries which aren't installed on your system.
|
677
|
+
|
678
|
+
Look through the error message you received for messages such as "Library not loaded" or "no such file", and see if they point to the name of a library which you can try installing. For instance, if you see that it's searching for `libquadmath.0.dylib` but not finding it, you can try installing this library. In this case, this library can be installed through apt with:
|
679
|
+
|
680
|
+
```bash
|
681
|
+
sudo apt install libquadmath0
|
653
682
|
```
|
654
683
|
|
655
|
-
|
684
|
+
or through brew via:
|
685
|
+
|
686
|
+
```bash
|
687
|
+
brew install gcc
|
688
|
+
```
|
689
|
+
|
690
|
+
Alternatively, you can run your own versions of the `c2x` and `atomsk` binaries with this project. Compile them yourself however you wish - see the projects at https://github.com/codenrx/c2x and https://github.com/pierrehirel/atomsk and follow their instructions to build a binary on your system. Once you've done so, add the binary to your `$PATH`, and this project will pick that up and use it in preference to the prepackaged binary.
|
691
|
+
|
692
|
+
On the other hand, it's possible that an error of this sort will occur if you have a non-working binary of one of these converters in your `$PATH`. If this might be the case, you can try removing it and see if the prepackaged binary works for you, or else recompile it to try to fix errors.
|
693
|
+
|
694
|
+
## Licensing
|
656
695
|
|
657
696
|
This project is provided under the Apache License version 2.0, the terms of which can be found in the file `LICENSE`.
|
658
697
|
|
659
|
-
This project redistributes compiled binaries for the Atomsk and c2x converters. These are both
|
698
|
+
This project redistributes compiled binaries for the Atomsk and c2x converters. These are both licensed under the
|
660
699
|
GNU General Public License version 3 and are redistributed per its terms. Any further redistribution of these binaries,
|
661
|
-
including redistribution of this project as a whole, including them, must also follow the terms of this
|
700
|
+
including redistribution of this project as a whole, including them, must also follow the terms of this license.
|
662
701
|
|
663
|
-
This requires conspicuously displaying notice of this
|
702
|
+
This requires conspicuously displaying notice of this license, providing the text of of the license (provided here in
|
664
703
|
the files `psdi_data_conversion/bin/LICENSE_C2X` and `psdi_data_conversion/bin/LICENSE_ATOMSK`), and appropriately
|
665
704
|
conveying the source code for each of these. Their respective source code may be found at:
|
666
705
|
|
@@ -33,7 +33,10 @@ This is the repository for the PSDI PF2 Chemistry File Format Conversion project
|
|
33
33
|
- [Installation and Setup](#installation-and-setup)
|
34
34
|
- [Running the App](#running-the-app)
|
35
35
|
- [Testing](#testing)
|
36
|
-
- [
|
36
|
+
- [Troubleshooting](#troubleshooting)
|
37
|
+
- [OSError: [Errno 24] Too many open files](#oserror-errno-24-too-many-open-files)
|
38
|
+
- [Errors running c2x or Atomsk converters](#errors-running-c2x-or-atomsk-converters)
|
39
|
+
- [Licensing](#licensing)
|
37
40
|
- [Contributors](#contributors)
|
38
41
|
- [Funding](#funding)
|
39
42
|
|
@@ -67,15 +70,17 @@ This is the repository for the PSDI PF2 Chemistry File Format Conversion project
|
|
67
70
|
- `test_data`
|
68
71
|
- (Files used for testing the project)
|
69
72
|
- `tests`
|
70
|
-
-
|
73
|
+
- `gui`
|
74
|
+
- (Unit tests for the GUI, aka the local version of the web app)
|
75
|
+
- `python`
|
76
|
+
- (Unit tests for the Python library and command-line application)
|
71
77
|
- `CHANGELOG.md` (Updates since initial public release)
|
72
78
|
- `CONTRIBUTING.md` (Guidelines and information for contributors to the project)
|
73
79
|
- `DOCKERFILE` (Dockerfile for image containerising PSDI's data conversion service)
|
74
|
-
- `LICENSE` (Apache
|
80
|
+
- `LICENSE` (Apache License version 2.0)
|
75
81
|
- `pyproject.toml` (Python project metadata and settings)
|
76
82
|
- `README.md` (This file)
|
77
83
|
- `requirements.txt` (Requirements for the web app deployment of this project)
|
78
|
-
- `run_local.sh` (Helper script to run the web app locally)
|
79
84
|
|
80
85
|
## Requirements
|
81
86
|
|
@@ -158,13 +163,21 @@ In addition to the dependencies listed above, this project uses the assets made
|
|
158
163
|
|
159
164
|
### Installation
|
160
165
|
|
161
|
-
The CLA and Python library are installed together. This
|
166
|
+
The CLA and Python library are installed together. This project is available on PyPI, and so can be installed via pip with:
|
167
|
+
|
168
|
+
```bash
|
169
|
+
pip install psdi-data-conversion
|
170
|
+
```
|
171
|
+
|
172
|
+
If you wish to install from source, this can be done most easily by cloning the project and then executing:
|
162
173
|
|
163
174
|
```bash
|
164
175
|
pip install .
|
165
176
|
```
|
166
177
|
|
167
|
-
|
178
|
+
from this project's directory. You can also replace the '.' in this command with the path to this project's directory to install it from elsewhere.
|
179
|
+
|
180
|
+
**Note:** This project uses git to determine the version number. If you clone the repository, you won't have to do anything special here, but if you get the source e.g. by extracting a release archive, you'll have to do one additional step before running the command above. If you have git installed, simply run `git init` in the project directory and it will be able to install. Otherwise, edit the project's `pyproject.toml` file to uncomment the line that sets a fixed version, and comment out the lines that set it up to determine the version from git - these are pointed out in the comments there.
|
168
181
|
|
169
182
|
Depending on your system, it may not be possible to install packages in this manner without creating a virtual environment to do so in. You can do this by first installing the `venv` module for Python3 with e.g.:
|
170
183
|
|
@@ -327,51 +340,25 @@ Enter https://data-conversion.psdi.ac.uk/ in a browser. Guidance on usage is giv
|
|
327
340
|
|
328
341
|
### Installation and Setup
|
329
342
|
|
330
|
-
|
343
|
+
This project is available on PyPI, and so can be installed via pip, including the necessary dependencies for the GUI, with:
|
331
344
|
|
332
345
|
```bash
|
333
|
-
pip install
|
346
|
+
pip install psdi-data-conversion'[gui]'
|
334
347
|
```
|
335
348
|
|
336
|
-
If
|
337
|
-
|
338
|
-
If you've cloned this repository, you can use the `run_local.sh` bash script to run the application. Otherwise (e.g. if you've installed from a wheel or PyPI), copy and paste the following into a script:
|
349
|
+
If you wish to install the project locally from source, this can be done most easily by cloning the project and then executing:
|
339
350
|
|
340
351
|
```bash
|
341
|
-
|
342
|
-
|
343
|
-
# The envvar MAX_FILESIZE can be used to set the maximum allowed filesize in MB - 0 indicates no maximum
|
344
|
-
if [ -z $MAX_FILESIZE ]; then
|
345
|
-
export MAX_FILESIZE=0
|
346
|
-
fi
|
347
|
-
|
348
|
-
# Logging control - "full" sets server-style logging, which is necessary to produce the output logs with the expected
|
349
|
-
# names. This should not be changed, or else errors will occur
|
350
|
-
export LOG_MODE=full
|
351
|
-
|
352
|
-
# The level to log at. Leave blank for defaults, which logs at INFO level for user output and ERROR level for the server
|
353
|
-
# log and stdout. If set to a different value here (e.g. DEBUG), all these channels will be set to that level
|
354
|
-
export LOG_LEVEL=
|
355
|
-
|
356
|
-
# The envvar SERVICE_MODE can be set to "true" to make this behave as if it's running as the public web service -
|
357
|
-
# uncomment the following line to enable that
|
358
|
-
# export SERVICE_MODE=true
|
359
|
-
|
360
|
-
# Uncomment the following line to enable debug mode
|
361
|
-
# export FLASK_ENV=development
|
362
|
-
|
363
|
-
# Execute a local run of the application from the proper path
|
364
|
-
|
365
|
-
PACKAGE_PATH=`python -c "import psdi_data_conversion; print(psdi_data_conversion.__path__[0])"`
|
366
|
-
cd $PACKAGE_PATH/..
|
367
|
-
python -m flask --app psdi_data_conversion/app.py run
|
352
|
+
pip install .'[gui]'
|
368
353
|
```
|
369
354
|
|
370
|
-
If
|
355
|
+
**Note:** This project uses git to determine the version number. If you clone the repository, you won't have to do anything special here, but if you get the source e.g. by extracting a release archive, you'll have to do one additional step before running the command above. If you have git installed, simply run `git init` in the project directory and it will be able to install. Otherwise, edit the project's `pyproject.toml` file to uncomment the line that sets a fixed version, and comment out the lines that set it up to determine the version from git - these are pointed out in the comments there.
|
356
|
+
|
357
|
+
If your system does not allow installation in this manner, it may be necessary to set up a virtual environment. See the instructions in the [command-line application installation](#installation) section above for how to do that, and then try to install again once you've set one up and activated it.
|
371
358
|
|
372
359
|
### Running the App
|
373
360
|
|
374
|
-
|
361
|
+
Once installed, the command-line script `psdi-data-convert-gui` will be made available, which can be called to start the server. You can then access the website by going to <http://127.0.0.1:5000> in a browser (this will also be printed in the terminal, and you can CTRL+click it there to open it in your default browser). Guidance for using the app is given on each page of it. When you're finished with the app, key CTRL+C in the terminal where you called the script to shut down the server, or, if the process was backgrounded, kill the appropriate process.
|
375
362
|
|
376
363
|
In case of problems when using Chrome, try opening Chrome from the command line:
|
377
364
|
open -a "Google Chrome.app" --args --allow-file-access-from-files
|
@@ -394,28 +381,80 @@ To test the CLA and Python library, install the optional testing requirements lo
|
|
394
381
|
|
395
382
|
```bash
|
396
383
|
pip install .'[test]'
|
397
|
-
pytest
|
384
|
+
pytest tests/python
|
398
385
|
```
|
399
386
|
|
400
387
|
To test the local version of the web app, install the GUI testing requirements locally (which also include the standard GUI requirements and standard testing requirements), start the server, and test by executing the GUI test script:
|
401
388
|
|
402
389
|
```bash
|
403
390
|
pip install .'[gui-test]'
|
404
|
-
|
405
|
-
|
406
|
-
|
407
|
-
|
391
|
+
pytest tests/gui
|
392
|
+
```
|
393
|
+
|
394
|
+
Both of these sets of tests can also be run together if desired through:
|
395
|
+
|
396
|
+
```bash
|
397
|
+
pip install .'[gui-test]'
|
398
|
+
pytest
|
399
|
+
```
|
400
|
+
|
401
|
+
## Troubleshooting
|
402
|
+
|
403
|
+
This section presents solutions for commonly-encountered issues.
|
404
|
+
|
405
|
+
### OSError: [Errno 24] Too many open files
|
406
|
+
|
407
|
+
You may see the error:
|
408
|
+
|
409
|
+
```
|
410
|
+
OSError: [Errno 24] Too many open files
|
411
|
+
```
|
412
|
+
|
413
|
+
while running the command-line application, using the Python library, or running tests This error is caused by a program hitting the limit of the number of open filehandles allowed by the OS. This limit is typically set to 1024 on Linux systems and 256 on MacOS systems, and thus this issue occurs much more often on the latter. You can see what your current limit is by running the command:
|
414
|
+
|
415
|
+
```bash
|
416
|
+
ulimit -a | grep "open files"
|
417
|
+
```
|
418
|
+
|
419
|
+
This limit can be temporarily increased for the current terminal session by running the command:
|
420
|
+
|
421
|
+
```bash
|
422
|
+
ulimit -n 1024 # Or another, higher number
|
423
|
+
```
|
424
|
+
|
425
|
+
First, try increasing the limit and then redo the operation which caused this error to see if this resolves it. If this does, you can make this change permanent in a few ways, the easiest of which is to add this command to your `.bashrc` file so it will be set for every new terminal session.
|
426
|
+
|
427
|
+
If you see this error when the filehandle limit is already set to a high value such as 1024, this may indicate the presence of a bug in the project which causes a leaking of filehandles, so please open an issue about it, pasting the error you get and the details of your system, particularly including your current filehandle limit.
|
428
|
+
|
429
|
+
### Errors running c2x or Atomsk converters
|
430
|
+
|
431
|
+
We provide support for the c2x and Atomsk converters by packing binaries which support common Linux and MacOS platforms with this project, but we cannot guarantee universal support for these binaries. In particular, they may rely on dynamically-linked libraries which aren't installed on your system.
|
432
|
+
|
433
|
+
Look through the error message you received for messages such as "Library not loaded" or "no such file", and see if they point to the name of a library which you can try installing. For instance, if you see that it's searching for `libquadmath.0.dylib` but not finding it, you can try installing this library. In this case, this library can be installed through apt with:
|
434
|
+
|
435
|
+
```bash
|
436
|
+
sudo apt install libquadmath0
|
408
437
|
```
|
409
438
|
|
410
|
-
|
439
|
+
or through brew via:
|
440
|
+
|
441
|
+
```bash
|
442
|
+
brew install gcc
|
443
|
+
```
|
444
|
+
|
445
|
+
Alternatively, you can run your own versions of the `c2x` and `atomsk` binaries with this project. Compile them yourself however you wish - see the projects at https://github.com/codenrx/c2x and https://github.com/pierrehirel/atomsk and follow their instructions to build a binary on your system. Once you've done so, add the binary to your `$PATH`, and this project will pick that up and use it in preference to the prepackaged binary.
|
446
|
+
|
447
|
+
On the other hand, it's possible that an error of this sort will occur if you have a non-working binary of one of these converters in your `$PATH`. If this might be the case, you can try removing it and see if the prepackaged binary works for you, or else recompile it to try to fix errors.
|
448
|
+
|
449
|
+
## Licensing
|
411
450
|
|
412
451
|
This project is provided under the Apache License version 2.0, the terms of which can be found in the file `LICENSE`.
|
413
452
|
|
414
|
-
This project redistributes compiled binaries for the Atomsk and c2x converters. These are both
|
453
|
+
This project redistributes compiled binaries for the Atomsk and c2x converters. These are both licensed under the
|
415
454
|
GNU General Public License version 3 and are redistributed per its terms. Any further redistribution of these binaries,
|
416
|
-
including redistribution of this project as a whole, including them, must also follow the terms of this
|
455
|
+
including redistribution of this project as a whole, including them, must also follow the terms of this license.
|
417
456
|
|
418
|
-
This requires conspicuously displaying notice of this
|
457
|
+
This requires conspicuously displaying notice of this license, providing the text of of the license (provided here in
|
419
458
|
the files `psdi_data_conversion/bin/LICENSE_C2X` and `psdi_data_conversion/bin/LICENSE_ATOMSK`), and appropriately
|
420
459
|
conveying the source code for each of these. Their respective source code may be found at:
|
421
460
|
|