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

test schematics + data to review changes in rect3ddiagram.cpp #19

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ldpgh
Copy link

@ldpgh ldpgh commented Dec 13, 2016

Support work on ticket "3D cartesian plot is broken" (#386)

All schematics are modified manually to have Qucs Version 0. 0.9 by an external texteditor. It allows to compare the results across all Qucs executables from Version 0.0.9 to current version.

All schematics are placed in a single project to easily cycle trough them. It may require an improvement of run.py script to handle it automatically if project name is provided rather than the test file.

Test schematic

  • rect3ddiagram_1x_diode_dblswp_qucs.sch ... large amount of datapoint, same as reported in the #386
  • rect3ddiagram_1x_small_000_test_sch.sch ... orientation=<0,0,0>, it's like looking perpendicular at a plane
  • rect3ddiagram_1x_small_test_sch.sch ... simple surface mesh
  • rect3ddiagram_1x_test_sch.sch ... simple surface mesh + table to see the values
  • rect3ddiagram_1x_waves3x_sweeps4x_test_sch.sch ... 4 nested sweeps with 3 equations plotted
  • rect3ddiagram_1x_waves3x_test_sch.sch ... 2 nested sweeps with 3 equations plotted
  • rect3ddiagram_7x_test_sch.sch ... 7x rect3ddiagrams with different orientations

.
Data Files

  • Simple Qucsator simulation creates the data.
  • The data files needed for the content are not provided by intention. If needed no problem to add the datasets.

.
Just Proof of concept ❗

  • 2 images plus the compare script ... maybe skip the files
  • img_diff.py, qucs_rect3ddiagram_0.0.19.png, , qucs_rect3ddiagram_0.0.9.png
    • simple openCV based image comparison
    • not supported by run.py in the moment
    • images for script testing

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.

1 participant