{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":133414800,"defaultBranch":"master","name":"CRISPResso2","ownerLogin":"pinellolab","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2018-05-14T20:01:18.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/30343453?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1713806703.0","currentOid":""},"activityList":{"items":[{"before":"5638a1f6ffa973231f23422e9c757fa8cd4af7cc","after":"1bc9e906f0ded81f80761d1ec375ee50a4f882a9","ref":"refs/heads/master","pushedAt":"2024-04-26T22:26:27.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Bump version to 2.3.1 and change default CRISPRessoPooled behavior to change in 2.3.2 (#428)","shortMessageHtmlLink":"Bump version to 2.3.1 and change default CRISPRessoPooled behavior to…"}},{"before":"d6011f29db16d8fc1c1e7222457b7f9a1f671de6","after":"5638a1f6ffa973231f23422e9c757fa8cd4af7cc","ref":"refs/heads/master","pushedAt":"2024-04-25T00:00:43.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Spelling fixes","shortMessageHtmlLink":"Spelling fixes"}},{"before":"f4858a30c43374f54058b3ad9c1e965e1ab7fb46","after":"d6011f29db16d8fc1c1e7222457b7f9a1f671de6","ref":"refs/heads/master","pushedAt":"2024-04-24T15:33:53.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Extract `jinja_partials` and fix CRISPRessoPooled fastp errors (#425)\n\n* Updated README (#64)\r\n\r\n* Updating README to fix argument, email, and formatting\r\n\r\n* removing superfluous files\r\n\r\n* Add link to CRISPRessoPro, move CRISPRessoPro section to end, and fix JSON formatting\r\n\r\n* Remove link to CRISPRessoPro\r\n\r\n* Replace Docker badge with link to tags\r\n\r\n* Add bullet points to Guardrails section and improve formatting\r\n\r\n* Fix typo and removed colons from guardrails\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Extract jinja_partials (#65)\r\n\r\n* Extract jinja_partials code\r\n\r\n* Remove Plotly dependency from setup.py\r\n\r\n* Fix CRISPRessoPooled flash errors (#68)\r\n\r\n* Fix replacing flash intermediate files with fastp intermediate files\r\n\r\nThis also moves where the files are added to `files_to_remove` up to\r\nnear where they are created.\r\n\r\n* Update to run test branch with paired end Pooled test\r\n\r\n* Add pooled-paired-sim test to integration tests\r\n\r\n* Replace flash and trimmomatic with fastp and remove plotly from Github Actions environment\r\n\r\n* Change test branch back to master\r\n\r\n---------\r\n\r\nCo-authored-by: Trevor Martin <60452953+trevormartinj7@users.noreply.github.com>","shortMessageHtmlLink":"Extract jinja_partials and fix CRISPRessoPooled fastp errors (#425)"}},{"before":"c3dbff0fccd44b0b1a9c246dd2aa629ddc515787","after":"f4858a30c43374f54058b3ad9c1e965e1ab7fb46","ref":"refs/heads/master","pushedAt":"2024-04-23T23:00:28.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Updated README (#64) (#424)\n\n* Updating README to fix argument, email, and formatting\r\n\r\n* removing superfluous files\r\n\r\n* Add link to CRISPRessoPro, move CRISPRessoPro section to end, and fix JSON formatting\r\n\r\n* Remove link to CRISPRessoPro\r\n\r\n* Replace Docker badge with link to tags\r\n\r\n* Add bullet points to Guardrails section and improve formatting\r\n\r\n* Fix typo and removed colons from guardrails\r\n\r\n---------\r\n\r\nCo-authored-by: Trevor Martin <60452953+trevormartinj7@users.noreply.github.com>","shortMessageHtmlLink":"Updated README (#64) (#424)"}},{"before":"9b9b1e71a7d9c183f4c82c0d0aed0053266caf95","after":null,"ref":"refs/heads/fix_pooled_debug","pushedAt":"2024-04-22T17:25:03.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"}},{"before":"20903c14877e5166b1b8a7b50b8fcab450ea3ca6","after":"c3dbff0fccd44b0b1a9c246dd2aa629ddc515787","ref":"refs/heads/master","pushedAt":"2024-04-22T17:24:59.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Update CRISPRessoPooledCORE.py (#423)\n\nFix bug in error reporting if duplicate names are present","shortMessageHtmlLink":"Update CRISPRessoPooledCORE.py (#423)"}},{"before":"20903c14877e5166b1b8a7b50b8fcab450ea3ca6","after":"9b9b1e71a7d9c183f4c82c0d0aed0053266caf95","ref":"refs/heads/fix_pooled_debug","pushedAt":"2024-04-22T17:18:16.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Update CRISPRessoPooledCORE.py\n\nFix bug in error reporting if duplicate names are present","shortMessageHtmlLink":"Update CRISPRessoPooledCORE.py"}},{"before":null,"after":"20903c14877e5166b1b8a7b50b8fcab450ea3ca6","ref":"refs/heads/fix_pooled_debug","pushedAt":"2024-04-22T17:17:52.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Remove extra imports from CRISPRessoCore (#67) (#422)","shortMessageHtmlLink":"Remove extra imports from CRISPRessoCore (#67) (#422)"}},{"before":"4aae57e5be475cd717792265bee36a71a99425de","after":"20903c14877e5166b1b8a7b50b8fcab450ea3ca6","ref":"refs/heads/master","pushedAt":"2024-04-18T22:55:39.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Remove extra imports from CRISPRessoCore (#67) (#422)","shortMessageHtmlLink":"Remove extra imports from CRISPRessoCore (#67) (#422)"}},{"before":"768c3c05bf1786a2a32e135b6e145cd6503c3db1","after":"4aae57e5be475cd717792265bee36a71a99425de","ref":"refs/heads/master","pushedAt":"2024-04-18T16:00:19.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Cole/refactor jinja undefined (#66) (#421)\n\n* Replace Jinja2 PackageLoader with FileSystemLoader\r\n\r\nThe PackageLoader doesn't work with a fairly recent version of Jinja2 (3.0.1)\r\nand Python 3.9. Replacing with FileSystemLoader work with the older version and\r\nthe latest version.\r\n\r\n* Fix undefined variable `amplicon_name` in report template\r\n\r\n* Refactor logging Jinja2 undefined variable warnings\r\n\r\n* Revert plot_11a update\r\n\r\n* Update intedration test branch\r\n\r\n* Update jinja to warn on undefined but not fail. Fix all undefined warnings\r\n\r\n* Fix github integration tests ref\r\n\r\n* One more undefined variable\r\n\r\n---------\r\n\r\nCo-authored-by: Samuel Nichols ","shortMessageHtmlLink":"Cole/refactor jinja undefined (#66) (#421)"}},{"before":"7e18f08cc1ac5f247a0fd1bbb394ccd9b0a07c2e","after":"768c3c05bf1786a2a32e135b6e145cd6503c3db1","ref":"refs/heads/master","pushedAt":"2024-04-09T23:30:10.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Fix Jinja2 undefined variables (#63) (#417)\n\n* Replace Jinja2 PackageLoader with FileSystemLoader\r\n\r\nThe PackageLoader doesn't work with a fairly recent version of Jinja2 (3.0.1)\r\nand Python 3.9. Replacing with FileSystemLoader work with the older version and\r\nthe latest version.\r\n\r\n* Fix undefined variable `amplicon_name` in report template\r\n\r\n* Revert plot_11a update\r\n\r\n* Update intedration test branch\r\n\r\n* Update branch for integration tests","shortMessageHtmlLink":"Fix Jinja2 undefined variables (#63) (#417)"}},{"before":"235dc29c0cd0fcca2e999148d4660acf00b07221","after":"7e18f08cc1ac5f247a0fd1bbb394ccd9b0a07c2e","ref":"refs/heads/master","pushedAt":"2024-04-05T22:36:41.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"fix: change all U+00A0 to U+0020 (#400)","shortMessageHtmlLink":"fix: change all U+00A0 to U+0020 (#400)"}},{"before":"fa03d16ddd10e3163c035bd1a8d18416f7da35a8","after":"235dc29c0cd0fcca2e999148d4660acf00b07221","ref":"refs/heads/master","pushedAt":"2024-04-05T22:36:17.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Fastp, args as data, guardrails, and PE fix (#415)\n\n* Change CRISPResso_status.txt format to JSON (#46)\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* add json read for status file\r\n\r\n* changed Formatter to json format\r\n\r\n* fixed json access variable name: message\r\n\r\n* changed perentage_complete to numeric\r\n\r\n* changed status file to .json\r\n\r\n* Create integration_tests.yml\r\n\r\n* Simplify name\r\n\r\n* CRISPRESSO2_DIR environment variable\r\n\r\n* Up one dir\r\n\r\n* ls workspace\r\n\r\n* Install CRISPResso and ydiff\r\n\r\n* Clone repo instead of checkout\r\n\r\n* submodule\r\n\r\n* ls\r\n\r\n* CRISPResso2_copy\r\n\r\n* ls\r\n\r\n* Update env\r\n\r\n* Simplify\r\n\r\n* Pull from githubactions branch\r\n\r\n* Pull githubactions repo\r\n\r\n* Checkout githubactions\r\n\r\n* Run tests individually\r\n\r\n* Pin plotly version\r\n\r\n* Run all tests even if one fails\r\n\r\n* Test on another branch\r\n\r\n* Switch branch with token\r\n\r\n* Update integration_tests.yml\r\n\r\n* New makefile commands\r\n\r\n* changed file to .json\r\n\r\n* changed status to json file\r\n\r\n* Make JSON human readable by adding new lines\r\n\r\n* GitHub actions integration tests (#48)\r\n\r\n* GitHub actions clean (#40)\r\n\r\n* Create pytest.yml\r\n\r\n* Create pylint.yml\r\n\r\n* Create .pylintrc\r\n\r\n* Create test_env.yml\r\n\r\n* Full path\r\n\r\n* Remove conda install\r\n\r\n* Replace path\r\n\r\n* Pytest tests\r\n\r\n* pip -e\r\n\r\n* Create integration_tests.yml\r\n\r\n* Simplify name\r\n\r\n* CRISPRESSO2_DIR environment variable\r\n\r\n* Up one dir\r\n\r\n* ls workspace\r\n\r\n* Install CRISPResso and ydiff\r\n\r\n* Clone repo instead of checkout\r\n\r\n* submodule\r\n\r\n* ls\r\n\r\n* CRISPResso2_copy\r\n\r\n* ls\r\n\r\n* Update env\r\n\r\n* Simplify\r\n\r\n* Pull from githubactions branch\r\n\r\n* Pull githubactions repo\r\n\r\n* Checkout githubactions\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Run tests individually\r\n\r\n* Pin plotly version\r\n\r\n* Run all tests even if one fails\r\n\r\n* Test on another branch\r\n\r\n* Switch branch with token\r\n\r\n* Update integration_tests.yml\r\n\r\n* Introduce pandas sorting in CRISPRessoCompare (#47)\r\n\r\n* New makefile commands\r\n\r\n* Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\r\n\r\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* On push no branches\r\n\r\n* On push no branches\r\n\r\n* All in one file\r\n\r\n* Fix yml errors\r\n\r\n* Rename jobs\r\n\r\n* Remove old workflow files\r\n\r\n* Remove paths\r\n\r\n* Run jobs in parallel\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n* Move read filtering to after merging in CRISPResso (#39)\r\n\r\n* Move read filtering to after merging\r\n\r\nThis is in an effort to be consistent with the behavior and results of\r\nCRISPRessoPooled.\r\n\r\n* Properly assign the correct file names for read filtering\r\n\r\n* Add space around operators\r\n\r\n* GitHub actions on pr (#51)\r\n\r\n* Run integration tests on pull_request\r\n\r\n* Run pytest on pull_request\r\n\r\n* Run pylint on pull_request\r\n\r\n* Run tests on PR only when opening PR (#53)\r\n\r\n* Update reports (#52)\r\n\r\n* Update report changes\r\n\r\n* Switch branch of integration test repo\r\n\r\n* Remove extraneous `crispresso_data_path`\r\n\r\n* Point integration tests back to master\r\n\r\n* point to test branch\r\n\r\n* pointed CI config to testing branch\r\n\r\n* Update integration_tests.yml\r\n\r\npoint to master\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\nCo-authored-by: Samuel Nichols \r\n\r\n* Trevor/fastp integration (#50)\r\n\r\n* Update check_program to check versions and create check_fastq function\r\n\r\n* Update fastq arg, implement fastp in get_most_frequent_reads\r\n\r\n* Bump version to 2.3.0\r\n\r\n* Deprecate Flash and Trimmomatic parameters, and update fastp params\r\n\r\n* Update guess_amplicons and guess_guides to remove max_paired_end_reads_overlap\r\n\r\n* Implement trimming of single end reads\r\n\r\n* Merge (and trim) reads in CRISPRessoCORE with fastp\r\n\r\n* Modify error handling to account for fastp errors\r\n\r\n* Replace flash and trimmomatic with fastp in Docker dependencies\r\n\r\n* Update LICENSE.txt with fastp info\r\n\r\n* Remove min and max amplicon length (no longer needed)\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Implement trimming with fastp in CRISPRessoPooled\r\n\r\n* Implemend merging (and trimming) with fastp in CRISPRessoPooled\r\n\r\n* Fixed minor fastp errors\r\n\r\n* Move read filtering to after merging in CRISPResso (#39)\r\n\r\n* Move read filtering to after merging\r\n\r\nThis is in an effort to be consistent with the behavior and results of\r\nCRISPRessoPooled.\r\n\r\n* Properly assign the correct file names for read filtering\r\n\r\n* Add space around operators\r\n\r\n* GitHub actions on pr (#51)\r\n\r\n* Run integration tests on pull_request\r\n\r\n* Run pytest on pull_request\r\n\r\n* Run pylint on pull_request\r\n\r\n* Run tests on PR only when opening PR (#53)\r\n\r\n* Update reports (#52)\r\n\r\n* Update report changes\r\n\r\n* Switch branch of integration test repo\r\n\r\n* Remove extraneous `crispresso_data_path`\r\n\r\n* Point integration tests back to master\r\n\r\n* Update where the test point to\r\n\r\n* Fix 'Prime-edited' key not found (#32)\r\n\r\n* Move 'Prime-edited' amplicon name check\r\n\r\nBy moving this, it will check if there is an amplicon named\r\n'Prime-edited' (which is a reserved name) even if the\r\n`prime_editing_pegRNA_extension_seq` parameter is empty.\r\n\r\n* Only search for scaffold integration when pegRNA extension seq is provided\r\n\r\n* Remove spaces at the end of lines\r\n\r\n* Docker size (#49)\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* GitHub actions integration tests (#48)\r\n\r\n* GitHub actions clean (#40)\r\n\r\n* Create pytest.yml\r\n\r\n* Create pylint.yml\r\n\r\n* Create .pylintrc\r\n\r\n* Create test_env.yml\r\n\r\n* Full path\r\n\r\n* Remove conda install\r\n\r\n* Replace path\r\n\r\n* Pytest tests\r\n\r\n* pip -e\r\n\r\n* Create integration_tests.yml\r\n\r\n* Simplify name\r\n\r\n* CRISPRESSO2_DIR environment variable\r\n\r\n* Up one dir\r\n\r\n* ls workspace\r\n\r\n* Install CRISPResso and ydiff\r\n\r\n* Clone repo instead of checkout\r\n\r\n* submodule\r\n\r\n* ls\r\n\r\n* CRISPResso2_copy\r\n\r\n* ls\r\n\r\n* Update env\r\n\r\n* Simplify\r\n\r\n* Pull from githubactions branch\r\n\r\n* Pull githubactions repo\r\n\r\n* Checkout githubactions\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Run tests individually\r\n\r\n* Pin plotly version\r\n\r\n* Run all tests even if one fails\r\n\r\n* Test on another branch\r\n\r\n* Switch branch with token\r\n\r\n* Update integration_tests.yml\r\n\r\n* Introduce pandas sorting in CRISPRessoCompare (#47)\r\n\r\n* New makefile commands\r\n\r\n* Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\r\n\r\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* On push no branches\r\n\r\n* On push no branches\r\n\r\n* All in one file\r\n\r\n* Fix yml errors\r\n\r\n* Rename jobs\r\n\r\n* Remove old workflow files\r\n\r\n* Remove paths\r\n\r\n* Run jobs in parallel\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n* 3.4->2.08\r\n\r\n* Put ttf-mscorefonts-installer back above apt-get clean\r\n\r\n* restore slash, replace fastp with trimmomatic and flash, add autoremove step\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n* initial readme modifications\r\n\r\n* Updated readme to remove deprecated commands, updated help text to reflect new version and fastp\r\n\r\n* Pointing test branch back at master\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Samuel Nichols \r\n\r\n* Guardrails clean history (#34)\r\n\r\n* Include guardrail functions\r\n\r\n* Add CRISPRessoReports subtree\r\n\r\n* Refactor to use CRISPRessoReports module\r\n\r\n* Include guardrail functions\r\n\r\n* Functional guardrails, needs reports update\r\n\r\n* Add guardrail partial\r\n\r\n* fix guardrials partial\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* GitHub actions integration tests (#48)\r\n\r\n* GitHub actions clean (#40)\r\n\r\n* Create pytest.yml\r\n\r\n* Create pylint.yml\r\n\r\n* Create .pylintrc\r\n\r\n* Create test_env.yml\r\n\r\n* Full path\r\n\r\n* Remove conda install\r\n\r\n* Replace path\r\n\r\n* Pytest tests\r\n\r\n* pip -e\r\n\r\n* Create integration_tests.yml\r\n\r\n* Simplify name\r\n\r\n* CRISPRESSO2_DIR environment variable\r\n\r\n* Up one dir\r\n\r\n* ls workspace\r\n\r\n* Install CRISPResso and ydiff\r\n\r\n* Clone repo instead of checkout\r\n\r\n* submodule\r\n\r\n* ls\r\n\r\n* CRISPResso2_copy\r\n\r\n* ls\r\n\r\n* Update env\r\n\r\n* Simplify\r\n\r\n* Pull from githubactions branch\r\n\r\n* Pull githubactions repo\r\n\r\n* Checkout githubactions\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Run tests individually\r\n\r\n* Pin plotly version\r\n\r\n* Run all tests even if one fails\r\n\r\n* Test on another branch\r\n\r\n* Switch branch with token\r\n\r\n* Update integration_tests.yml\r\n\r\n* Introduce pandas sorting in CRISPRessoCompare (#47)\r\n\r\n* New makefile commands\r\n\r\n* Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\r\n\r\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* On push no branches\r\n\r\n* On push no branches\r\n\r\n* All in one file\r\n\r\n* Fix yml errors\r\n\r\n* Rename jobs\r\n\r\n* Remove old workflow files\r\n\r\n* Remove paths\r\n\r\n* Run jobs in parallel\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n* Update C cythonized files\r\n\r\n* Add exact numbers to guardrails printouts\r\n\r\n* Remove extraneous whitespace from CRISPRessoCOREResources.pyx\r\n\r\n* Fix calculation of `total_mods` from being negative\r\n\r\nThe issue was that `all_deletion_coordinates` just tells you how many deletions\r\nwere present, but not how long the deletion is.\r\n\r\n* Changes to message\r\n\r\n* Remove old tag\r\n\r\n* Point tests at guardrails\r\n\r\n* Restore C2 pro check\r\n\r\n* Save message with guardrail name\r\n\r\n* Point tests repo at master\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\n\r\n* Fix case sensitivity in Prime Editing mode (#54)\r\n\r\n* Move read filtering to after merging in CRISPResso (#39)\r\n\r\n* Move read filtering to after merging\r\n\r\nThis is in an effort to be consistent with the behavior and results of\r\nCRISPRessoPooled.\r\n\r\n* Properly assign the correct file names for read filtering\r\n\r\n* Add space around operators\r\n\r\n* GitHub actions on pr (#51)\r\n\r\n* Run integration tests on pull_request\r\n\r\n* Run pytest on pull_request\r\n\r\n* Run pylint on pull_request\r\n\r\n* Run tests on PR only when opening PR (#53)\r\n\r\n* Update reports (#52)\r\n\r\n* Update report changes\r\n\r\n* Switch branch of integration test repo\r\n\r\n* Remove extraneous `crispresso_data_path`\r\n\r\n* Point integration tests back to master\r\n\r\n* Make all amplicons in amplicon_seq_arr uppercase\r\n\r\nThis fixes https://github.com/pinellolab/CRISPResso2/issues/396\r\n\r\n* Allow RNA values to be provided for prime_editing_pegRNA_scaffold_seq\r\n\r\n* Fix 'Prime-edited' key not found (#32)\r\n\r\n* Move 'Prime-edited' amplicon name check\r\n\r\nBy moving this, it will check if there is an amplicon named\r\n'Prime-edited' (which is a reserved name) even if the\r\n`prime_editing_pegRNA_extension_seq` parameter is empty.\r\n\r\n* Only search for scaffold integration when pegRNA extension seq is provided\r\n\r\n* Remove spaces at the end of lines\r\n\r\n* Docker size (#49)\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* GitHub actions integration tests (#48)\r\n\r\n* GitHub actions clean (#40)\r\n\r\n* Create pytest.yml\r\n\r\n* Create pylint.yml\r\n\r\n* Create .pylintrc\r\n\r\n* Create test_env.yml\r\n\r\n* Full path\r\n\r\n* Remove conda install\r\n\r\n* Replace path\r\n\r\n* Pytest tests\r\n\r\n* pip -e\r\n\r\n* Create integration_tests.yml\r\n\r\n* Simplify name\r\n\r\n* CRISPRESSO2_DIR environment variable\r\n\r\n* Up one dir\r\n\r\n* ls workspace\r\n\r\n* Install CRISPResso and ydiff\r\n\r\n* Clone repo instead of checkout\r\n\r\n* submodule\r\n\r\n* ls\r\n\r\n* CRISPResso2_copy\r\n\r\n* ls\r\n\r\n* Update env\r\n\r\n* Simplify\r\n\r\n* Pull from githubactions branch\r\n\r\n* Pull githubactions repo\r\n\r\n* Checkout githubactions\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Run tests individually\r\n\r\n* Pin plotly version\r\n\r\n* Run all tests even if one fails\r\n\r\n* Test on another branch\r\n\r\n* Switch branch with token\r\n\r\n* Update integration_tests.yml\r\n\r\n* Introduce pandas sorting in CRISPRessoCompare (#47)\r\n\r\n* New makefile commands\r\n\r\n* Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\r\n\r\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* On push no branches\r\n\r\n* On push no branches\r\n\r\n* All in one file\r\n\r\n* Fix yml errors\r\n\r\n* Rename jobs\r\n\r\n* Remove old workflow files\r\n\r\n* Remove paths\r\n\r\n* Run jobs in parallel\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n* 3.4->2.08\r\n\r\n* Put ttf-mscorefonts-installer back above apt-get clean\r\n\r\n* restore slash, replace fastp with trimmomatic and flash, add autoremove step\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n* Guardrails clean history (#34)\r\n\r\n* Include guardrail functions\r\n\r\n* Add CRISPRessoReports subtree\r\n\r\n* Refactor to use CRISPRessoReports module\r\n\r\n* Include guardrail functions\r\n\r\n* Functional guardrails, needs reports update\r\n\r\n* Add guardrail partial\r\n\r\n* fix guardrials partial\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* GitHub actions integration tests (#48)\r\n\r\n* GitHub actions clean (#40)\r\n\r\n* Create pytest.yml\r\n\r\n* Create pylint.yml\r\n\r\n* Create .pylintrc\r\n\r\n* Create test_env.yml\r\n\r\n* Full path\r\n\r\n* Remove conda install\r\n\r\n* Replace path\r\n\r\n* Pytest tests\r\n\r\n* pip -e\r\n\r\n* Create integration_tests.yml\r\n\r\n* Simplify name\r\n\r\n* CRISPRESSO2_DIR environment variable\r\n\r\n* Up one dir\r\n\r\n* ls workspace\r\n\r\n* Install CRISPResso and ydiff\r\n\r\n* Clone repo instead of checkout\r\n\r\n* submodule\r\n\r\n* ls\r\n\r\n* CRISPResso2_copy\r\n\r\n* ls\r\n\r\n* Update env\r\n\r\n* Simplify\r\n\r\n* Pull from githubactions branch\r\n\r\n* Pull githubactions repo\r\n\r\n* Checkout githubactions\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Run tests individually\r\n\r\n* Pin plotly version\r\n\r\n* Run all tests even if one fails\r\n\r\n* Test on another branch\r\n\r\n* Switch branch with token\r\n\r\n* Update integration_tests.yml\r\n\r\n* Introduce pandas sorting in CRISPRessoCompare (#47)\r\n\r\n* New makefile commands\r\n\r\n* Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\r\n\r\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* On push no branches\r\n\r\n* On push no branches\r\n\r\n* All in one file\r\n\r\n* Fix yml errors\r\n\r\n* Rename jobs\r\n\r\n* Remove old workflow files\r\n\r\n* Remove paths\r\n\r\n* Run jobs in parallel\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n* Update C cythonized files\r\n\r\n* Add exact numbers to guardrails printouts\r\n\r\n* Remove extraneous whitespace from CRISPRessoCOREResources.pyx\r\n\r\n* Fix calculation of `total_mods` from being negative\r\n\r\nThe issue was that `all_deletion_coordinates` just tells you how many deletions\r\nwere present, but not how long the deletion is.\r\n\r\n* Changes to message\r\n\r\n* Remove old tag\r\n\r\n* Point tests at guardrails\r\n\r\n* Restore C2 pro check\r\n\r\n* Save message with guardrail name\r\n\r\n* Point tests repo at master\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\n\r\n---------\r\n\r\nCo-authored-by: Samuel Nichols \r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: trevormartinj7 \r\n\r\n* Batch d3 clean (#55)\r\n\r\n* imports C2Pro plots if available\r\n\r\n* added --use_matplotlib flag\r\n\r\n* added C2Pro\r\nmatched api funciton signatures\r\n\r\n* added api args for plotly\r\n\r\n* added **kwargs\r\n\r\n* renamed config to custom_config, more specificity\r\n\r\n* added backend flag for plotly kaleido\r\n\r\n* added pro_installed boolean for templates, added plotly dependency to report templates\r\n\r\n* Squashed commit of the following:\r\n\r\ncommit c909ea3b34e87ce637e00dac075d2bb2f8bfb954\r\nAuthor: McKay \r\nDate: Thu Feb 15 15:55:23 2024 -0700\r\n\r\n added plotly dependency for pro\r\n\r\ncommit 76b3601f6a0144f100266153f1c999e0c5de65de\r\nAuthor: Samuel Nichols \r\nDate: Fri Jan 12 09:56:19 2024 -0700\r\n\r\n Squashed commit of the following:\r\n\r\n commit 603f2eff9d1aa21ae95f3e134da303b8018d3a33\r\n Author: Samuel Nichols \r\n Date: Fri Jan 12 09:48:20 2024 -0700\r\n\r\n fix guardrials partial\r\n\r\n commit 22fc03183a8070c30dfb74d5c23575ac19019855\r\n Author: Samuel Nichols \r\n Date: Fri Jan 12 08:54:01 2024 -0700\r\n\r\n Add guardrail partial\r\n\r\n commit e55f6b21972b578261bc5a864ce1d653d98f9e34\r\n Author: Samuel Nichols \r\n Date: Mon Jan 8 07:50:59 2024 -0700\r\n\r\n Functional guardrails, needs reports update\r\n\r\n commit 6e968e9699ed59a47d88191d03768e042d8b60a4\r\n Merge: 32b49685 e948ce10\r\n Author: Samuel Nichols \r\n Date: Mon Dec 18 13:34:36 2023 -0700\r\n\r\n Merge branch 'guardrails-clean-history' of https://github.com/edilytics/CRISPResso2 into guardrails-clean-history\r\n\r\n commit 32b49685da320501dad2b0ebbb57887b66220ba8\r\n Author: Samuel Nichols \r\n Date: Fri Dec 15 15:27:04 2023 -0700\r\n\r\n Include guardrail functions\r\n\r\n commit 4e309cf6f732565d635de3d4c5d074ada3027e2d\r\n Author: Cole Lyman \r\n Date: Mon Dec 18 10:51:55 2023 -0700\r\n\r\n Refactor to use CRISPRessoReports module\r\n\r\n commit e648dc087c0055bc5d2fca13c64071a371dea941\r\n Author: Cole Lyman \r\n Date: Mon Dec 18 10:51:11 2023 -0700\r\n\r\n Add CRISPRessoReports subtree\r\n\r\n commit e948ce107ebb0d1d99010ed12e937f34b5e607d4\r\n Author: Samuel Nichols \r\n Date: Fri Dec 15 15:27:04 2023 -0700\r\n\r\n Include guardrail functions\r\n\r\n commit d33c748871a625facfe8d792e29c77ab9779138f\r\n Author: Kendell Clement \r\n Date: Tue Nov 7 16:31:06 2023 -0700\r\n\r\n Include parameter --assign_ambiguous_alignments_to_first_reference in readme\r\n\r\n commit a1435f7f491a6a61434f3051e39f39a4c9bf1edc\r\n Author: Kendell Clement \r\n Date: Wed Oct 11 17:17:30 2023 -0600\r\n\r\n Enable quantification by sgRNA (#348)\r\n\r\n This PR includes:\r\n - storing the sgRNA-specific editing locations in the crispresso2_info object. Previously, each amplicon would record the indices of quantification windows across the guide, but not for individual guides. This stores the information for each guide in crispresso2_info['results']['refs'][reference_name]['sgRNA_include_idxs']\r\n - a script (count_sgRNA_specific_edits.py) to parse through an allele table output from a completed CRISPResso run (`--write_detailed_allele_table` flag required) to count edits in each sgRNA separately.\r\n\r\n I don't have a good double-edited sample handy, but it can be run on the demo HDR data [hdr.fastq.gz](http://crispresso.pinellolab.org/static/demo/hdr.fastq.gz) using the command:\r\n\r\n ```\r\n\r\n CRISPResso -r1 hdr.fastq.gz -a acatttgcttctgacacaactgtgttcactagcaacctcaaacagacaccatggtgcatctgactcctgTggagaagtctgccgttactgccctgtggggcaaggtgaacgtggatgaagttggtggtgaggccctgggcaggttggtatcaaggtta -e acatttgcttctgacacaactgtgttcactagcaacctcaaacagacaccatggtgcaCctgactccGgaggagaagtctgccgttactgcGctgtggggcaaggtgaacgtggatgaagttggtggtgaggccctgggcaggttggtatcaaggtta -c atggtgcatctgactcctgTggagaagtctgccgttactgccctgtggggcaaggtgaacgtggatgaagttggtggtgaggccctgggcag -g TGCACCATGGTGTCTGTTTG,GATGAAGTTGGTGGTGAGGCCC --write_detailed_allele_table -n hdr3 -p max -gn guide1,guide2\r\n ```\r\n\r\n ```\r\n python CRISPResso2/scripts/count_sgRNA_specific_edits.py -f CRISPResso_on_hdr3\r\n ```\r\n\r\n This produces:\r\n ```\r\n Processed 25000 alleles\r\n Reference: Reference (2391/23415 modified reads)\r\n UNMODIFIED: 21024\r\n MODIFIED guide1: 2359\r\n MODIFIED guide2: 32\r\n Reference: HDR (856/1577 modified reads)\r\n UNMODIFIED: 721\r\n MODIFIED guide1: 854\r\n MODIFIED guide1 + guide2: 1\r\n MODIFIED guide2: 1\r\n ```\r\n\r\n commit 2e3da02fdbed2fa8ae02a277763d65a502459827\r\n Author: Cole Lyman \r\n Date: Tue Oct 10 15:29:08 2023 -0600\r\n\r\n changed tuple to list for matplotlib change (#31) (#346)\r\n\r\n Co-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\n\r\n commit cd3c332135fe4db0f9218e3d87263d5c65838ed9\r\n Author: Kendell Clement \r\n Date: Sun Oct 1 01:54:46 2023 -0600\r\n\r\n rename script to camel case\r\n\r\n commit 7c719d65fb36ac7654db9040f226564ea28fcab9\r\n Author: Kendell Clement \r\n Date: Sun Oct 1 01:53:44 2023 -0600\r\n\r\n Add new script for counting high quality bases\r\n\r\n commit f97cd2795e89464bcc9321ccfdbca3e6af2bcb4f\r\n Author: Kendell Clement \r\n Date: Thu Sep 14 15:15:30 2023 -0600\r\n\r\n Prime editing alignment params (#336)\r\n\r\n Adds two parameters to control alignment of pegRNA components: --prime_editing_gap_open_penalty and --prime_editing_gap_extend_penalty.\r\n\r\n CRISPResso checks to see whether the pegRNA spacer and extension sequence are in the correct orientation, but sometimes they could align in the incorrect orientation with a higher score (e.g. via insertion of multiple gaps, whereas a single long gap would be preferred). Introducing these two parameters allows users to adjust the alignment parameters specifically for these prime-editing checks without adjusting the global alignment parameters which will be applied to reads that are aligned to the WT reference/prime-editing reference sequences.\r\n\r\n The new prime_editing_gap_open_penalty is set to -50, a higher gap open penalty than the default needleman_wunsch_gap_open penalty (-20). This commit breaks backward-reproducibility, but mostly in the checking of pegRNA component orientation - so previously some CRISPResso runs would have failed and produced an error, but now they will (hopefully) succeed. To achieve complete backward reproducibility, add the flag --prime_editing_gap_open_penalty -20 to runs.\r\n\r\n commit 64cbf36dae85cffa2c15e73f2a7ee8aa1077d917\r\n Author: Cole Lyman \r\n Date: Thu Sep 7 16:43:30 2023 -0600\r\n\r\n Fix samtools piping (#325)\r\n\r\n * Remove samtools pipe stderr to stdout\r\n\r\n Sometimes some of the libraries that samtools depends on don't have the correct\r\n version information, and as such samtools will report this to stderr when run.\r\n Because we pipe the output of samtools, we expect it to be valid SAM format, but\r\n when these library version messages are reported, it breaks CRISPRessoWGS.\r\n\r\n * Remove extra spacing at end of lines and add missing comma in WGS\r\n\r\n * Log stderr from samtools in CRISPRessoWGS\r\n\r\n commit 8feff4101f27406d9d88ace97d31a518276bff3f\r\n Author: Cole Lyman \r\n Date: Fri Sep 1 09:43:56 2023 -0600\r\n\r\n Replace link to CRISPResso schematic with raw URL in README (#329)\r\n\r\n * Replace link to CRISPResso schematic with raw URL\r\n\r\n * Add new lines to the beginning of unordered lists\r\n\r\n commit 2e9e6bff5bcc536d5e2ba1440d1ab96d9d47efd6\r\n Author: Kendell Clement \r\n Date: Thu Aug 10 00:52:12 2023 -0600\r\n\r\n Try to unbreak CircleCI\r\n\r\n commit ae5b95246cb0f6d66c4cbfb50cf8f5a9626b0827\r\n Author: Kendell Clement \r\n Date: Thu Aug 10 00:17:27 2023 -0600\r\n\r\n Center command line text messages\r\n\r\n commit 4d9c71ecf2248c9bb1e10430178dc318b6621c8b\r\n Author: Kendell Clement \r\n Date: Thu Aug 10 00:17:07 2023 -0600\r\n\r\n Fix bug in prime-editing scaffold-incorporation plotting\r\n\r\n If read is too short, scaffold incorporation detection will fail because it will check beyond the length of the read.\r\n\r\n commit 2b36a1a5c35e8a93516ce8baf464595615e0f402\r\n Author: Kendell Clement \r\n Date: Wed Aug 9 15:29:48 2023 -0600\r\n\r\n CRISPRessoPooled --compile_postrun_references bug fixes\r\n\r\n commit 3e04d1d402bcf95edd39fc7c8c9af61bb380f9db\r\n Author: Kendell Clement \r\n Date: Tue Aug 8 23:30:15 2023 -0600\r\n\r\n Fix missing ' in Pooled --demultiplex_only_at_amplicons\r\n\r\n commit 06af527f9e2020c5cf251e7f1cec0b1eca1c1664\r\n Author: Cole Lyman \r\n Date: Mon Jul 24 10:47:46 2023 -0600\r\n\r\n Sort pandas dataframes by # of reads and sequences so that the order is consistent (#316)\r\n\r\n * Make sorting stable\r\n\r\n * Including c files\r\n\r\n * Sort by #Reads instead of %Reads to avoid floating point errors\r\n\r\n ---------\r\n\r\n Co-authored-by: Samuel Nichols \r\n\r\n commit de05533b3511a84f3b6b14fc2ef64db041613261\r\n Author: Cole Lyman \r\n Date: Thu Jul 6 13:54:45 2023 -0600\r\n\r\n Fix multiprocessing lambda pickling (#311)\r\n\r\n * Fix running plots in parallel\r\n\r\n The reason the plots were running slower before this change is because I was\r\n calling the plot function, not passing it to `submit`. So it was essentially\r\n running in serial, but worse because it was still spinning up/down the\r\n processes.\r\n\r\n * Fix multiprocessing lambda pickling (#20)\r\n\r\n * Refactor process_futures to be a dict\r\n\r\n This makes debugging much easier because you can associate the arguments to the\r\n future with the results.\r\n\r\n * Fix the pickling error when running in multiprocessing\r\n\r\n Only top-level functions (not lambdas) can be pickled to use in multiprocessing\r\n pools, thus the lambdas are converted to a regular function.\r\n\r\n * Further fixes to pickling multiprocessing error (#21)\r\n\r\n * Refactor process_futures to be a dict\r\n\r\n This makes debugging much easier because you can associate the arguments to the\r\n future with the results.\r\n\r\n * Fix the pickling error when running in multiprocessing\r\n\r\n Only top-level functions (not lambdas) can be pickled to use in multiprocessing\r\n pools, thus the lambdas are converted to a regular function.\r\n\r\n * Use Counter instead of defaultdict in CRISPRessoCORE\r\n\r\n * Update process_futures to dict in Batch and Aggregate\r\n\r\n commit ebb016dff46c280dce8c3c09e8ac0e0cc25d4d74\r\n Author: Kendell Clement \r\n Date: Mon Jul 3 17:12:09 2023 -0600\r\n\r\n Enable CRISPRessoPooled multiprocessing when os allows multi-thread file append\r\n\r\n commit 7285da0e987b77b72c8885bb35940e0f50c146bd\r\n Author: Kendell Clement \r\n Date: Fri Jun 23 16:50:33 2023 -0600\r\n\r\n Fix print bug for invalid fastq\r\n\r\n commit 9acdeac67441f9a1d55ac94b153bcb68fb89b92c\r\n Author: kclem \r\n Date: Wed Jun 21 16:03:48 2023 -0600\r\n\r\n Slugify before creating filename - replaces invalid characters in batch names with _\r\n\r\n commit f97e29c67de4c80b8d6b9cf334f363be4b514ade\r\n Author: Cole Lyman \r\n Date: Wed Jun 21 14:43:43 2023 -0600\r\n\r\n Add verbosity argument to CRISPRessoAggregate (#18) fixes #306 (#307)\r\n\r\n * Add verbosity argument to CRISPRessoAggregate (#18)\r\n\r\n * Allow for amplicon and guide seqs to be some variant of NA in batch (#19)\r\n\r\n This was discovered when attempting to infer amplicon sequences in batch mode on\r\n the web interface, NAs were supplied for the amplicon sequences to the sub\r\n CRISPResso commands.\r\n\r\n commit 32e1e9797da5c3033cdc588e92f06b8813961953\r\n Author: Mark Clement \r\n Date: Wed Jun 21 14:01:00 2023 -0600\r\n\r\n Allow for interrogation of overlapping sgRNA sites\r\n\r\n commit 7248ba8c4deee125ad1ec12fdf1294a84d5f6f93\r\n Author: Kendell Clement \r\n Date: Mon Jun 12 12:16:47 2023 -0600\r\n\r\n Check input fastq file format\r\n\r\n Asserts input format of fastq files - including if gzipped files are missing the gz suffix.\r\n\r\n commit 83c8ab8f462e7d8c1d04c08c1a398b874f517251\r\n Author: Kendell Clement \r\n Date: Mon Jun 5 13:41:55 2023 -0600\r\n\r\n Fix CRISPRessoArgParser\r\n\r\n commit 14a2c8577f566e1b72d5f4e72cd6cd22079610be\r\n Author: Kendell Clement \r\n Date: Mon Jun 5 13:29:31 2023 -0600\r\n\r\n Cosmetic updates for command-line use\r\n\r\n - version bump to 2.2.13\r\n - If no args are provided, the command line version will print out an abbreviated help message\r\n - parameters can be excluded from CRISPRessoArgParser\r\n\r\n commit 1cd54bc1d03360c3d8121ba9e66b3589fe1cf252\r\n Author: Cole Lyman \r\n Date: Thu May 11 14:31:47 2023 -0600\r\n\r\n Fix multiprocessing error, don't start pool when only using single thread (#302)\r\n\r\n * Update README to have consistent use of `--base_editor_output` (#16)\r\n\r\n * Add files via upload\r\n\r\n * Only start process pools when using multiple processes\r\n\r\n This is mainly to solve the issue when running on AWS Lambda, but this should\r\n improve single core performance overall.\r\n\r\n ---------\r\n\r\n Co-authored-by: Kendell Clement \r\n\r\n commit 92a705c939b370373a70cf6ae9f1616de33288b9\r\n Author: Cole Lyman \r\n Date: Thu May 11 14:31:06 2023 -0600\r\n\r\n Update `base_editor` parameters in README and add Plot Harness (#301)\r\n\r\n * Update README to have consistent use of `--base_editor_output` (#16)\r\n\r\n * Add files via upload\r\n\r\n ---------\r\n\r\n Co-authored-by: Kendell Clement \r\n\r\n commit 7d46c4490235df45c5546b1b470e4e6a99727031\r\n Author: Cole Lyman \r\n Date: Wed May 10 15:41:33 2023 -0600\r\n\r\n Clarify CRISPRessoWGS intended use (#303)\r\n\r\n * Update README to have consistent use of `--base_editor_output` (#16)\r\n\r\n * Add sample plotting jupyter notebook\r\n\r\n * Add clarifying info to CRISPRessoWGS description\r\n\r\n Clarify WGS usage\r\n\r\n commit 833a701787bb47674b3e921c38cac6189c775cf7\r\n Author: Kendell Clement \r\n Date: Thu May 4 17:02:46 2023 -0400\r\n\r\n Remove debug print statements\r\n\r\n commit 712eb2a11825e8d36f2870deb12b35486bd633fb\r\n Author: Kendell Clement \r\n Date: Thu May 4 16:40:07 2023 -0400\r\n\r\n Allow dashes in filenames resolve #73\r\n\r\n commit a439f094745b2b5e7f032f0777d4c67e6d6f93c5\r\n Author: Kendell Clement \r\n Date: Sat Apr 22 23:41:58 2023 -0400\r\n\r\n Raise exceptions from within futures in plot_pool\r\n\r\n commit 7e807a60de2a9d18bccd034b87106ceaf7153338\r\n Author: Kendell Clement \r\n Date: Sat Apr 22 23:38:56 2023 -0400\r\n\r\n Fix future pandas indexing warning\r\n\r\n Pandas error was \"FutureWarning: Calling float on a single element Series is deprecated and will raise a TypeError in the future. Use float(ser.iloc[0]) instead\"\r\n\r\n commit 304a92aa7a7ef8c705cb070dce25d9a2e5745ba9\r\n Author: Cole Lyman \r\n Date: Thu Apr 20 13:59:27 2023 -0600\r\n\r\n Remove debug print statements fixes #295 (#297)\r\n\r\n The format string option used here is only available in Python version >=3.8.\r\n\r\n commit 478c06f784603e96d20f96e91993fdcc4ac35c8a\r\n Author: Kendell Clement \r\n Date: Thu Apr 13 12:09:26 2023 -0400\r\n\r\n Update plotCustomAllelePlot.py script for #292 (#293)\r\n\r\n Update type of 'max_rows' param to int\r\n Fix location of 'args' in crispresso2_info object\r\n\r\n commit bcdae39e05d530f4a4e78738c3b30f7664981919\r\n Author: Kendell Clement \r\n Date: Mon Mar 27 13:18:34 2023 -0400\r\n\r\n Update pooled parameter format\r\n\r\n commit 546446e36e7e68b527767d6c31ec341a49df2059\r\n Author: Kendell Clement \r\n Date: Tue Feb 14 16:26:23 2023 -0500\r\n\r\n Fix running plots in parallel (#286)\r\n\r\n The reason the plots were running slower before this change is because I was\r\n calling the plot function, not passing it to `submit`. So it was essentially\r\n running in serial, but worse because it was still spinning up/down the\r\n processes.\r\n\r\n Co-authored-by: Cole Lyman \r\n\r\n commit d75f32a2eb5aeaaee866c09e5655a3e27af8b1a1\r\n Author: kclem \r\n Date: Fri Feb 10 15:45:15 2023 -0500\r\n\r\n Fix #283 to avoid filename collisions\r\n\r\n Previously, amplicon names longer than 21bp were truncated, but the check for uniqueness wasn't working, so it would overwrite some plot files. This fixes the filename collision and enforces uniqueness in reference filename prefixes. Thanks @mbiokyle29\r\n\r\n commit e577318006cd17b2725bd028e5e56634c6eb829a\r\n Author: kclem \r\n Date: Mon Feb 6 16:37:25 2023 -0500\r\n\r\n Case-insensitive headers accepted in CRISPRessoPooled\r\n\r\n commit d34927620a4a6126a9988b3041e76f60728abbfe\r\n Author: Kendell Clement \r\n Date: Tue Jan 31 13:48:33 2023 -0500\r\n\r\n Fix print statement in CORE\r\n\r\n commit ee88b7ed89c395f68225a50dea44a2ad69d5e9a5\r\n Author: Kendell Clement \r\n Date: Tue Jan 31 13:22:51 2023 -0500\r\n\r\n Version bump to 2.2.12\r\n\r\n commit 1d4679c72d0c8b4154317c9aff5179217198e2d7\r\n Author: Kendell Clement \r\n Date: Tue Jan 31 13:01:31 2023 -0500\r\n\r\n Status Updates + Pooled Mixed Mode Update (#279)\r\n\r\n * Implement logging handler to overwrite the latest log status to file\r\n\r\n * Add StatusHandler to CRISPRessoCORE log\r\n\r\n This will take the latest log output and write it to a file (`status.txt`), the\r\n catch being that with each log the file is overwritten so that one can easily\r\n tell where CRISPResso currently is and what the error is (if any). These changes\r\n include some slight refactoring in order to accomodate any potential parameter\r\n exceptions.\r\n\r\n * Add StatusHandler to CRISPRessoBatch and refactor `logger.warn` to `warn`\r\n\r\n * Add StatusHandler to CRISPRessoPooled and a little refactoring\r\n\r\n * Implement `percent_complete` to the status log\r\n\r\n * Add StatusHandler to CRISPRessoAggregate log\r\n\r\n * Add StatusHandler to CRISPRessoCompare log\r\n\r\n * Add StatusHandler to CRISPRessoPooledWGSCompare log\r\n\r\n * Add StatusHandler to CRISPRessoWGS log\r\n\r\n * Rename `status.txt` to `CRISPResso_status.txt`\r\n\r\n * Modify status log names to match the tool they are generated from\r\n\r\n * Add percent_complete stages to CRISPRessoCORE\r\n\r\n These also include log statements of each plot that is being generated as well\r\n as fixing some variable name collisions with `ind`.\r\n\r\n * Format the percentage in the log to be 2 decimal places\r\n\r\n * Change all plotting logs from `info` to `debug` and simplify progress\r\n\r\n This refactors how the progress of the plots is calculated, making it much\r\n simplier. Before this change we would of had to keep track of the number of\r\n times `percent_complete` was output, but now it simply updates the percent\r\n complete after each amplicon is finished processing. Hopefully this will make\r\n things easier to mantain even though it will be a little less \"accurate\" (not\r\n sure how accurate the original implementation was...).\r\n\r\n * Implemented shared console log handler across all CRISPResso* calls\r\n\r\n This allows for easy changes to logging formatting, which was inspired by having\r\n to change the default logging level. The default logging level needs to be set\r\n at `logging.DEBUG` in order for the debug log statements to not be ignored for\r\n the running and status logs.\r\n\r\n * Add ability to set the verbosity level to each CRISPResso* tool\r\n\r\n This allows users to set a verbosity level between 1 and 4 using the\r\n `-v`/`--verbosity` CLI parameter. If the `--debug` flag is present, then the\r\n level will default to 4, being the most verbose.\r\n\r\n * Implement showing the last seen `percent_compelte` when none is provided\r\n\r\n * Keep track of and log when multiple parallel runs are completed\r\n\r\n These changes modify `CRISPRessoMultiProcessing.run_crispresso_cmds` such that\r\n we can now display when a run is completed. This potentially breaks how\r\n signals and interupts are handled with multiple runs happening, but this needs\r\n to be reviewed.\r\n\r\n * Add debug and percentage complete to CRISPRessoBatch\r\n\r\n * Add percent complete to CRISPRessoPooled\r\n\r\n * Add debug and percent_complete message to CRISPRessoAggregate\r\n\r\n * Add `percent_complete` to CRISPRessoCompare\r\n\r\n * Add `percent_complete` to CRISPRessoPooledWGSCompare\r\n\r\n * Add status and `percent_complete` to CRISPRessoMeta\r\n\r\n * Add `verbosity` arguments to CRISPRessoCompare and CRISPRessoPooledWGSCompare\r\n\r\n * Fixing documentation to match pooled headers\r\n\r\n * Header removal bug fix change documentation to guide_seq\r\n\r\n * Update documentation and help feature for CRISPRessoPooled\r\n\r\n * Remove extra newlines from CRISPRessoPooled -h\r\n\r\n * Make variable names as clear as my firstborn child's name\r\n\r\n * Update one more variable name\r\n\r\n * Fix bug to flow CRISPRessoPooled options to sub command\r\n\r\n * Make amplicon file args variable name clear\r\n\r\n * Update how parameters are set and retrieved from parameter object\r\n\r\n The refactor in the previous commit changed the type of the arguments to a\r\n dictionary which doesn't have the parameters as attributes, and this commit\r\n fixes that error.\r\n\r\n * Add note in output header for change in default CRISPRessoPooled\r\n\r\n In the next release (2.3.0) the `--demultiplex_only_at_amplicons` will be the\r\n default when running in mixed-mode. This is to allow for inexact alignments of\r\n the reads and the amplicons to the genome. For more context, see this issue\r\n https://github.com/pinellolab/CRISPResso2/issues/276\r\n\r\n * Clarify the verbosity parameter help message\r\n\r\n * Separate out parameters to `normalize_name` in CRISPRessoCORE\r\n\r\n * Separate out parameters to `normalize_name` in CRISPRessoWGS\r\n\r\n * Separate out parameters to `normalize_name` in CRISPRessoPooled\r\n\r\n * Separate out parameters to `normalize_name` in CRISPRessoCompare\r\n\r\n * Fix bug in CRISPRessoPooled by replacing `database_id` with `normalize_name`\r\n\r\n * Refactor `run_crispresso_cmds` to not require a `logger`\r\n\r\n This commit implements the functionality to make the `logger` object optional by\r\n seeing which module called the `run_crispresso_cmds` function and obtaining the\r\n correct object from that module name.\r\n\r\n The function also immediately returns when no commands are passed to it.\r\n\r\n * Add amplicon name to plotting debug statements in CRISPRessoCORE\r\n\r\n ---------\r\n\r\n Co-authored-by: Cole Lyman \r\n Co-authored-by: Cole Lyman \r\n Co-authored-by: Cole Lyman \r\n Co-authored-by: Samuel Nichols \r\n\r\n commit ff7eca76e6a3a08af4ac18ac4e88d20f2a06b1f9\r\n Author: Kendell Clement \r\n Date: Thu Jan 26 15:27:27 2023 -0500\r\n\r\n CRISPRessoPooled custom header fix (#278)\r\n\r\n * Fixing documentation to match pooled headers\r\n\r\n * Header removal bug fix change documentation to guide_seq\r\n\r\n * Update documentation and help feature for CRISPRessoPooled\r\n\r\n * Remove extra newlines from CRISPRessoPooled -h\r\n\r\n * Make variable names as clear as my firstborn child's name\r\n\r\n * Update one more variable name\r\n\r\n Co-authored-by: Samuel Nichols \r\n\r\n commit 104866e1080c973bb025d1a5ba59b19dca1658af\r\n Author: Cole Lyman \r\n Date: Thu Jan 5 14:00:26 2023 -0700\r\n\r\n Fix deprecated numpy type names (fixes #269) (#270)\r\n\r\n In the most recent version of numpy (1.24) some of the types have been\r\n deprecated. This commit fixes these errors.\r\n\r\n commit 58a8e42df88b66fad6b4f6ad04a5b9d9d43d01b4\r\n Author: Cole Lyman \r\n Date: Thu Jan 5 06:49:35 2023 -0700\r\n\r\n Add snippet about installing CRISPResso2 via bioconda on Apple silicon (#274)\r\n\r\n I have suffered enough trying to debug my installation, so hopefully this helps\r\n someone else.\r\n\r\n Co-authored-by: Cole Lyman \r\n\r\n commit b9851e98104602eb78c2b384105267624295e9d3\r\n Author: Cole Lyman \r\n Date: Thu Dec 22 13:30:23 2022 -0700\r\n\r\n Fix bug when pooled bam is input (#265)\r\n\r\n This change checks to see if a bam file was input, and if so it doesn't try to\r\n remove any intermediate files because there aren't any.\r\n\r\n Co-authored-by: Cole Lyman \r\n\r\n commit b822612642043e75a19042941f69b457ce51f517\r\n Author: Kendell Clement \r\n Date: Mon Dec 19 15:26:45 2022 -0500\r\n\r\n Delete vscode settings\r\n\r\n commit b99aa624dec68ef7d19264340ce0cafa829625f4\r\n Author: Kendell Clement \r\n Date: Mon Dec 19 13:29:14 2022 -0500\r\n\r\n Clarify input param help for pooled bam\r\n\r\n commit 3fae1e8b821ec6b1890bff6561fa8fa67dc49a04\r\n Author: Kendell Clement \r\n Date: Mon Dec 19 13:28:54 2022 -0500\r\n\r\n Fix #235 - Cigar string is * if read unaligned\r\n\r\n Previously, the bam would set the cigar string to 0 if the read was unaligned. This breaks the sam->bam conversion and causes the errors in #235.\r\n\r\n commit c65ba07dc5a983453cdf7bb1e27005230dac6f1b\r\n Author: Cole Lyman \r\n Date: Thu Dec 8 13:48:17 2022 -0700\r\n\r\n Add deprecation notice (#260)\r\n\r\n * Add FLASh and Trimmomatic deprecation notice to CLI output\r\n\r\n * Add Edilytics email address to CLI output\r\n\r\n commit 2a30e5a45f5350ee7c6435bce1cd4edc4d31668a\r\n Author: Kendell Clement \r\n Date: Tue Dec 6 12:16:19 2022 -0500\r\n\r\n Format filterReadsOnSequencePresence script\r\n\r\n commit 9d764414edd88a46ad5e4f496e4f1c8d5d60ce3e\r\n Author: Kendell Clement \r\n Date: Fri Dec 2 22:12:54 2022 -0500\r\n\r\n Clarify default CRISPRessoPooled settings for use_legacy_bowtie2_options_string\r\n\r\n commit 9ddea40f7f02b546941ddaa4c71fc5283075051a\r\n Author: kclem \r\n Date: Mon Nov 14 10:33:04 2022 -0500\r\n\r\n Add check for prime editing extension sequence in prime edited sequence\r\n\r\n if the user specifies the prime_editing_override_prime_edited_ref_seq, it could not contain the extension seq (if they don't provide the extension seq in the appropriate orientation), so check that here. Extension sequence should be provided reverse-complement to the prime edited sequence.\r\n\r\n commit 152f2dd5001da7090641ee8a1326bde9f7e8104e\r\n Author: kclem \r\n Date: Wed Nov 9 11:53:41 2022 -0500\r\n\r\n Version bump to 2.2.11a\r\n\r\n commit 9ed356e3a0c6c316d0860d121772f80ddca6de1d\r\n Author: kclem \r\n Date: Wed Nov 9 11:47:30 2022 -0500\r\n\r\n Add param to override prime editing sequence checks\r\n\r\n CRISPResso checks that prime editing guides are provided in the proper orientation (e.g. pegRNA 3'->5', spacer sequence 5'->3') and checks these orientations by alignment. Sometimes, the alignment can be better in the opposite direction, and this parameter allows these checks to be overridden. Otherwise, these checks would halt the program and produce the output 'The prime editing pegRNA spacer sequence appears to be given in the 3\\'->5\\' order. The prime editing pegRNA spacer sequence (--prime_editing_pegRNA_spacer_seq) must be given in the RNA 5\\'->3\\' order.'\r\n\r\n commit 39dd80afb98a22b7edb6f801c363d86bb77eeb5b\r\n Author: kclem \r\n Date: Wed Nov 9 10:06:51 2022 -0500\r\n\r\n Update filterReadsOnSequencePresence.py\r\n\r\n commit fe55526927e3fb6e17c9a8a6f59c7057bc1e14eb\r\n Author: Kendell Clement \r\n Date: Mon Nov 7 22:25:16 2022 -0500\r\n\r\n Add script to filter input based on sequence presence\r\n\r\n commit 713e57a19c35180035ca35e11a5820065eda0198\r\n Author: Kendell Clement \r\n Date: Tue Oct 18 16:02:26 2022 -0400\r\n\r\n Allow spaces in read names for CRISPRessoWGS\r\n\r\n commit 39ce008bdddccdd8229c0ba185dce78bc2f66968\r\n Author: Cole Lyman \r\n Date: Sat Oct 8 21:09:58 2022 -0600\r\n\r\n Fix typo of CRISPResssoPlot when plotting nucleotide quilt (#250)\r\n\r\n commit 6a2b342c8503b7327c0a2414edfbd16912d60ca5\r\n Author: Kendell Clement \r\n Date: Sat Oct 8 23:08:47 2022 -0400\r\n\r\n Batch amplicon plots (#251)\r\n\r\n * Error out if HDR amplicon matches existing amplicon\r\n\r\n * Add check for amplicon sequence uniqueness\r\n\r\n * Fix bug with bam_input not having bam_output\r\n\r\n * Test for no returned lines in auto mode, version bump to 2.2.11\r\n\r\n * Fix pandas deprecation of df.append\r\n\r\n commit 726b2b93d6e419a1b0aa6a968c97edc55b4cc5a8\r\n Author: Kendell Clement \r\n Date: Thu Oct 6 16:32:02 2022 -0400\r\n\r\n Fix CRISPRessoBatch plot pool bug when plots are suppressed\r\n\r\n commit 7e5049c4dfb88cbc87c91935a91d1f51120a10c2\r\n Author: Cole Lyman \r\n Date: Wed Sep 21 21:04:51 2022 -0600\r\n\r\n Fix batch quilt plot name (#249)\r\n\r\n This fixes an incorrectly named allele quilt plot input in CRISPRessoBatch.\r\n\r\n commit 1821ca5029c5a1485733f13ab3f2048b4f1fa04e\r\n Author: Kendell Clement \r\n Date: Thu Sep 15 15:49:08 2022 -0400\r\n\r\n Version bump to 2.2.10\r\n\r\n commit c5f79aebfc1ae209f4ee320df250eed89a02787c\r\n Author: Cole Lyman \r\n Date: Wed Sep 14 14:24:55 2022 -0600\r\n\r\n Parallel plot refactor (#247)\r\n\r\n * Fix duplicate plotting in CRISPRessoBatch aggregate\r\n\r\n * Refactor mulltiprocessing plots in CRISPRessoBatch\r\n\r\n * Refactor multiprocessing plots in CRISPRessoCORE\r\n\r\n * Refactor multiprocessing plots for CRISPRessoAggregate\r\n\r\n commit 4ed5e24e6cc1dd8068e2391573ae2438acd32db2\r\n Author: Kendell Clement \r\n Date: Tue Sep 13 14:12:11 2022 -0400\r\n\r\n print files in curr dir if Aggregate can't find files\r\n\r\n commit ce25bc06f29988e7a10afd0b6a09ba0caf0950e0\r\n Author: Kendell Clement \r\n Date: Mon Sep 12 10:32:57 2022 -0400\r\n\r\n Spelling typo\r\n\r\n commit c15f01c75083403f17c58c121b2afe97e9f2a1ec\r\n Author: Kendell Clement \r\n Date: Tue Sep 6 17:49:52 2022 -0400\r\n\r\n Add helper function to create alignment scoring matrix\r\n\r\n New scoring matrix can be created using CRISPResso2Align.make_matrix()\r\n\r\n commit c80f82838c5a228b79ad4484092877cfee08e02c\r\n Author: Cole Lyman \r\n Date: Mon Aug 22 18:28:33 2022 -0600\r\n\r\n Add `zip_output` (#240)\r\n\r\n * Making zip of results\r\n\r\n * Zip command added, if zip is true place_report_in_output_folder is also true, zip removes all files while zipping\r\n\r\n * Adding --zip to compare and pooled/wgs compare\r\n\r\n * Add more formatting changes to CRISPRessoShared\r\n\r\n * Refactoring propagate_crispress_options so only one version exists\r\n\r\n * Zip added to arguments_to_ignore and warning added when changing arguments\r\n\r\n * Restore styling\r\n\r\n * Update README to include --zip\r\n\r\n * Rename --zip to --zip_output\r\n\r\n * Change --zip to --zip_output in CompareCORE and PooledWGSCompareCORE\r\n\r\n * Bug fix arg to args\r\n\r\n Co-authored-by: Samuel Nichols \r\n\r\n commit 5de3d7286d8e33c7cf4d3615fce715806e72f511\r\n Author: Kendell Clement \r\n Date: Thu Aug 11 21:42:34 2022 -0400\r\n\r\n Fix fix to aggregate for CRISPRessoWGS\r\n\r\n commit a2294c266f43b14969a5d6474076f31a77a57173\r\n Author: Kendell Clement \r\n Date: Thu Aug 11 21:40:50 2022 -0400\r\n\r\n Fix bug in aggregate for WGS\r\n\r\n commit 7ce3eb4abe4b8ceac933272ac9cb16a8bedf26a3\r\n Author: Kendell Clement \r\n Date: Mon Aug 8 21:53:45 2022 -0400\r\n\r\n Update CRISPRessoWGS to allow non-word characters in region names\r\n\r\n commit 040ac0033d6e250f4e3a412101874cf5e914e08a\r\n Author: kclem \r\n Date: Mon Aug 8 16:04:59 2022 -0400\r\n\r\n Enable processing of cram files by CRISPRessoWGS\r\n\r\n Adds --reference to samtools view when viewing cram files\r\n\r\n commit cf112a0caba8789e28530cc09171285ec6ea9b4c\r\n Author: kclem \r\n Date: Mon Aug 8 14:55:46 2022 -0400\r\n\r\n Auto amplicon detection for interleaved input\r\n\r\n Enables processing of interleaved fastq files for guess_guides and guess_amplicons, as well as get_most_frequent_reads. When interleaved input is present, the input is first separated into R1/R2 files, then processing is performed.\r\n\r\n commit 4ba524dc7b947feca8a0f743837844f9febc2171\r\n Author: Cole Lyman \r\n Date: Thu Aug 4 11:32:11 2022 -0600\r\n\r\n Potential fix for aggregate plots in Batch mode (#237)\r\n\r\n commit 6097a8a104d3f156ef7c08e196ac37e32bf04c71\r\n Author: Kendell Clement \r\n Date: Thu Jul 21 22:45:48 2022 -0400\r\n\r\n Fix pct_vectors in crispresso2_info json object\r\n\r\n commit 65a079d86d6f386793397398f839c46014b54543\r\n Author: Kendell Clement \r\n Date: Wed Jul 20 23:46:37 2022 -0400\r\n\r\n Fix more readme spelling bugs\r\n\r\n commit e817376ecd54cdea1f29e303ca25b9e7d1d38333\r\n Author: Kendell Clement \r\n Date: Wed Jul 20 23:42:23 2022 -0400\r\n\r\n Fix bug in readme spelling\r\n\r\n commit 49740ba1d66ed6d13a9e154b8b17bc8b5186581d\r\n Author: Kendell Clement \r\n Date: Wed Jul 20 16:10:09 2022 -0400\r\n\r\n Fix loading of crispresso info from WGS and Pooled\r\n\r\n commit b68a43271115251b18e8955e285ccc18f549e8cd\r\n Author: Kendell Clement \r\n Date: Thu Jul 14 14:11:04 2022 -0400\r\n\r\n Add plotly to dockerfile\r\n\r\n commit b0b7d41d697304d0d5fc93e3346c9de1b98ba41d\r\n Author: Kendell Clement \r\n Date: Thu Jul 14 14:10:00 2022 -0400\r\n\r\n Fix #231 Allow N's in bam output (Try 2)\r\n\r\n commit c460b3e73fd06a230dbac2e37c86b833144ebf94\r\n Author: Kendell Clement \r\n Date: Thu Jul 14 14:09:10 2022 -0400\r\n\r\n Revert \"Fix #231 Allow N's in bam output\"\r\n\r\n This reverts commit 2f6ad1dbe05210af9ccc1b1f17783cd212a888d3.\r\n\r\n commit 2f6ad1dbe05210af9ccc1b1f17783cd212a888d3\r\n Author: Kendell Clement \r\n Date: Thu Jul 14 13:52:37 2022 -0400\r\n\r\n Fix #231 Allow N's in bam output\r\n\r\n commit 0a2419e518dc9b3520058c3927f98b31cd51347e\r\n Author: Cole Lyman \r\n Date: Fri Jul 8 21:10:01 2022 -0600\r\n\r\n Fix bug when name is provided instead of amplicon_name in pooled input file (#229)\r\n\r\n Also, raise an exception (instead of incorrectly executing) when there are not\r\n enough matched parameters in the pooled input file.\r\n\r\n commit cb58212379803788c04ca5793baaa760cbbeaa81\r\n Author: Cole Lyman \r\n Date: Fri Jul 8 21:09:49 2022 -0600\r\n\r\n Fix bug…","shortMessageHtmlLink":"Fastp, args as data, guardrails, and PE fix (#415)"}},{"before":"b2cfb911171dd8550f057e67239f51a8dfff91aa","after":"fa03d16ddd10e3163c035bd1a8d18416f7da35a8","ref":"refs/heads/master","pushedAt":"2024-03-28T22:28:36.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Decrease Docker image size and fix PE naming and parameter behavior (#404)\n\n* Fix 'Prime-edited' key not found (#32)\r\n\r\n* Move 'Prime-edited' amplicon name check\r\n\r\nBy moving this, it will check if there is an amplicon named\r\n'Prime-edited' (which is a reserved name) even if the\r\n`prime_editing_pegRNA_extension_seq` parameter is empty.\r\n\r\n* Only search for scaffold integration when pegRNA extension seq is provided\r\n\r\n* Remove spaces at the end of lines\r\n\r\n* Docker size (#49)\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* GitHub actions integration tests (#48)\r\n\r\n* GitHub actions clean (#40)\r\n\r\n* Create pytest.yml\r\n\r\n* Create pylint.yml\r\n\r\n* Create .pylintrc\r\n\r\n* Create test_env.yml\r\n\r\n* Full path\r\n\r\n* Remove conda install\r\n\r\n* Replace path\r\n\r\n* Pytest tests\r\n\r\n* pip -e\r\n\r\n* Create integration_tests.yml\r\n\r\n* Simplify name\r\n\r\n* CRISPRESSO2_DIR environment variable\r\n\r\n* Up one dir\r\n\r\n* ls workspace\r\n\r\n* Install CRISPResso and ydiff\r\n\r\n* Clone repo instead of checkout\r\n\r\n* submodule\r\n\r\n* ls\r\n\r\n* CRISPResso2_copy\r\n\r\n* ls\r\n\r\n* Update env\r\n\r\n* Simplify\r\n\r\n* Pull from githubactions branch\r\n\r\n* Pull githubactions repo\r\n\r\n* Checkout githubactions\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Run tests individually\r\n\r\n* Pin plotly version\r\n\r\n* Run all tests even if one fails\r\n\r\n* Test on another branch\r\n\r\n* Switch branch with token\r\n\r\n* Update integration_tests.yml\r\n\r\n* Introduce pandas sorting in CRISPRessoCompare (#47)\r\n\r\n* New makefile commands\r\n\r\n* Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\r\n\r\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* On push no branches\r\n\r\n* On push no branches\r\n\r\n* All in one file\r\n\r\n* Fix yml errors\r\n\r\n* Rename jobs\r\n\r\n* Remove old workflow files\r\n\r\n* Remove paths\r\n\r\n* Run jobs in parallel\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n* 3.4->2.08\r\n\r\n* Put ttf-mscorefonts-installer back above apt-get clean\r\n\r\n* restore slash, replace fastp with trimmomatic and flash, add autoremove step\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Samuel Nichols \r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>","shortMessageHtmlLink":"Decrease Docker image size and fix PE naming and parameter behavior (#…"}},{"before":"d171561b4a98f1f5b323b9f1b19c027ef40b059a","after":"b2cfb911171dd8550f057e67239f51a8dfff91aa","ref":"refs/heads/master","pushedAt":"2024-03-28T20:41:31.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Fix the assignment of multiple quantification window coordinates (#38) (#403)\n\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\n\r\n\r\n* Extract out quantification window coordinate function\r\n\r\n* Refactor get_quant_window_coordinates function into two\r\n\r\nThe rationale behind this is that the behavior around the cloned amplicon is\r\nquite different than if the qwc are specified directly for the amplicon.\r\n\r\n* Handling qwc: add unit tests, refactor some more and add documentation\r\n\r\n* Extract out get_relative_coordinates function\r\n\r\nThis function just computes the relative indexes without doing an alignment.\r\n\r\n* Add clarifying unit tests for `get_relative_coordinates`\r\n\r\n* Refactor cloned indexes to use ref_positions instead of s1inds\r\n\r\n* fixed function for getting cloned qwc idxs\r\n\r\n* added tests for cloned qwc function\r\n\r\n* Introduce pandas sorting in CRISPRessoCompare (#47)\r\n\r\n* Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\r\n\r\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\n\r\n\r\n---------\r\n\r\n\r\n\r\n* removed if check\r\n\r\n* implemented last test\r\n\r\n* changed NT to BadParameterException\r\n\r\n* changed tests, NT to BadParameter exceptions\r\n\r\n* Uncomment and correct tests for `get_relative_coordinates`\r\n\r\n* finished qwc tests\r\n\r\n* 0 is an acceptable qwc\r\n\r\n* new get_relative_coords function\r\n\r\n* added relative coordinate tests\r\n\r\n* removed unused functions\r\n\r\n* formatting\r\n\r\n* check for 0 qwc\r\n\r\n* remove test code\r\n\r\n* remove comment\r\n\r\n* Move read filtering to after merging in CRISPResso (#39)\r\n\r\n* Move read filtering to after merging\r\n\r\nThis is in an effort to be consistent with the behavior and results of\r\nCRISPRessoPooled.\r\n\r\n* Properly assign the correct file names for read filtering\r\n\r\n* Add space around operators\r\n\r\n* GitHub actions on pr (#51)\r\n\r\n* Run integration tests on pull_request\r\n\r\n* Run pytest on pull_request\r\n\r\n* Run pylint on pull_request\r\n\r\n* Run tests on PR only when opening PR (#53)\r\n\r\n* Update reports (#52)\r\n\r\n* Update report changes\r\n\r\n* Switch branch of integration test repo\r\n\r\n* Remove extraneous `crispresso_data_path`\r\n\r\n* Point integration tests back to master\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: McKay \r\nCo-authored-by: Samuel Nichols ","shortMessageHtmlLink":"Fix the assignment of multiple quantification window coordinates (#38) ("}},{"before":"e87d92ebaf95a1147b657a4c356df137d9aae04e","after":"d171561b4a98f1f5b323b9f1b19c027ef40b059a","ref":"refs/heads/master","pushedAt":"2024-03-28T20:33:57.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Update integration_tests.yml","shortMessageHtmlLink":"Update integration_tests.yml"}},{"before":"cee64891a6d0a803906c686d27b1c597db91345a","after":"e87d92ebaf95a1147b657a4c356df137d9aae04e","ref":"refs/heads/master","pushedAt":"2024-03-28T20:01:03.000Z","pushType":"pr_merge","commitsCount":4,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Update reports (#52)\n\n* Update report changes\r\n\r\n* Switch branch of integration test repo\r\n\r\n* Remove extraneous `crispresso_data_path`\r\n\r\n* Point integration tests back to master","shortMessageHtmlLink":"Update reports (#52)"}},{"before":"ad3bd4ef552cc1b2f7628617c200174ab56cf255","after":"cee64891a6d0a803906c686d27b1c597db91345a","ref":"refs/heads/master","pushedAt":"2024-03-18T14:57:41.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"GitHub actions integration tests (#48)\n\n* GitHub actions clean (#40)\r\n\r\n* Create pytest.yml\r\n\r\n* Create pylint.yml\r\n\r\n* Create .pylintrc\r\n\r\n* Create test_env.yml\r\n\r\n* Full path\r\n\r\n* Remove conda install\r\n\r\n* Replace path\r\n\r\n* Pytest tests\r\n\r\n* pip -e\r\n\r\n* Create integration_tests.yml\r\n\r\n* Simplify name\r\n\r\n* CRISPRESSO2_DIR environment variable\r\n\r\n* Up one dir\r\n\r\n* ls workspace\r\n\r\n* Install CRISPResso and ydiff\r\n\r\n* Clone repo instead of checkout\r\n\r\n* submodule\r\n\r\n* ls\r\n\r\n* CRISPResso2_copy\r\n\r\n* ls\r\n\r\n* Update env\r\n\r\n* Simplify\r\n\r\n* Pull from githubactions branch\r\n\r\n* Pull githubactions repo\r\n\r\n* Checkout githubactions\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* Run tests individually\r\n\r\n* Pin plotly version\r\n\r\n* Run all tests even if one fails\r\n\r\n* Test on another branch\r\n\r\n* Switch branch with token\r\n\r\n* Update integration_tests.yml\r\n\r\n* Introduce pandas sorting in CRISPRessoCompare (#47)\r\n\r\n* New makefile commands\r\n\r\n* Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\r\n\r\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly\r\n\r\n* Bug Fix - 367 (#35)\r\n\r\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n* On push no branches\r\n\r\n* On push no branches\r\n\r\n* All in one file\r\n\r\n* Fix yml errors\r\n\r\n* Rename jobs\r\n\r\n* Remove old workflow files\r\n\r\n* Remove paths\r\n\r\n* Run jobs in parallel\r\n\r\n---------\r\n\r\nCo-authored-by: mbowcut2 <55161542+mbowcut2@users.noreply.github.com>\r\nCo-authored-by: Cole Lyman ","shortMessageHtmlLink":"GitHub actions integration tests (#48)"}},{"before":"0c834b36e72dbc6674a5cfa608c398b5f411f52f","after":"ad3bd4ef552cc1b2f7628617c200174ab56cf255","ref":"refs/heads/master","pushedAt":"2024-03-14T21:56:18.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Bug Fix - 367 (#35)\n\n* - Fixed references to ref_names_for_pe\r\n\r\n* removed extra tabs\r\n\r\n* trying to match empty line, no tabs\r\n\r\n* - changed references to ref_names[0]\r\n\r\n* Mckay/pd warnings (#45)\r\n\r\n* refactor errors='ignore' to try except\r\n\r\n* refactored integer slice to iloc[]\r\n\r\n* moved to_numeric try except to function\r\n\r\n* Refactor to_numeric_ignore_errors to to_numeric_ignore_columns\r\n\r\nThis change is slightly cleaner because it addresses the root issue that some\r\ncolumns are strings (and can therefore not be converted to numeric types). Now\r\nif an error does occur when converting the dfs to numeric types it won't be\r\nswallowed up.\r\n\r\n* Add documentation to to_numeric_ignore_columns\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman \r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman ","shortMessageHtmlLink":"Bug Fix - 367 (#35)"}},{"before":"0af8fe1377ea7a548250115d3a3adc9af7881395","after":"0c834b36e72dbc6674a5cfa608c398b5f411f52f","ref":"refs/heads/master","pushedAt":"2024-03-14T19:58:32.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRessoWGS params (#42)\n\n* Extract out split_interleaved_fastq function to CRISPRessoShared\r\n\r\n* Implement splitting interleaved fastq files in CRISPRessoPooled\r\n\r\n* Suppress split_interleaved_input from CRISPRessoWGS parameters\r\n\r\n* Suppress other parameters in CRISPRessoWGS\r\n\r\n* Move where interleaved fastq files are split to be trimmed properly","shortMessageHtmlLink":"Fix interleaved fastq input in CRISPRessoPooled and suppress CRISPRes…"}},{"before":"de182d268518202a07c59780c3de0da760ef7409","after":"0af8fe1377ea7a548250115d3a3adc9af7881395","ref":"refs/heads/master","pushedAt":"2024-03-14T19:41:43.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Introduce pandas sorting in CRISPRessoCompare (#47)","shortMessageHtmlLink":"Introduce pandas sorting in CRISPRessoCompare (#47)"}},{"before":"448d244c1f990046a8a09662d220387a47a87edb","after":"de182d268518202a07c59780c3de0da760ef7409","ref":"refs/heads/master","pushedAt":"2024-03-07T22:04:14.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"fix #377 to write info files in CRISPRessoPooled","shortMessageHtmlLink":"fix #377 to write info files in CRISPRessoPooled"}},{"before":"2b163927faa4ca37a1dc8294ffd1c18dd057b62e","after":"448d244c1f990046a8a09662d220387a47a87edb","ref":"refs/heads/master","pushedAt":"2024-03-06T18:06:23.000Z","pushType":"pr_merge","commitsCount":4,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"flattened allele data for plot","shortMessageHtmlLink":"flattened allele data for plot"}},{"before":"31c65dfbe9d9e146eb1d12a91e0bfaad76b263dd","after":"2b163927faa4ca37a1dc8294ffd1c18dd057b62e","ref":"refs/heads/master","pushedAt":"2024-02-13T17:14:56.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Fix #376 - CRISPRessoPooled chunking\n\nWhen running CRISPressoPooled with > 10000000 reads aligned, if there are reads overlapping the chunk boundary, instead of increasing the end with 500 bases, it takes the last position of chromosome as end position. This fixes the bug.","shortMessageHtmlLink":"Fix #376 - CRISPRessoPooled chunking"}},{"before":"751d9ea215707e145eda1446b20328215f90d6b7","after":"31c65dfbe9d9e146eb1d12a91e0bfaad76b263dd","ref":"refs/heads/master","pushedAt":"2024-02-05T23:45:45.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Cole/fix assigning multiple qwc (#37)\n\n* Remove extraneous whitespace\r\n\r\n* Implement the ability to assign multiple quantification window coordinates\r\n\r\n* Updating documentation for quantification window coordinates\r\n\r\n* Update to allow 0 to prevent qwc being set. Update description.\r\n\r\n---------\r\n\r\nCo-authored-by: Cole Lyman ","shortMessageHtmlLink":"Cole/fix assigning multiple qwc (#37)"}},{"before":"3f84c7dccf62885583eb6b74825c2d63557a6aa2","after":"751d9ea215707e145eda1446b20328215f90d6b7","ref":"refs/heads/master","pushedAt":"2024-02-05T19:31:56.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Fix #374 naming of CRISPRessoPooled file name","shortMessageHtmlLink":"Fix #374 naming of CRISPRessoPooled file name"}},{"before":"3bc04213fcc730d6a7dcd51067998f7c220df135","after":"3f84c7dccf62885583eb6b74825c2d63557a6aa2","ref":"refs/heads/master","pushedAt":"2024-01-25T07:13:53.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Revert \"denoise multiprocessing\"\n\nThis reverts commit 3bc04213fcc730d6a7dcd51067998f7c220df135.","shortMessageHtmlLink":"Revert \"denoise multiprocessing\""}},{"before":"4723e4ac81c38d0fb2da7682f4213a0586bfc881","after":"3bc04213fcc730d6a7dcd51067998f7c220df135","ref":"refs/heads/master","pushedAt":"2024-01-25T07:12:00.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"denoise multiprocessing","shortMessageHtmlLink":"denoise multiprocessing"}},{"before":"ba8882dff10602a55cb2d1e4317863d55a9b50b7","after":"4723e4ac81c38d0fb2da7682f4213a0586bfc881","ref":"refs/heads/master","pushedAt":"2024-01-25T06:47:43.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Improve multiprocessing in Batch to utilize more processors\n\nPreviously, sub-crispresso runs are run on 1 process (with the rationale that parallelization of the alignment bit is the most important for batches). Now, with parallelized plotting, if more processes are provided than batches, each sub-crispresso run will be run with int(#processes/#batches) processes, thus maximizing the processor utilization for small batches.","shortMessageHtmlLink":"Improve multiprocessing in Batch to utilize more processors"}},{"before":"08a25495ee53e293ec5d85b843f381d561a428e8","after":"ba8882dff10602a55cb2d1e4317863d55a9b50b7","ref":"refs/heads/master","pushedAt":"2024-01-23T06:12:07.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"kclem","name":"Kendell Clement","path":"/kclem","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/12242609?s=80&v=4"},"commit":{"message":"Include all jinja templates recursively","shortMessageHtmlLink":"Include all jinja templates recursively"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEO5G7eQA","startCursor":null,"endCursor":null}},"title":"Activity · pinellolab/CRISPResso2"}