Import / Export in XML about sci-corpus

[tiagocampo]

What is the issue? Be more specific

[dpizetta]

Certificate that Import and Export are working and treat main errors to not crash the program. Or basically, say to user what happens.

[tiagocampo]

Hmmm, then this is an enhancement of the actual functionality

[dpizetta]

You didnt implement the export function, so this is not closed.

[tiagocampo]

hhahaha, I forgot about it!

[dpizetta]

Export is not working:

Traceback (most recent call last):
File "F:\Downloads\sci-corpus-master (6)\sci-corpus-master\sci_corpus\sci_corpus_main.py", line 853, in exportFile
self.container.export_(path)
File "F:\Downloads\sci-corpus-master (6)\sci-corpus-master\sci_corpus\container.py", line 481, in export_
tree.write(path, pretty_print=True, xml_declaration=True)
TypeError: write() got an unexpected keyword argument 'pretty_print'

[tiagocampo]

Here works fine
Em 04/05/2014 11:57, "Daniel Pizetta" [email protected] escreveu:

Export is not working:

Traceback (most recent call last):
File "F:\Downloads\sci-corpus-master
(6)\sci-corpus-master\sci_corpus\sci_corpus_main.py", line 853, in
exportFile
self.container.export_(path)
File "F:\Downloads\sci-corpus-master
(6)\sci-corpus-master\sci_corpus\container.py", line 481, in export_
tree.write(path, pretty_print=True, xml_declaration=True)
TypeError: write() got an unexpected keyword argument 'pretty_print'

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/30#issuecomment-42134802
.

[tiagocampo]

Pretty_print is an argument to print the levels of a XML file with identation. Its not necessary, although if a human decides to read the xml file it will be much easer.

This option is part of the lmxl.ElementTree packge and not of xml.ElementTree. We have to decide what packge we want to use. I vote for lxml, as our program depends on other 3 non-standard packages, add one more will cause no harm.

[dpizetta]

For me its ok :)

On Mon, May 5, 2014 at 4:14 PM, tiagocampo [email protected] wrote:

Pretty_print is an argument to print the levels of a XML file with
identation. Its not necessary, although if a human decides to read the xml
file it will be much easer.

This option is part of the lmxl.ElementTree packge and not of
xml.ElementTree. We have to decide what packge we want to use. I vote for
lxml, as our program depends on other 3 non-standard packages, add one more
will cause no harm.

Reply to this email directly or view it on GitHubhttps://github.com//issues/30#issuecomment-42226266
.