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

Merging changes from Martin's (althonos) repo to Tom's (tomnl) #2

Merged
merged 135 commits into from
Oct 25, 2016
Merged

Merging changes from Martin's (althonos) repo to Tom's (tomnl) #2

merged 135 commits into from
Oct 25, 2016

Conversation

Tomnl
Copy link
Member

@Tomnl Tomnl commented Oct 25, 2016

My mzml2isa repo is now quite far out of date but is still referenced in search engines. To avoid confusion I am updating the repo with the most recent changes from Martin.

I will then transfer ownership to ISA-Tools. The development of the tool will then continue at ISA-Tools/mzml2isa. See issue 1.

althonos added 30 commits June 3, 2016 12:54
…ools. Registered a system command 'mzml2isa' to call from terminal. Factored xpaths to be called independently from versions. Removed build folder from repo as it is generated automaticaly by setup.py
… highlights for code in README.md. Made it so the parser now sorts files alphabeticaly (so that ouput is consistent and easier to read.
…sts on other mzML files (from HUPO-PSI/mzML repo). Added a to-do list.
…ss to attributes. Made two separate cases for instruments parsing. Added a build_env method that looks for specific vocabulary of parsed file. Added metabolights.py script that downloads studies from MetaboLight and build isa files out of i
…sa on every study of MetaboLights (EBI) containing .mzML
…are now processed into an url to the obolibrary url concerning said number.
…ght term in isa study file). United most variables of ISA_Tab into a dict for easy access with formating. Changed the way default ISA files are read so now that only the two first lines are needed (without that it led to a bug where only the nth first studies were written in the ISA study file, n being the number of lines of the default study file...
…ion. Corrected crash occuring when no instruments were found in mzML file.
…lity. Started building a system for investigation and study files templating with user-generated informations. Edited **scripts/metabolights-dl.py** so that it now parses the files it downloads. Corrected a case of obo file staying unclosed after parsing.
…e section to mzml module.

Removed m/z range warning for imzML.
@Tomnl Tomnl merged commit 49a402f into ISA-tools:master Oct 25, 2016
@althonos
Copy link
Member

Psssst @Tomnl, could you add me as a contributor now that you're omnipotent ? 😄

@Tomnl
Copy link
Member Author

Tomnl commented Oct 27, 2016

Alas, I am no longer omnipotent. My admin rights were only temporary.

Please could someone from ISA-tools (@djcomlab @proccaserra, @agbeltran) add the mzml2isa repo to the mzml2isa-galaxy team and then change the team name to 'xml2isa'.

I think we will be sorted after that.

@agbeltran
Copy link

I have added the mzml2isa repository to the team and changed the name.

@althonos
Copy link
Member

Great ! Thanks Alejandra !

@Tomnl
Copy link
Member Author

Tomnl commented Oct 27, 2016

Thanks 👍

@althonos
Copy link
Member

May I also have write access to ISA-Tools/mzml2isa ? Or do you want me to continue developing on my fork, submitting occassional patches through pull requests ?

@Tomnl
Copy link
Member Author

Tomnl commented Oct 27, 2016

Sorry I forgot to ask for admin access.

@agbeltran : Sorry to be a pain but could you change the settings for the repo xml2isa to be admin. e.g. like the mzml2isa-galaxy and nmrml2isa-galaxy

@agbeltran
Copy link

Done. And I see that @althonos is already in the team.

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.

5 participants