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

Accept commands in uppercase as LISO does #3

Open
cwipf opened this issue Aug 7, 2014 · 2 comments
Open

Accept commands in uppercase as LISO does #3

cwipf opened this issue Aug 7, 2014 · 2 comments

Comments

@cwipf
Copy link

cwipf commented Aug 7, 2014

This little patch fixes it for me:

diff --git a/load_liso.m b/load_liso.m
index b323f69..71420fa 100644
--- a/load_liso.m
+++ b/load_liso.m
@@ -36,7 +36,7 @@ while true
     args = regexp(s, '\s+', 'split');


-    switch args{1}
+    switch lower(args{1})
         case 'r'     % r name value node1 node2 (resistor)
             name = args{2};
             value = circuit.parse_value(args{3});
@@ -80,7 +80,7 @@ while true
             startfreq = circuit.parse_value(args{3});
             stopfreq = circuit.parse_value(args{4});
             steps = circuit.parse_value(args{5});
-            switch args{2}
+            switch lower(args{2})
                 case 'lin'
                     f = linspace(startfreq, stopfreq, steps);
                 case 'log'
@tobin
Copy link
Owner

tobin commented Aug 7, 2014

Are component and node names case-sensitive in LISO?

@cwipf
Copy link
Author

cwipf commented Aug 7, 2014

Two component names differing only by case -- seems to work but issues a warning "name is already used, this may cause confusion"
Two node names differing only by case -- issues a warning "several pointers to same location" and gives the wrong result

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

No branches or pull requests

2 participants