| Age | Commit message (Expand) | Author |
|---|
| 2022-09-12 | Cleanup `PYTHONPATH`...Only add a value to the path if it is actually set
| Frederick Muriuki Muriithi |
| 2022-09-12 | Add tests for correlations...* Add a test to catch a regression
| Frederick Muriuki Muriithi |
| 2022-09-09 | Fix bug in display_mapping_results chromosome query...A tuple of chromosome names was being passed into the query as a single
argument, while it expected a separate argument for each chromosome
| zsloan |
| 2022-09-09 | Fix bug in retrieve_other_names...After a recent change, it always tries to unpack 6 values from the
query, but it should only be unpacking 4 values if not type ProbeSet (so
it was throwing an error for anything not ProbeSet)
| zsloan |
| 2022-09-09 | handle cases where tissue dataset vals are empty list | Alexander_Kabui |
| 2022-09-09 | tissue:fetch for query formatting | Alexander_Kabui |
| 2022-09-09 | Fix query: remove string interpolation. | Frederick Muriuki Muriithi |
| 2022-09-09 | Retain only a single import of the same module | Frederick Muriuki Muriithi |
| 2022-09-09 | Merge pull request #732 from BonfaceKilz/bug/fix-broken-correlations...Bug/fix broken correlations | Muriithi Frederick Muriuki |
| 2022-09-09 | Fetch sample values correctly...* wqflask/base/data_set.py (MrnaAssayDataSet.retrieve_sample_data):
Fix indentation.
| Munyoki Kilyungi |
| 2022-09-09 | Add missing import...* wqflask/base/trait.py: Import utility.hmac
| Munyoki Kilyungi |
| 2022-09-09 | Import missing module. | Frederick Muriuki Muriithi |
| 2022-09-09 | Check whether trait_list and sample_data exist before operations | Frederick Muriuki Muriithi |
| 2022-09-09 | Remove `None` values from dataset | Frederick Muriuki Muriithi |
| 2022-09-09 | Filter out Samples without values | Frederick Muriuki Muriithi |
| 2022-09-09 | Refactor: Add tests and handle edge case...Remove mutation of state, and handle the edge case where the
sub-sequence could be an empty sequence.
| Frederick Muriuki Muriithi |
| 2022-09-09 | Use `database_connection()`...Call the `database_connection()` function to get a connection to the
database.
| Frederick Muriuki Muriithi |
| 2022-09-09 | Replace `g.db.execute` with `cursor.execute` | Frederick Muriuki Muriithi |
| 2022-09-09 | Comment out pickling to avoid error...Pickling was failing with
"TypeError: cannot pickle 'Connection' object"
This commit avoids the error by temporarily commenting out the line to
prevent the pickling to actually test the working of the system on
the CI/CD server.
| Frederick Muriuki Muriithi |
| 2022-09-09 | If port is null, use MySQL's default 3306 | Frederick Muriuki Muriithi |
| 2022-09-09 | Use `urllib.parse.urljoin` for urls...Use the `urllib.parse.urljoin` function to join urls rather than
string concatenation for uniformity.
| Frederick Muriuki Muriithi |
| 2022-09-08 | Filter export by checked rows (when any rows are checked) | zsloan |
| 2022-09-08 | Fix getCheckedRows so that it only selects checkbox input elements | zsloan |
| 2022-09-08 | Convert variables to camelCase and remove some unused variables | zsloan |
| 2022-09-08 | Remove commented out code | zsloan |
| 2022-09-08 | Remove console log | zsloan |
| 2022-09-08 | Remove some unused initialized variables | zsloan |
| 2022-09-08 | Make GN_SERVER_URL the genenetwork3 API endpoint....From now on, GN_SERVER_URL will be the genenetwork3 API endpoint. For
example, GN_SERVER_URL=https://gn3.genenetwork.org/api
* wqflask/wqflask/collect.py (view_collection),
wqflask/wqflask/partial_correlations_views.py (partial_correlations,
poll_partial_correlation_results),
wqflask/wqflask/static/new/javascript/dataset_select_menu_orig.js (defaultStatusCodeFunctions):
Do not append api to GN_SERVER_URL.
* etc/default_settings.py (GN_SERVER_URL): Append /api.
| Arun Isaac |
| 2022-09-08 | Replace "asserts" with ValueErrors...* wqflask/wqflask/api/correlation.py (do_correlation): Raise a
ValueError if variables are not in the start_vars.
| Munyoki Kilyungi |
| 2022-09-08 | Delete unused imports...* wqflask/wqflask/external_tools/send_to_geneweaver.py: Delete unused
imports.
* wqflask/wqflask/external_tools/send_to_webgestalt.py: Ditto.
| Munyoki Kilyungi |
| 2022-09-08 | Simplify how chromosomes are fetched...* wqflask/base/species.py: Import "Any" and "Union".
(TheSpecies): Add type hints. Get rid of redundant "if ... else"
statement.
(IndChromosome.mb_length): Add type hints.
(Chromosomes): Inject conn and add type hints.
| Munyoki Kilyungi |
| 2022-09-08 | Add type-hints to database_connection...* wqflask/wqflask/database.py: Import Protocol, Any and Iterator.
(Connection): New protocol class for type-hints.
(read_from_pyfile): Add type-hints.
(sql_uri): Ditto.
(database_connection): Ditto.
| Munyoki Kilyungi |
| 2022-09-08 | Test the creation of the_species, indchromosomes and chromosome objs...* wqflask/tests/unit/base/test_species.py (TestTheSpecies): Delete.
(TestIndChromosome): Move to pytest and parametrize.
(TestChromosomes): Ditto.
(test_species): Ditto.
(test_create_ind_chromosome): Ditto.
(test_create_chromosomes): Ditto.
| Munyoki Kilyungi |
| 2022-09-08 | Removing object notation after a cursor fetch...* wqflask/wqflask/show_trait/SampleList.py
(SampleList.get_attributes): Remove object notation.
(SampleList.get_extra_attribute_values): Ditto.
| Munyoki Kilyungi |
| 2022-09-08 | Remove redundant cursor.fetchone()...* wqflask/base/data_set.py (DataSet.retrieve_other_names): Call
cursor.fetchone() only once.
| Munyoki Kilyungi |
| 2022-09-08 | Use "%s" in sql query...* wqflask/base/mrna_assay_tissue_data.py: Delete db_tools.
(MrnaAssayTissueData.get_symbol_values_pairs): Re-format query.
* wqflask/tests/unit/base/test_mrna_assay_tissue_data.py:
(test_get_trait_symbol_and_tissue_values): Add test for above.
| Munyoki Kilyungi |
| 2022-09-08 | Move test case to mrna_assay_tissue_data test module...* wqflask/tests/unit/base/test_mrna_assay_tissue_data.py
* wqflask/tests/unit/wqflask/correlation/test_correlation_functions.py
(TestCorrelationFunctions.test_tissue_corr_computation): Delete.
(TestCorrelationFunctions): Move this ...
(test_mrna_assay_tissue_data_initialisation): ... here
| Munyoki Kilyungi |
| 2022-09-08 | Inject database connection to mrna_assay_tissue_data class...* wqflask/base/mrna_assay_tissue_data.py: Imports: Delete
database_connection, escape, and database_connector.
(MrnaAssayTissueData): Inject conn. Re-format queries. Rework 'if
... else' logic. Re-work how results are assigned to
'self.data[symbol]' - remove dot-notation.
(MrnaAssayTissueData.get_symbol_values_pairs): Move box-comments to
doc-string. Rework how results are assigned to 'symbol_values_dict' -
remove dot-notation.
* wqflask/tests/unit/base/test_mrna_assay_tissue_data.py
(test_mrna_assay_tissue_data_initialisation): New test.
* wqflask/wqflask/correlation/correlation_functions.py: Import
database_connection.
(get_trait_symbol_and_tissue_values): Inject conn object.
| Munyoki Kilyungi |
| 2022-09-08 | Rewrite "test_tissue_corr_computation" as a pytest...* wqflask/tests/unit/wqflask/correlation/test_correlation_functions.py
(TestCorrelationFunctions): Convert this unittest class to ...
(test_tissue_corr_computation): ... a pytest function.
| Munyoki Kilyungi |
| 2022-09-08 | Use "%s" instead of "%2.6f" or "%d" in query....* wqflask/wqflask/interval_analyst/GeneUtil.py (loadGenes): Use "%s".
| Munyoki Kilyungi |
| 2022-09-08 | Fix sql queries..."%s" should only be used outside table names and column names
otherwise a string literal will be inserted thereby leading to errors
in the sql statements.
* wqflask/base/data_set.py (geno_mrna_confidentiality): Use f-strings
for table/columns/clause.
* wqflask/base/trait.py (retrieve_trait_info): Ditto.
* wqflask/wqflask/gsearch.py (GSearch.__init__): Ditto.
* wqflask/wqflask/interval_analyst/GeneUtil.py (loadGenes): Ditto.
* wqflask/wqflask/snp_browser/snp_browser.py
(SnpBrowser.get_browser_results): Ditto.
| Munyoki Kilyungi |
| 2022-09-08 | Deleting utility.logger...* wqflask/utility/logger.py: Delete.
| Munyoki Kilyungi |
| 2022-09-08 | Add missing "," in cursor.execute()...* wqflask/base/data_set.py (DatasetGroup.get_mapping_methods): Fix
failing query by adding a missing ",".
| Munyoki Kilyungi |
| 2022-09-08 | Delete unused imports...* wqflask/wqflask/search_results.py: Remove unused imports.
| Munyoki Kilyungi |
| 2022-09-08 | Remove intermediate variables that are only used once...* wqflask/base/data_set.py (DatasetGroup.__init__): Remove "query"
variable.
(DataSet.chunk_dataset): Fix indentation.
* wqflask/wqflask/search_results.py (get_GO_symbols): Remove
"this_term" variable.
* wqflask/wqflask/views.py (search_page): Remove "the_search"
variable.
| Munyoki Kilyungi |
| 2022-09-08 | Force pytest to use "importlib" import-mode to be able to run...Having "--import-mode=importlib" enables pytest to find the imports in
our tests; otherwise we get import errors.
* pyproject.toml: New pytest conf file.
| Munyoki Kilyungi |
| 2022-09-08 | Replace "g.db" mocks with "database_connection" mocks...* wqflask/tests/unit/base/test_trait.py: Delete "wqflask.app" import.
(TestRetrieveTraitInfo.setUp): Delete it.
(TestRetrieveTraitInfo.tearDown): Ditto.
(TestRetrieveTraitInfo.test_retrieve_trait_info_with_empty_dataset):
Replace "g" mock with "database_connection" mock.
(test_retrieve_trait_info_with_empty_trait_info): Ditto.
(test_retrieve_trait_info_with_non_empty_trait_info): Ditto.
(test_retrieve_trait_info_utf8_parsing): Ditto.
* wqflask/tests/wqflask/show_trait/test_show_trait.py (TestTraits):
(test_get_nearest_marker): Ditto.
(test_get_nearest_marker_empty_db): Ditto.
| Munyoki Kilyungi |
| 2022-09-08 | Adjust expected table width...* wqflask/tests/wqflask/show_trait/test_show_trait.py
(TestTraits.test_get_table_widths): Use new values for the table width
to make the tests pass.
| Munyoki Kilyungi |
| 2022-09-08 | Condense test-cases for "get_ncbi_summary_request" into one test...* wqflask/tests/wqflask/show_trait/test_show_trait.py:
(TestTraits.test_get_ncbi_summary_request_success): Condense this ...
(TestTraits.test_get_ncbi_summary_request): ... and this ...
(test_get_ncbi_summary_request): ... here.
| Munyoki Kilyungi |
| 2022-09-08 | Parametrize test cases for "check_if_attr_exists"...* wqflask/tests/wqflask/show_trait/test_show_trait.py: Import pytest.
(TestTraits.test_check_if_attr_exists_truthy): Delete it.
(TestTraits.test_check_if_attr_exists_empty_attr): Ditto.
(TestTraits.test_check_if_attr_exists_falsey): Ditto.
(TestTraits.test_check_if_attr_exists_falsey): Ditto.
test_check_if_attr_exists: Add new parametrized test case to test
"check_if_attr_exists".
| Munyoki Kilyungi |
