Age | Commit message (Collapse) | Author |
|
|
|
* wqflask/wqflask/user_manager.py:
Remove some stale comments.
(imports): Apply pep-8 and remove unused imports.
|
|
displayed in the "view collection" page
* wqflask/wqflask/templates/collections/view.html - Apparently the
correct way to check if a value can be cast to a float in Jinja2 is to
say "value != 0"; it previous said "value > 0" which exlcuded all
negative values
|
|
changed the text for the GN1 link button
* wqflask/wqflask/templates/show_trait_details.html - Changed Add button
class to btn-success (to make it green like the other Add buttons) and
changed text from "View in GN1" to "Go to GN1"
|
|
changing the image they're referencing
* wqflask/wqflask/templates/base.html - Added CSS changing the table
sort arrow image directly into
base.html because it requires using url_for to reference the images in
the GUIX profile (so it needs to be in the template and can't just be
added to a CSS file)
|
|
* wqflask/wqflask/collect.py (process_traits): Check whether the
passed "unprocessed_traits" is a byte otherwise an error is generated
when bytes(the default) are passed.
|
|
* wqflask/tests/wqflask/test_collect.py: Add it.
|
|
the results on the initial page aren't lost
* wqflask/wqflask/static/new/javascript/search_results.js - Changed the
add to collection method name to "add_to_collection" because it was
previously "add" which is a terrible method name
* wqflask/wqflask/templates/collections/add.html - Added target="_blank"
to the add to collection form so it will open in a new tab/window
|
|
* wqflask/tests/base/test_trait.py: Add app_context to test class,
otherwise an error related to working outside the app_context is
generated. Adding the app_context is required when mocking "Flask.g".
|
|
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)"
|
|
* wqflask/tests/base/test_trait.py
(test_retrieve_trait_info_with_non_empty_lrs): Check
trait.LRS_score_repr is set correctly if trait.lrs is None.
(test_retrieve_trait_info_with_empty_lrs_field): Check
trait.LRS_score_repr and test_trait.LRS_location_repr is set correctly
if trait.lrs is None.
(test_retrieve_trait_info_with_empty_chr_field): Check
test_trait.LRS_score_repr and test_trait.LRS_location_repr is set
correctly if trait.locus is None.
|
|
* wqflask/tests/base/test_data_set.py (imports): Run:
`find . -type f -name "*py" -print0 | xargs -0 sed -i \
"s|import mock|from unittest import mock|g"`
* wqflask/tests/base/test_trait.py(imports): Ditto.
* wqflask/tests/utility/test_authentication_tools.py(imports): Ditto.
* wqflask/tests/utility/test_hmac.py(imports). Ditto.
* wqflask/tests/wqflask/api/test_gen_menu.py: Ditto
|
|
* wqflask/tests/base/test_trait.py: Remove decode function call which
is no longer supported in python3.
|
|
* wqflask/wqflask/db_info.py: Run pep8 through buffers. Fixes
inconsistent use of tabs and spaces in indentation which trips up
unittests.
|
|
* wqflask/wqflask/db_info.py:
Run: 2to3-3.8 -w wqflask/wqflask/db_info.py
|
|
* wqflask/wqflask/db_info.py: Replace ^M with proper unix line
endings. See <http://ergoemacs.org/emacs/emacs_line_ending_char.html>
|
|
|
|
* wqflask/wqflask/static/new/javascript/initialize_show_trait_tables.js
- attribute cells' alignment and column names were wrong because they were referring to an attribute offset by 1 in the attribute list
-
|
|
* wqflask/wqflask/static/new/javascript/show_trait.js - Changed
edit_data_change to use DataTables API to fetch the list of names,
values, and (where applicable) standard errors in a way that just looks
at those columns (previously it iterated through all rows).
block_by_index was also changed to look just at columns
|
|
efficient
* wqflask/wqflask/show_trait/show_trait.py - Add an "if" statement to
avoid unnecessarily running some code and changed a list to a set for
the purposes of searching if it contains an element
|
|
* wqflask/wqflask/templates/mapping_results.html - aligned score,
position, additive effect, and dominance column headers to the right
|
|
* wqflask/wqflask/__init__.py: Move imports from views.py to
"__init__.py". Fixes failing build.
* wqflask/wqflask/views.py: Ditto.
|
|
* wqflask/tests/base/test_trait.py: In python2 it's necessary to
decode the utf-8 string.
|
|
|
|
there was some mistake/error during registration
* wqflask/wqflask/user_login.py - Previously it would still save a user
even if there was some problem like their e-mail already being
associated with an account. It would display the error, but still run
save_user. I fixed this so that it no longer saves the user if there are
any errors
|
|
color to match the high color in other tables
* wqflask/wqflask/static/new/css/marker_regression.css - changed
background color when selecting a row (i.e. "highlight color")
|
|
header on the mapping results page to include the mapping scale
* wqflask/wqflask/marker_regression/run_mapping.py - changed how
"display_pos" is defined for marker obs
* wqflask/wqflask/templates/mapping_results.html - Fixed issues with
last commit and changed position header name to include mapping scale
|
|
results and also changes the -log(p) to use an endash instead
* wqflask/wqflask/marker_regression/run_mapping.py - added "display_pos"
to each marker object that is a combination of Chr and Mb/cM
* wqflask/wqflask/templates/mapping_results.html - Included
"display_pos" in results table and replaced -log(p) with using an endash
|
|
use the DataTables API to pull sample values
* wqflask/wqflask/static/new/javascript/show_trait.js - Changed the way
sample name/val/var are pulled from the table to instead use DataTables'
API + removed an unused event listener
* wqflask/wqflask/static/new/javascript/initialize_show_trait_tables.js
- added "var" before a defined variable for consistency
|
|
|
|
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/tests/utility/test_hmac.py: Mock app.config's
"SECRET_HMAC_CODE" value.
|
|
Same as:
https://github.com/genenetwork/genenetwork2/pull/422/commits/46443ec8d2cdfd7c60358a889d90a90e4f7daaf4
|
|
* wqflask/tests/utility/test_authentication_tools.py
test_check_resource_availability_non_default_mask): Mock flask's
global 'g' variable properly.
* wqflask/tests/base/test_trait.py: Ditto.
* wqflask/tests/utility/test_authentication_tools.py: Ditto.
|
|
* wqflask/base/trait.py: Apply pep8.
* wqflask/utility/authentication_tools.py: Ditto.
|
|
|
|
* wqflask/utility/hmac.py: Apply pep8 and fix typo.
|
|
* wqflask/tests/utility/test_hmac.py: New tests.
|
|
SampleList instead of calling the method
* wqflask/wqflask/show_trait/show_trait.py - see above
* wqflask/wqflask/templates/show_trait.html - see above + removed some
commented out code
|
|
that when converted to a dict (which is needed to use it in the
javascript)
* wqflask/wqflask/show_trait/SampleList.py - Directly set se_exists as a
property of SampleList instead of requiring a method be called; I could
have set self.se_exists by calling the se_exists method, but that would
have resulted in the convoluted line "self.se_exists = self.se_exists()"
and the se_exists method only consisted of one line anyways
|
|
|
|
* wqflask/tests/base/test_trait.py: New tests.
|
|
* wqflask/utility/authentication_tools.py: Apply pep8 formatting to
file.
|