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

Add test with spice netlist for checking qucsconv #13

Merged
merged 1 commit into from
Mar 8, 2016

Conversation

in3otd
Copy link
Contributor

@in3otd in3otd commented Mar 8, 2016

A simple test for checking qucsconv, see Qucs/qucs#487

This should fail the netlist check, after merging Qucs/qucs#488 it should pass, except maybe for some 32 vs 64 bit numerical differences (the reference results were generated on a 32 b OS)

@felix-salfelder
Copy link
Member

perfect. i would like to merge this. then figure out, how to update the tests git remote in qucs.

but: it seems i do not have write permission to this repo...

@guitorri
Copy link
Member

guitorri commented Mar 8, 2016

@felix-salfelder you should be able to push now to qucs-test.

To you have to pull the latest for the qucs-test submodule in qucs then push it to the qucs remote.
Something like:

cd qucs-test
git checkout master
git pull
cd ..
git checkout -b update-submodule
git commit -am "Updated qucs-test submodule"
git push [remote] update-submodule

@felix-salfelder felix-salfelder merged commit 74c8324 into Qucs:master Mar 8, 2016
@felix-salfelder
Copy link
Member

@guitorri thanks. see qucs/#489

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

Successfully merging this pull request may close these issues.

3 participants