-
Notifications
You must be signed in to change notification settings - Fork 177
Error in yaml #582
Comments
They both run for me and generate footprints. And they both have I'm on Windows 10 with Python 3.7.2 and the PyYAML 3.13 module installed. Do you see this issue with any other scripts? Not sure this helps, but let me know if there's some info I can provide. |
@jneiva08 you need to install |
@pointhi thanks for the tip
|
So the interpreter doesn't barf on importing a non-existent module as long as it's not actually used? Interesting. TIL. |
@chschlue yep, that's python. You can also import packages inside a function and so on. |
Another potential option is to just use the built-in JSON module. |
@calebreister That being said, it's just my standpoint as of today. I don't mean to start a war or battle, but since this isn't the first time I've seen a similar comment I thought it was past time for somebody to respond to you about it. |
@evanshultz Makes sense. My main issue with YAML is that it tends to be error-prone due to the significant whitespace (although this also applies to python). Since YAML is backwards-compatible with JSON, it works fine to feed JSON files into the YAML module (which is in fact what I did in my scripts). Personally, I find JSON more user-friendly by virtue of simplicity, but it is a user-by-user decision. Perhaps it would be a good idea to convert the global config files over to JSON so that it is at least possible for users to opt to use the built-in JSON module. |
I like to know if someone can help-me
I'm trying to execute the following script
./conn_molex_micro-fit-3.0_tht_top_dual_row.py
and it is giving errorTraceback (most recent call last): File "./conn_molex_micro-fit-3.0_tht_top_dual_row.py", line 26, in <module> import yaml ImportError: No module named yaml
when try
./conn_molex_micro-fit-3.0_tht_side_dual_row.py
it worksI'm using linux
The text was updated successfully, but these errors were encountered: