Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ci hooks #276

Closed
wants to merge 51 commits into from
Closed

Ci hooks #276

wants to merge 51 commits into from

Conversation

manulera
Copy link
Collaborator

@manulera manulera commented Oct 3, 2024

No description provided.

BjornFJohansson and others added 20 commits May 1, 2024 10:01
* closes #238

* fix notebook
* Documentation notebooks (#271)

* added documentation for dseq.py

* fixed docs for Dseq.py

* fixed docs for Dseq.py

* add hook

* fix exclusions and run hook

* strip outputs of Dseq.ipynb

* update Dseq notebook

* placed notebooks in docs

* added looped and shifted methods in Dseq docs

* added docs for importing files using pydna

* added some Dseqrecord docs

* added docs+examples for Dseq_Features

* added origin spanning feats and removing feats for Dseq_Features

* some comments and todos

* added restriction page

* updated Importing_Seq page

* updated Dseq_Feature Page w/examples

* added restrict+ligate notebook

* added some PCR notes

* added examples, prints

* added tm and design

* fixed tm_default docs

* added gibson assembly

* some comments on current progress

* gibson

* fixed comments, added gibson example and CRISPR

* added Restriction example

* added Gibson example

* closes #256

* run pre-commit

* alternative example for gibson

* gibson updated example

* modified crispr

* modified crispr

* feedback changes for #260

* best practices for qualifiers

---------

Co-authored-by: Pei-Lun Xie <[email protected]>
Co-authored-by: Pei-Lun Xie <[email protected]>

* remove unnecessary file

* remove dir exclusions

* exclude test notebooks

---------

Co-authored-by: Pei-Lun Xie <[email protected]>
Co-authored-by: Pei-Lun Xie <[email protected]>
Copy link

codecov bot commented Oct 3, 2024

Codecov Report

Attention: Patch coverage is 83.79205% with 53 lines in your changes missing coverage. Please review.

Project coverage is 93.57%. Comparing base (e32977f) to head (44c16a7).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/pydna/seq.py 61.70% 18 Missing ⚠️
src/pydna/seqrecord.py 54.16% 11 Missing ⚠️
src/pydna/dseqrecord.py 60.00% 7 Missing and 1 partial ⚠️
src/pydna/parsers.py 89.36% 3 Missing and 2 partials ⚠️
src/pydna/dseq.py 94.66% 2 Missing and 2 partials ⚠️
src/pydna/readers.py 62.50% 1 Missing and 2 partials ⚠️
src/pydna/tm.py 87.50% 1 Missing and 1 partial ⚠️
src/pydna/utils.py 94.28% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #276      +/-   ##
==========================================
+ Coverage   93.00%   93.57%   +0.57%     
==========================================
  Files          40       40              
  Lines        3744     3909     +165     
  Branches      562      585      +23     
==========================================
+ Hits         3482     3658     +176     
+ Misses        217      199      -18     
- Partials       45       52       +7     
Files with missing lines Coverage Δ
src/pydna/__init__.py 90.10% <100.00%> (+0.82%) ⬆️
src/pydna/amplify.py 98.65% <100.00%> (ø)
src/pydna/assembly.py 97.59% <100.00%> (+0.02%) ⬆️
src/pydna/common_sub_strings.py 100.00% <100.00%> (ø)
src/pydna/crispr.py 87.23% <ø> (ø)
src/pydna/design.py 97.51% <100.00%> (+0.31%) ⬆️
src/pydna/myenzymes.py 100.00% <100.00%> (ø)
src/pydna/tm.py 96.92% <87.50%> (-3.08%) ⬇️
src/pydna/utils.py 93.11% <94.28%> (+0.02%) ⬆️
src/pydna/readers.py 80.00% <62.50%> (-20.00%) ⬇️
... and 5 more

... and 8 files with indirect coverage changes

@manulera manulera closed this Oct 3, 2024
@manulera manulera deleted the ci_hooks branch October 3, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants