Modeldb integration: all names now comply with PEP8

[uricohen]

Additional feedback is most welcome! Also added fields for year, authors, description, reference, url

[mattions]

What is the to_text method for? (f.e.: https://github.com/uricohen/neuronvisio/commit/ce07356bf44ca07008559cb9e92fa310fc7b672e#L0R59)

Can't we use the classic str() to return the value in a string form?

[uricohen]

As you can see, it does 3 things:

• handle both strings and lists (converted to comma separated values)
• strip trailing and leading whitespaces which the xml formatting caused
• handle empty values to be empty string rather than None

If there's a better way to achieve any of those, tell me. Uri

On Fri, Sep 23, 2011 at 10:20 AM, Michele Mattioni < reply@reply.github.com>wrote:

What is the to_text method for? (f.e.: https://github.com/uricohen/neuronvisio/commit/ce07356bf44ca07008559cb9e92fa310fc7b672e#L0R59 )

Can't we use the classic str() to return the value in a string form?

Reply to this email directly or view it on GitHub: https://github.com/mattions/neuronvisio/pull/24#issuecomment-2176042

        Uri Cohen