Age | Commit message (Collapse) | Author |
|
Rules used are:
E20,E211,E22,E224,E224,E225,E226,E227,E228,E231,E241,E242,
E251,E252,E26,E265,E266,E27,E301,E302,E303,E304,E305,E306,
E401,E501,E70,E701,W291,W292,W293,W391,W504,E101,E11,E121,
E122,E123,E124,E125,E126,E127,E128,E129,E131,E133
|
|
|
|
|
|
|
|
|
|
|
|
Run:
python -m autopep8 --in-place --recrusive ./ --select\
E20,E211,E22,E224,E224,E225,E226,E227,E228,E231,E241,\
E242,E251,E252,E26,E265,E266,E27 -p 3
|
|
|
|
See: https://is.gd/pL7IJF
Ran:
find . \( -type d -name .git -prune \) -o -type f -print0 | xargs -0 sed -i 's/(object)//g'
|
|
* wqflask/base/trait.py (jsonable_table_row): Delete it.
|
|
* wqflask/base/data_set.py (Dataset.get_accession_id): Delete it.
|
|
|
|
* wqflask/base/data_set.py (riset): Delete class method.
|
|
@dataclasses should be used to hold only data. Since dataset only
encapsulates data, re-writing it using a dataclass makes it more
readable and easier to follow.
|
|
necessary for them to show up correctly when selecting traits from collections in pop-up windows, like when selecting cofactors for mapping)
|
|
|
|
|
|
|
|
resulting from a trait missing both pre_publication_description and pubmed ID
|
|
lists (I think the proxy expects them to not be lists, but then returns them as lists; need to check about this)
|
|
|
|
|
|
|
|
|
|
|
|
and added an assert that 'name' is passed instead of using if/else, since it should always be passed
|
|
Feature/corr table changes
|
|
for genotype/snp cofactors (the code previously only accounted for
probeset/phenotype cofactors, so it was treating genotypes/snps like
phenotype traits)
|
|
cofactors were genotypes (because it tried to get the
"description_display" which doesn't exist for genotype traits)
|
|
ID for the purposes of showing pre-publication abbreviation/description
|
|
to be compatible with Python 3 syntax
* wqflask/base/GeneralObject.py - edited docstring to be correct for Python 3
|
|
individual sample, because it was previously causing problems when only some samples had an N
* wqflask/base/webqtlCaseData.py - Removed code getting first_attr_start_pos
* wqflask/wqflask/show_trait/SampleList.py - Added code getting first_attr_start_pos and assigning it to all samples + removing one unnecessary sort
* wqflask/wqflask/static/new/javascript/initialize_show_trait_tables.js - Fixed issue that was causing the N column to not be displayed
|
|
|
|
|
|
|
|
Tests/add extra tests for collections
|
|
* wqflask/wqflask/wgcna/wgcna_analysis.py: Apply pep8
* wqflask/wqflask/collect.py: Ditto
* wqflask/base/trait.py: Ditto
|
|
* wqflask/base/data_set.py - Fixed issue where there was an error when trying to take the len of the query results when there were no results
|
|
self.datasets property because the group name was used as the key (instead of the group name + "Publish", which is the full dataset name for phenotypes)
* wqflask/base/data_set.py - Set "group_name" as a separate variable from "name" to avoid it being used as the key in self.datasets
|
|
* wqflask/base/trait.py (retrieve_trait_info): Add extra param to
ignore errors such as: "utf-8 codec can't decode byte 0xb5 in position
0: invalid start byte" thrown in Python3.
|
|
|
|
* wqflask/base/data_set.py - TempDataSet class contained some code
referring to the SQL DB, but temp traits aren't stored in the SQL DB anymore
|
|
github.com:BonfaceKilz/genenetwork2 into build/python3-migration
|
|
* wqflask/base/trait.py (retrieve_trait_info): If the
description_string or discription_display value return a None object,
evaluate to an empty string. Fixes errors related to *len(None)*.
|
|
|
|
* wqflask/base/trait.py (retrieve_trait_info): If the
description_string or discription_display value return a None object,
evaluate to an empty string. Fixes errors related to *len(None)*.
|
|
* wqflask/base/trait.py: Fixes error parsing error: "(GeneNetwork
error: float argument required, not NoneType)"
|
|
a separate file (initialize_show_trait_tables.js). The biggest
complication was getting the order of attributes in the rows to sync
with the order of atribute namees in the column headers. Previously this
logic was all in the template.
* wqflask/base/webqtlCaseData.py - added attribute first_attr_col as a
very awkward solution to passing the column position into the column
render function in situations where there are case attribute columns
(which can be variable in number)
* wqflask/wqflask/show_trait/show_trait.py - Replace "attribute_names"
in js_data with "attributes" (which allows the JS access to more
information) and also pass new se_exists and
has_num_cases variables with js_data, so the javascript has access to
whether or not those columns exist in the table
* wqflask/wqflask/static/new/javascript/show_trait.js - Change case
attribute-related logic to use js_data.attributes instead of
js_data.attribute_names
* wqflask/wqflask/templates/show_trait.html - Removed table
initialization from template
* wqflask/wqflask/static/new/javascript/initialize_show_trait_tables.js
- new file that initializes tables and reproduces what the template
logic used to do with JS logic
|
|
* wqflask/base/trait.py (retrieve_trait_info): Convert "target_string"
to a str, otherwise an exception will be thrown if it's None.
|
|
* wqflask/base/trait.py: Apply pep8.
* wqflask/utility/authentication_tools.py: Ditto.
|