-
Notifications
You must be signed in to change notification settings - Fork 231
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
Huge update - Compatibility with Ansys 2022 R2 (#142) #153
Open
zlatko-minev
wants to merge
1
commit into
master
Choose a base branch
from
develop
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,436
−1,390
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Update README.md * Update README.md * fix typo * update information on conda forge install * add conda badge to readme * add information on pypi install and name * fix typo * add info on pypi install and note on package name * add pypi badge to readme via fury.io * Merge remote-tracking branch 'upstream/master' into Asaf_branch * Update core_quantum_analysis.py allows to print result and print variation to be used togther where the variation to be printed is give by an integer * Update core_quantum_analysis.py * Update core_quantum_analysis.py * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md * Update about.rst * Update about.rst * Update about.rst * Update installation.rst * Update README.md * Update README.md * Sorted Qs and freqs Instead of having the sorting of the dataframe as string indices (e.g. '0', '1', '10', '2'...) it is now sorted as integers ('0', '1', '2', ... , '10'). Only relevant for >10 variations. * Update README.md * Make package name in setup.py match name on PyPI * Remove old conda recipe See https://github.com/conda-forge/pyepr-quantum-feedstock for conda recipe * Version 0.8.4 * Create greetings.yml * Create manual.yml * Create Q3D setup within HfssDesign * Create publish-to-pypi * Rename publish-to-pypi to publish-to-pypi.yml * Update README.md * Update version from 0.8.4 to 0.8.4.2 * Fixed bug with port impedance in ansys.py * Modify how to connect to Ansys * Update config_user.py * Edit yml file * Update publish-to-pypi.yml * Updated version number * If a project or design is missing in Ansys app, then return None and … (#68) * If a project or design is missing in Ansys app, then return None and give a warning. * Update warnings to be more accurate. * Revert accidental change. * typo and logger add for no active design * Add method to allow user to add a new Q3d to project referenced in ProjectInfo * Updated version number * Fix get_setup method * correction to enable q3d design setup default + removing error in get_objects_in_group() (#70) * correction to enable q3d design setup default * prevent unnecessary error when modeler is not defined in get_objects_in_group * 209 if design exists add it (#71) * Save before switching branches. * Add changes which were part of merge. * Update where the flag needs to be passed. * Need method instead of getting a list of designs. * Remove the previous solution, Decide with different solution. * Change to version 0.8.4.5 . * Remove reference to yapf. * Ansys version update for Z matrices * Update ansys.py for Ansys 2020 * get_setup fixed to use the passed setup name + typos * up-rev * Create pyEPR.bib * Update pyEPR.bib * Update README.md * Update README.md * Update README.md * Added ansys calculator functions Two functions that return vectors tangent and normal to a surface. I used it to calculate surface participation ratios, e.g.: p_metal = t* (E_surface_metal / UE) print(f'Metal-Air/Metal-Substrate participation ratio, p_MA = p_MS = {p_metal:.3}') #SA participation ratio vecE_normal = vecE.normal2surface('chip_holder1').__div__(epsilon_r) #Complex Vector vecE_tangent = vecE.tangent2surface('chip_holder1').__mul__(epsilon_r) #Complex Vector vec_E_total = vecE_normal.__add__(vecE_tangent) E_squared = vec_E_total.dot(vecE).__abs__().real().__pow__(2) E_surface = E_squared.integrate_surf(name='chip_holder1').evaluate() E_surface -= E_surface_metal p_SA = t* (E_surface / UE) print(f'Air-Substrate participation ratio, p_SA = {p_SA:.3}') * Typos mainly Also one change where an error is raised when `pint` is not properly imported. * Update ansys.py * Update ansys.py * Update README.md * Update pyEPR.bib * Update pyEPR.bib * Update pyEPR.bib * Update core_quantum_analysis.py Get numeric frequencies results should retrieve f_ND, not f_1. Some typos. * Updates linguist's calculation for pyEPR * Typos * Mostly typos, some corrections In core_quantum_analysis: Rows 671-674: update to modes that are not the first. Row 917: trying to get pandas to plot with the right label. Unsuccessfully. * clean_up_solutions functions and some typos clean_up_solutions function for an Ansys design (to avoid analysis of all variations). Also removed index name for frequencies and typos * Update core_quantum_analysis.py Fixed #83 * Tutorial2: Correct import of CalcObject to pyEPR.ansys * Indentation after was wrong and variable swp_var was never defined * fixed mode number error in QuantumAnalysis fixed KeyError when '0' is not in variations for QuantumAnalysis. updated type hint for QuantumAnalysis.analyze_variation fix mode number error in QuantumAnalysis * Update back_box_numeric.py * fix log plot bug if q__coupling isnot there * Update core_quantum_analysis.py * Update the init and setup files before creating a tag. * Update back_box_numeric.py Fix typo * Add pylint CI check * Fix old keyword argument name in first tutorial * Check for infinite values in the Quality Factors. * Change syntax to pass pylint. In particular, separate numpy from pandas. * Add int cast to pass pylint. * Change syntax to pass pylint. * Disable the pylint warning. * Use updated tag to include fix for issue #96. * Systematically fix typos in code and documentation (#99) * Test compiling docs in CI (#104) * Add dissipative elements as arguments to `ProjectInfo` (#103) * Fix docstrings formatting (#101) * https://securitylab.github.com/research/github-actions-preventing-pwn-requests/ is the reason to update. This allows first time users that have forked the repository to make a pull request. (#110) * Replace attrdict to addict for python 3.10 compatibility (#108) * Replace Attrdict to addict.Dict for python>=3.10 compatibility * Add one line for filtering addict.Dict from obj's attributes. * replace attrdict to addict * add python 3.10 to classifiers * Add python 3.9 in setup.py * Prepare for new tag for pyepr for pypi. (#112) * Update the README to reflect addict vs attrdict in both files. * Make the type hint more accurate. * Current code expects None values, as opposed to empty list. * Prepare to use next tag for Metal. * Add variation labels to plots in `hfss_report_full_convergence` (#119) * Improve assorted docstrings * Add variation labels to HFSS convergence report * Add other parametric sweep options (#117) * fix keyboard mash? * Implemented all available options for parametric sweep definitions and parametric sweep from file. * Addition of demo tutorial and associated csv file * Add DrivenTerminal support (#121) * Add single surface `DistributedAnalysis.get_Qsurface` (#123) * Fix EPR sign bug that caused EPR to always be positive (#125) * Update README.md * Update version to 0.8.5.6 for EPR sign bug fix (#126) * Fix EPR bug and bump to version 0.8.5.7 * Pass lv=lv to evaluate * Update version to 0.8.5.7 in __init__.py * Variation support to `calc_p_electric_volume` (#132) * Support synchronised variables in `Optimetrics.create_setup` (#130) * Fix code block formattings * Implement synchronised sweep support in Optimetrics * Don't extend `hfss_report_full_convergence` vertically (#135) * Support loss tangents for different surfaces in `do_EPR_analysis` (#144) * Fix #145 .gitignore (#146) * Add dtype to empty Pandas Series (#139) * Refactor `DataFrame.append` to `pd.concat` (#138) * Replace usage of np.float with float (#147) (#150) The epr_numerical_diagonalization function uses np.float which has been deprecated in Numpy. Refer https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations The continued usage of np.float yields the following error in the epr_numerical_diaganolization function when later versions of Numpy is used. "AttributeError: module 'numpy' has no attribute 'float'." This commit replace np.float with float to solve this error. * Correctly select a few modes in analyze_variation (#149) * Fix mode selection in analyze_variation Addressing issue #148: analyze_variation incorrectly chooses Pj, Sj, Om, PHI_zpf when passed a subset of modes * Remove redundant frequency selection The frequencies are already correctly selected outside of this if-statement, so this is not necessary * Update setup.py * Update __init__.py * Update core_distributed_analysis.py (#152) out-commenting a property not supported in latest version. Only visual impact --------- Co-authored-by: Zlatko Minev <[email protected]> Co-authored-by: Nathan Shammah <[email protected]> Co-authored-by: SQClab <[email protected]> Co-authored-by: Barkay Guttel <[email protected]> Co-authored-by: willsALMANJ <[email protected]> Co-authored-by: Will Shanks <[email protected]> Co-authored-by: Dennis Wang <[email protected]> Co-authored-by: Priti Ashvin Shah <[email protected]> Co-authored-by: Marco Facchini <[email protected]> Co-authored-by: Ashish Panigrahi <[email protected]> Co-authored-by: Xinyu <[email protected]> Co-authored-by: CHAO ZHOU <[email protected]> Co-authored-by: Priti A Shah <[email protected]> Co-authored-by: Niko Savola <[email protected]> Co-authored-by: Niko Savola <[email protected]> Co-authored-by: GyeonghunKim <[email protected]> Co-authored-by: Thomas G McConkey <[email protected]> Co-authored-by: Zach Parrott <[email protected]> Co-authored-by: Patrick J. O'Brien <[email protected]> Co-authored-by: Jagatheesan Jack <[email protected]> Co-authored-by: Clara Fontaine <[email protected]> Co-authored-by: Christian Kraglund Andersen <[email protected]>
@AndersenQubitLab - this is the same PR but not to pull into main, can you double check it looks ok and not replacing any new functionality? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update README.md
Update README.md
fix typo
update information on conda forge install
add conda badge to readme
add information on pypi install and name
fix typo
add info on pypi install and note on package name
add pypi badge to readme via fury.io
Merge remote-tracking branch 'upstream/master' into Asaf_branch
Update core_quantum_analysis.py
allows to print result and print variation to be used togther where the variation to be printed is give by an integer
Update core_quantum_analysis.py
Update core_quantum_analysis.py
Update README.md
Update README.md
Update README.md
Update README.md
Update README.md
Update README.md
Update README.md
Update README.md
Update README.md
Update about.rst
Update about.rst
Update about.rst
Update installation.rst
Update README.md
Update README.md
Sorted Qs and freqs
Instead of having the sorting of the dataframe as string indices (e.g. '0', '1', '10', '2'...) it is now sorted as integers ('0', '1', '2', ... , '10'). Only relevant for >10 variations.
Update README.md
Make package name in setup.py match name on PyPI
Remove old conda recipe
See https://github.com/conda-forge/pyepr-quantum-feedstock for conda recipe
Version 0.8.4
Create greetings.yml
Create manual.yml
Create Q3D setup within HfssDesign
Create publish-to-pypi
Rename publish-to-pypi to publish-to-pypi.yml
Update README.md
Update version from 0.8.4 to 0.8.4.2
Fixed bug with port impedance in ansys.py
Modify how to connect to Ansys
Update config_user.py
Edit yml file
Update publish-to-pypi.yml
Updated version number
If a project or design is missing in Ansys app, then return None and … (If a project or design is missing in Ansys app, then return None and … #68)
If a project or design is missing in Ansys app, then return None and give a warning.
Update warnings to be more accurate.
Revert accidental change.
typo and logger add for no active design
Add method to allow user to add a new Q3d to project referenced in ProjectInfo
Updated version number
Fix get_setup method
correction to enable q3d design setup default + removing error in get_objects_in_group() (correction to enable q3d design setup default + removing error in get_objects_in_group() #70)
correction to enable q3d design setup default
prevent unnecessary error when modeler is not defined in get_objects_in_group
209 if design exists add it (209 if design exists add it #71)
Save before switching branches.
Add changes which were part of merge.
Update where the flag needs to be passed.
Need method instead of getting a list of designs.
Remove the previous solution, Decide with different solution.
Change to version 0.8.4.5 .
Remove reference to yapf.
Ansys version update for Z matrices
Update ansys.py for Ansys 2020
get_setup fixed to use the passed setup name + typos
up-rev
Create pyEPR.bib
Update pyEPR.bib
Update README.md
Update README.md
Update README.md
Added ansys calculator functions
Two functions that return vectors tangent and normal to a surface. I used it to calculate surface participation ratios, e.g.:
p_metal = t* (E_surface_metal / UE)
print(f'Metal-Air/Metal-Substrate participation ratio, p_MA = p_MS = {p_metal:.3}')
Also one change where an error is raised when
pint
is not properly imported.Update ansys.py
Update ansys.py
Update README.md
Update pyEPR.bib
Update pyEPR.bib
Update pyEPR.bib
Update core_quantum_analysis.py
Get numeric frequencies results should retrieve f_ND, not f_1. Some typos.
Updates linguist's calculation for pyEPR
Typos
Mostly typos, some corrections
In core_quantum_analysis:
Rows 671-674: update to modes that are not the first. Row 917: trying to get pandas to plot with the right label. Unsuccessfully.
clean_up_solutions function for an Ansys design (to avoid analysis of all variations). Also removed index name for frequencies and typos
Fixed #83
Tutorial2: Correct import of CalcObject to pyEPR.ansys
Indentation after was wrong and variable swp_var was never defined
fixed mode number error in QuantumAnalysis
fixed KeyError when '0' is not in variations for QuantumAnalysis. updated type hint for QuantumAnalysis.analyze_variation
fix mode number error in QuantumAnalysis
Update back_box_numeric.py
fix log plot bug if q__coupling isnot there
Update core_quantum_analysis.py
Update the init and setup files before creating a tag.
Update back_box_numeric.py
Fix typo
Add pylint CI check
Fix old keyword argument name in first tutorial
Check for infinite values in the Quality Factors.
Change syntax to pass pylint. In particular, separate numpy from pandas.
Add int cast to pass pylint.
Change syntax to pass pylint.
Disable the pylint warning.
Use updated tag to include fix for issue Plot Hamiltonian results is not working due to inf values in data frame. #96.
Systematically fix typos in code and documentation (Systematically fix typos in code and documentation #99)
Test compiling docs in CI (Test compiling docs in CI #104)
Add dissipative elements as arguments to
ProjectInfo
(Add dissipative elements as arguments toProjectInfo
#103)Fix docstrings formatting (Fix docstrings formatting #101)
https://securitylab.github.com/research/github-actions-preventing-pwn-requests/ is the reason to update. This allows first time users that have forked the repository to make a pull request. (https://securitylab.github.com/research/github-actions-preventing-pwn… #110)
Replace attrdict to addict for python 3.10 compatibility (Replace attrdict to addict for python 3.10 compatibility #108)
Replace Attrdict to addict.Dict for python>=3.10 compatibility
Add one line for filtering addict.Dict from obj's attributes.
replace attrdict to addict
add python 3.10 to classifiers
Add python 3.9 in setup.py
Prepare for new tag for pyepr for pypi. (Prepare for new tag for pyepr for pypi. #112)
Update the README to reflect addict vs attrdict in both files.
Make the type hint more accurate.
Current code expects None values, as opposed to empty list.
Prepare to use next tag for Metal.
Add variation labels to plots in
hfss_report_full_convergence
(Add variation labels to plots inhfss_report_full_convergence
#119)Improve assorted docstrings
Add variation labels to HFSS convergence report
Add other parametric sweep options (Add other parametric sweep options #117)
fix keyboard mash?
Implemented all available options for parametric sweep definitions and parametric sweep from file.
Addition of demo tutorial and associated csv file
Add DrivenTerminal support (Add DrivenTerminal support #121)
Add single surface
DistributedAnalysis.get_Qsurface
(Add single surfaceDistributedAnalysis.get_Qsurface
#123)Fix EPR sign bug that caused EPR to always be positive (Fix EPR sign bug that caused EPR to always be positive #125)
Update README.md
Update version to 0.8.5.6 for EPR sign bug fix (Update version to 0.8.5.6 #126)
Fix EPR bug and bump to version 0.8.5.7
Pass lv=lv to evaluate
Update version to 0.8.5.7 in init.py
Variation support to
calc_p_electric_volume
(Variation support tocalc_p_electric_volume
#132)Support synchronised variables in
Optimetrics.create_setup
(Support synchronised variables inOptimetrics.create_setup
#130)Fix code block formattings
Implement synchronised sweep support in Optimetrics
Don't extend
hfss_report_full_convergence
vertically (Don't extendhfss_report_full_convergence
vertically #135)Support loss tangents for different surfaces in
do_EPR_analysis
(Support loss tangents for different interfaces (surfaces) indo_EPR_analysis
#144)Fix The .gitignore file is missing _config_user #145 .gitignore (Fix #145 .gitignore #146)
Add dtype to empty Pandas Series (Add dtypes to empty Pandas Series #139)
Refactor
DataFrame.append
topd.concat
(RefactorDataFrame.append
topd.concat
#138)Replace usage of np.float with float (Numpy 1.24 floating problem #147) (Replace usage of np.float with float (#147) #150)
The epr_numerical_diagonalization function uses np.float which has been deprecated in Numpy.
Refer https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
The continued usage of np.float yields the following error in the epr_numerical_diaganolization function when later versions of Numpy is used.
"AttributeError: module 'numpy' has no attribute 'float'."
This commit replace np.float with float to solve this error.
Correctly select a few modes in analyze_variation (Correctly select a few modes in analyze_variation #149)
Fix mode selection in analyze_variation
Addressing issue #148: analyze_variation incorrectly chooses Pj, Sj, Om, PHI_zpf when passed a subset of modes
The frequencies are already correctly selected outside of this if-statement, so this is not necessary
Update setup.py
Update init.py
Update core_distributed_analysis.py (Update core_distributed_analysis.py #152)
out-commenting a property not supported in latest version. Only visual impact