Skip to content

FIX: fixed the issue where the freq/time column got interchanged with y axis value for lna analysis and tdr #6185

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

Merged

Conversation

althaf-ansys
Copy link
Collaborator

@althaf-ansys althaf-ansys commented May 22, 2025

…ith y axis value for lna analysis and tdr

Description

Please provide a brief description of the changes made in this pull request.

Issue linked

Please mention the issue number or describe the problem this pull request addresses.

Checklist

  • I have tested my changes locally.
  • I have added necessary documentation or updated existing documentation.
  • I have followed the coding style guidelines of this project.
  • I have added appropriate tests (unit, integration, system).
  • I have reviewed my changes before submitting this pull request.
  • I have linked the issue or issues that are solved by the PR if any.
  • I have agreed with the Contributor License Agreement (CLA).

@althaf-ansys althaf-ansys requested a review from maxcapodi78 May 22, 2025 10:03
@althaf-ansys althaf-ansys requested a review from a team as a code owner May 22, 2025 10:03
@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@maxcapodi78 maxcapodi78 changed the title fixed the issue where the freq/time column got interchanged with y axis value for lna analysis and tdr FIX: fixed the issue where the freq/time column got interchanged with y axis value for lna analysis and tdr May 22, 2025
maxcapodi78
maxcapodi78 previously approved these changes May 22, 2025
@maxcapodi78 maxcapodi78 enabled auto-merge (squash) May 22, 2025 10:51
Copy link

codecov bot commented May 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.23%. Comparing base (a37ccf5) to head (a9638b3).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6185      +/-   ##
==========================================
- Coverage   85.25%   85.23%   -0.02%     
==========================================
  Files         170      170              
  Lines       64119    64119              
==========================================
- Hits        54663    54655       -8     
- Misses       9456     9464       +8     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@maxcapodi78 maxcapodi78 merged commit cdcdf1c into main May 22, 2025
81 of 86 checks passed
@maxcapodi78 maxcapodi78 deleted the bugfix/column_data_interchanged_for_lna_analysis_and_tdr branch May 22, 2025 15:17
jsalant22 pushed a commit that referenced this pull request Jun 3, 2025
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