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

Dev #8

Merged
merged 105 commits into from
Oct 26, 2020
Merged

Dev #8

merged 105 commits into from
Oct 26, 2020

Conversation

ArnaudBelcour
Copy link
Contributor

Padmet has now an entrypoint padmet to use all the padmet.utils scripts with the corresponding subcommand. More information in the readme.

Add

  • Continuous Integration by using GitHub Actions.
  • New visualization scripts: visu_network.py, visu_path.py and visu_similarity_gsmn.py.
  • Script to create ontology from MetaCyc.
  • Extraction of protein sequence find by exonerate in prot2genome.
  • Option to pgdb_to_padmet to keep self-producing reactions.
  • The possibility to create newick file from pvclust dendrogram.
  • An option to keep temporary files in prot2genomes.
  • Misc in padmet Node about spontaneous reactions to keep them in pgdb_to_padmet.
  • Multiprocessing in compare_padmet.
  • An option to create a file for from_pgdb_to_padmet in pgdb_to_padmet.
  • Input and output compounds to pathway in pgdb_to_padmet.
  • Getter for Relation.
  • Output folder for padmet_stats.

Fix

  • Dependencies for padmet are now flexible (>=) instead of being fixed (==).
  • Consistency in padmet_to_padmet ';' to ','.
  • Issue in padmet_to_tsv when there is no padmetRef file.
  • Error when trying to create node label in dendrogram_reactions_distance.py.
  • Issue with pgdb_to_padmet (complex with compounds/classes/RNAs).
  • Issue in pgdb_to_padmet not taking into account attributes with " - ".
  • Issue in padmet_to_padmet when merging the same reaction but with different directions.
  • Issue with biopython >= 1.78 in gbk_to_faa.
  • Numerous typos.

Modification

  • Refactor padmet creation by implementing the instantiate_padmet function and the padmet/classes/instantiation.py script.
  • Completely rewrite compare_sbml to be able to compare multiple sbml files.
  • Better error message for menetools in flux_balance_analysis.
  • Update readme.
  • Update cobra dependency to 0.17.1 and fix issue with old function of cobra.
  • Replace intervene and upset graph by supervenn in dendrogram_reactions_distance.py.
  • Rename .csv extension into .tsv because we only use tab delimiter.
  • Regex for SBML ID.
  • Replace `present' in compare_padmet and comapre_sbml by 1 (or 0).

Optimization

  • Use a region - 10000 and + 10000 around the match query of tBlastn to use exonerate on it instead of using the all contig.
  • Try to reduce the number of Orthologues files analysed in extract_orthofinder.py.

Add test for sbml_to_padmet.
For wikiGenerator, sbml_to_curation_form, gbk_to_fasta, gene_to_targets.
Modify tmp folder for structural.
Extract protein sequence from exonerate.
Fix numerous issues with visu_path.
Add comments to visu_path and visu_network.
Use MDS to see similarity between reactions of your GSMNs.
User can provide color for each group in the group file.
Delete non-used import.
Remove pandas depedency from padmet_stats.
Add reversible reactions in visualization.
Since biopython 1.78 removes it.
Since the modification on Bio.Alphabet.
Issue when dealing with two reversible reactions that have inversed reactants and products. Issue occuring in padmet_to_padmet.
Issue when we have the same compounds multiple times in reactants or products.
This is linked to a major issue in padmet_to_padmet when we add relation. It should be possible to add the same relation linking reaction and compounds.
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.

None yet

1 participant