-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Audits #68
Audits #68
Conversation
sbreiff
commented
Jul 18, 2018
- added 4 checks that are meant to replace fourfront audits
- fixed url for foursight dev in getting_started.md
Updated branch from master
chalicelib/checks/wrangler_checks.py
Outdated
cell_line_types = ["primary cell", "primary cell line", "immortalized cell line", | ||
"in vitro differentiated cells", "induced pluripotent stem cell line", | ||
"stem cell", "stem cell derived cell line"] | ||
biosources = ff_utils.search_metadata('search/?type=Biosource', ff_env=connection.ff_env, page_limit=200) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this be search/?type=Biosource&frame=object
? Would be a bit more performant and it doesn't look like the check uses any embedded fields
chalicelib/checks/wrangler_checks.py
Outdated
''' | ||
check = init_check_res(connection, 'external_expsets_without_pub') | ||
|
||
ext = ff_utils.search_metadata('search/?award.project=External&type=ExperimentSet', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same frame=object
comment as above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As an aside, page_limit=200
makes some pretty big search requests that have to go through Fourfront. I would recommend setting that value lower, especially if you're getting embedded experiment sets
chalicelib/checks/wrangler_checks.py
Outdated
''' | ||
check = init_check_res(connection, 'expset_opfsets_unique_titles') | ||
|
||
opf_expsets = ff_utils.search_metadata('search/?type=ExperimentSet&other_processed_files.files.uuid%21=No+value', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For this and the next check, page_limit
on the search could be set to 20 or something else low