Skip to content

Commit

Permalink
Fixed toolflow files to build in Ubuntu 16.04LTS
Browse files Browse the repository at this point in the history
The requirements.txt file has been updated with the "numpy<1.19". Version 1.18.5was the last to support Python 3.5, which we are using. The newer version causes conflicts. Also castro.py causes constructor error due to python pyyaml updates. This was working with pyyaml version 3.13, but not version 5.1 and above. It suggested that I add "Loader=yaml.Loader" to the yaml.load() function and it no longer breaks - checkout yaml/pyyaml#266.
  • Loading branch information
AdamI75 authored and jack-h committed Feb 9, 2021
1 parent 40cb91a commit a8e79bb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion jasper_library/castro.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ def load(filename):
loads this class object from a yaml file and assert that it is of type Castro
'''
with open(filename, 'r') as fh:
c = yaml.load(fh)
c = yaml.load(fh, Loader=yaml.Loader)
assert isinstance(c, Castro)
return c

Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
numpy
numpy<1.9
colorlog
pyaml
odict
Expand Down

0 comments on commit a8e79bb

Please sign in to comment.