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

Build broken with new PyYaml==4.1 (or 4.1b1) #161

Closed
altendky opened this issue Jun 27, 2018 · 6 comments
Closed

Build broken with new PyYaml==4.1 (or 4.1b1) #161

altendky opened this issue Jun 27, 2018 · 6 comments
Labels

Comments

@altendky
Copy link
Collaborator

https://travis-ci.org/ebroecker/canmatrix/jobs/397500683

5.98s$ source ./test.sh
Obtaining file:///home/travis/build/ebroecker/canmatrix
Requirement already satisfied: future in /home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages (from canmatrix==0.6.post58+gdd34a77)
Requirement already satisfied: lxml in /home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages (from canmatrix==0.6.post58+gdd34a77)
Requirement already satisfied: xlrd in /home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages (from canmatrix==0.6.post58+gdd34a77)
Requirement already satisfied: xlwt in /home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages (from canmatrix==0.6.post58+gdd34a77)
Requirement already satisfied: xlsxwriter in /home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages (from canmatrix==0.6.post58+gdd34a77)
Requirement already satisfied: pyyaml in /home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages (from canmatrix==0.6.post58+gdd34a77)
Installing collected packages: canmatrix
  Running setup.py develop for canmatrix
Successfully installed canmatrix
arxml -> ['csv', 'dbc', 'dbf', 'json', 'kcd', 'sym', 'xls', 'xlsx', 'xml', 'yaml']
Traceback (most recent call last):
  File "./test.py", line 85, in <module>
    canmatrix.convert.convert(in_file, out_file)
  File "/home/travis/build/ebroecker/canmatrix/canmatrix/convert.py", line 198, in convert
    canmatrix.formats.dumpp(outdbs, outfileName, **options)
  File "/home/travis/build/ebroecker/canmatrix/canmatrix/formats.py", line 113, in dumpp
    dump(db, fileObject, exportType, **options)
  File "/home/travis/build/ebroecker/canmatrix/canmatrix/formats.py", line 89, in dump
    moduleInstance.dump(canMatrixOrCluster, fileObject, **options)
  File "/home/travis/build/ebroecker/canmatrix/canmatrix/yaml.py", line 68, in dump
    f.write(yaml.dump(newdb).encode('utf8'))
  File "/home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages/yaml/__init__.py", line 217, in dump
    return dump_all([data], stream, Dumper=Dumper, **kwds)
  File "/home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages/yaml/__init__.py", line 196, in dump_all
    dumper.represent(data)
  File "/home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages/yaml/representer.py", line 26, in represent
    node = self.represent_data(data)
  File "/home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages/yaml/representer.py", line 57, in represent_data
    node = self.yaml_representers[None](self, data)
  File "/home/travis/virtualenv/python3.5.5/lib/python3.5/site-packages/yaml/representer.py", line 229, in represent_undefined
    raise RepresenterError("cannot represent an object", data)
yaml.representer.RepresenterError: ('cannot represent an object', <canmatrix.canmatrix.CanMatrix object at 0x7fc24d0a3ef0>)

Nothing jumps out to me as an obvious cause in the announcements. Maybe the 'safe by default' aspect though?

@altendky altendky added the bug label Jun 27, 2018
@altendky
Copy link
Collaborator Author

On #pyyaml it was suggested I start with yaml/pyyaml#74 and yaml/pyyaml#177 to start learning what might be wrong.

@ebroecker
Copy link
Owner

is there anybody out there who uses the yaml-format?
Maybe we just should kick it out?

@altendky
Copy link
Collaborator Author

I can take a little time. I don't actively use yaml anywhere but I wonder if I should.

@ebroecker
Copy link
Owner

My plan was to add all information to json, so there would be no need for yaml anymore ...

@altendky
Copy link
Collaborator Author

altendky commented Jun 28, 2018

There are various differences between the formats so I don't think simply having all data in JSON alleviates any potential use of YAML. But... I don't really care. :] I was just following up on my commit resulting in a broken build, even if totally unrelated to my commit.

@ebroecker
Copy link
Owner

for me yaml works.
If not, create a new issue...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants