Skip to content

Conversation

rockywick
Copy link
Contributor

@rockywick rockywick commented Mar 14, 2025

Summary

This is regarding the issue #434

Details and comments

I have replaced the dataset used to a regression dataset and have modified the code to work for regression testing.

Result

Closes #434

Have changed the dataset and the code to work for regression tasks.
@CLAassistant
Copy link

CLAassistant commented Mar 14, 2025

CLA assistant check
All committers have signed the CLA.

The files have been updated to correct the previous errors
@coveralls
Copy link

Pull Request Test Coverage Report for Build 13886521511

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 90.637%

Totals Coverage Status
Change from base Build 13638000705: 0.0%
Covered Lines: 4366
Relevant Lines: 4817

💛 - Coveralls

Copy link
Collaborator

@edoaltamura edoaltamura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @rockywick! The code looks good to go. Let's keep the discussion about the two unit tests being almost identical open in the issue though.

@edoaltamura edoaltamura added type: enhancement ✨ Features or aspects to improve Community PR 🌐 PRs from contributors that are not 'members' of the Qiskit organization labels Mar 18, 2025
@edoaltamura edoaltamura merged commit 0cfcfbb into qiskit-community:main Mar 19, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community PR 🌐 PRs from contributors that are not 'members' of the Qiskit organization type: enhancement ✨ Features or aspects to improve
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve QSVR unit tests
4 participants