Skip to content

Comments

Exam_Project_PatternDetectionInProteins#36

Open
EngelsI wants to merge 7 commits intoMichielStock:masterfrom
EngelsI:master
Open

Exam_Project_PatternDetectionInProteins#36
EngelsI wants to merge 7 commits intoMichielStock:masterfrom
EngelsI:master

Conversation

@EngelsI
Copy link

@EngelsI EngelsI commented Jan 28, 2022

No description provided.

@MichielStock
Copy link
Owner

Looks very nice, I see that you have a lot of general data structures typed with Any, e.g. Dict{Any,Any}. These give no error in Julia but make your code much slower because the functions can't specialize on them during compilation. Solving these is an easy way to improve performance. You can do it by providing type information, e.g. Dict{Char, Vector{Bool}}. In detect_motifs, you might store Tuples ("adfk", 1.0, 2.0), these will also specialize.

@jstaut
Copy link

jstaut commented Jan 31, 2022

Review (Jasper Staut)

README
Edit: "The protein sequences must be provided in FASTA format"

Introduction
I'm not sure I understand what "design" patterns are in proteins?

The data
Should the word "phages" also be in italics when saying "Pseudomonas phages" and "Staphylococcus phages"?
Edit: [...] related (compaired -> compared) to the other two sets

2 The Similarity_matrix function
I am a bit confused on the "tree based (evolutionary) multiple sequence alignment"
After reading part 3 below and the iterate function, I think I understand why it is evolutionary, but how is it tree based?

3 Multiple_alignment function
Maybe state somewhere that the consensus is made by taking only matching AA's and inserting a '-' otherwise
point 7. "Basis of the evolutionary algorithm." -> maybe reformulate that sentence?

@jstaut
Copy link

jstaut commented Jan 31, 2022

Also, a very nice and neat project! ^_^

@EngelsI
Copy link
Author

EngelsI commented Jan 31, 2022

Also, a very nice and neat project! ^_^

Thank you. The feedback was very helpfull!
I adjusted the text like you said. There were indeed some missing details to make it more easy to follow our train of thoughts.

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.

3 participants