Closed uricohen closed 12 years ago
Hey Uri,
should I go ahead and merge it, or you want to clean the code first and then we merge?
On Tue, Nov 1, 2011 at 11:25 PM, Uri Cohen reply@reply.github.com wrote:
Updater.py is both a stand-alone program which updates ModelDB.xml from online site and also can be used as a python module (e.g. to implement an 'update' button in neuronvisio).
The code need some cleaning, but it is fully functional and tested. One issue I did have is current ModelDB have 2 articles which are in the review process:
- Prosthetic electrostimulation for information flow repair in a neocortical simulation (under rev.)
- Spike exchange methods for a Blue Gene/P supercomputer (Hines et al., accepted) I have Uploader.py ignoring such models (which lack a year) so we'll get the final version when one exists.
You can merge this Pull Request by running:
git pull https://github.com/uricohen/neuronvisio master
Or you can view, comment on it, or merge it online at:
https://github.com/mattions/neuronvisio/pull/29
-- Commit Summary --
- Added BeautifulSoup.py version 3.2.0
- Added 'article' and 'pubmed' links
-- File Changes --
A src/neuronvisio/modeldb/BeautifulSoup.py (2014) M src/neuronvisio/modeldb/ModelDB.xml (79854) A src/neuronvisio/modeldb/Updater.py (238)
-- Patch Links --
https://github.com/mattions/neuronvisio/pull/29.patch https://github.com/mattions/neuronvisio/pull/29.diff
Reply to this email directly or view it on GitHub: https://github.com/mattions/neuronvisio/pull/29
Ok, it's ready.
On Wed, Nov 2, 2011 at 11:24 AM, Michele Mattioni < reply@reply.github.com>wrote:
Hey Uri,
should I go ahead and merge it, or you want to clean the code first and then we merge?
On Tue, Nov 1, 2011 at 11:25 PM, Uri Cohen reply@reply.github.com wrote:
Updater.py is both a stand-alone program which updates ModelDB.xml from online site and also can be used as a python module (e.g. to implement an 'update' button in neuronvisio).
The code need some cleaning, but it is fully functional and tested. One issue I did have is current ModelDB have 2 articles which are in the review process:
- Prosthetic electrostimulation for information flow repair in a neocortical simulation (under rev.)
- Spike exchange methods for a Blue Gene/P supercomputer (Hines et al., accepted) I have Uploader.py ignoring such models (which lack a year) so we'll get the final version when one exists.
You can merge this Pull Request by running:
git pull https://github.com/uricohen/neuronvisio master
Or you can view, comment on it, or merge it online at:
https://github.com/mattions/neuronvisio/pull/29
-- Commit Summary --
- Added BeautifulSoup.py version 3.2.0
- Added 'article' and 'pubmed' links
-- File Changes --
A src/neuronvisio/modeldb/BeautifulSoup.py (2014) M src/neuronvisio/modeldb/ModelDB.xml (79854) A src/neuronvisio/modeldb/Updater.py (238)
-- Patch Links --
https://github.com/mattions/neuronvisio/pull/29.patch https://github.com/mattions/neuronvisio/pull/29.diff
Reply to this email directly or view it on GitHub: https://github.com/mattions/neuronvisio/pull/29
Reply to this email directly or view it on GitHub: https://github.com/mattions/neuronvisio/pull/29#issuecomment-2601155
Uri Cohen
this is great.
So now, if I run the Update.py, I will have a new XML with the new model. If I restart Neuronvisio is that one which is picked up?
Yes, updater actually save a new version of ModelDB.xml.
Go ahead and delete 2-3 random
On Wed, Nov 2, 2011 at 4:53 PM, Michele Mattioni < reply@reply.github.com>wrote:
this is great.
So now, if I run the Update.py, I will have a new XML with the new model. If I restart Neuronvisio is that one which is picked up?
Reply to this email directly or view it on GitHub: https://github.com/mattions/neuronvisio/pull/29#issuecomment-2604323
Uri Cohen
Updater.py is both a stand-alone program which updates ModelDB.xml from online site and also can be used as a python module (e.g. to implement an 'update' button in neuronvisio).
The code need some cleaning, but it is fully functional and tested. One issue I did have is current ModelDB have 2 articles which are in the review process: