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.

Sign up for GitHub

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

Jump to bottom

Feature/extend qknn #94

Merged
merged 17 commits into from
Oct 14, 2023
Merged

Feature/extend qknn #94

merged 17 commits into from
Oct 14, 2023

Conversation

BenediktRiegel
Copy link
Contributor

Extended the plugins QkNN and Quantum-Parzen-Window to allow duplicates in the input dataset.

@BenediktRiegel
Corrected int_to_bitlist
821c4c5
@BenediktRiegel
QAM using self.X now after assigning X to self.X
e22137e
@BenediktRiegel
SimpleHammingQkNN can now handle duplicates in training data
02b439c
@BenediktRiegel
Ensure black format
f5cb65f
@BenediktRiegel
Fixed schuld_hamming (previously, it chose the least similar label). …
0d6b4b6 
…schuld_hamming can now handle duplicates.
@BenediktRiegel
Ruan can now handle duplicate data. Fixed utils and fixed InputParame…
5add588 
…ters
@BenediktRiegel
Changed order of quantum backends
fc432de
@BenediktRiegel
Added option to not produce a visualization
fc1971d
@BenediktRiegel
Updated frontend.js
7a5b3ab
@BenediktRiegel
Removed prints and unnecessary function
dd2fcf4
@BenediktRiegel
Use custom template, instead of frontend.js
5c39e83
@BenediktRiegel
Merged main into feature/extend_qknn
9f0d63d
@BenediktRiegel
Updated visualize_change in template
e089f28
@BenediktRiegel
Removed check_duplicates import in schuld_hamming.
e9d45a3
@BenediktRiegel
Updated quantum_parzen_window to use a custom template, instead of fr…
1fb8b1c 
…ontend.js
@BenediktRiegel
Changed default backend value
5751a95
@BenediktRiegel
Updated version number
d38ebef
PhilWun
PhilWun approved these changes Aug 15, 2023
View reviewed changes
Copy link
Contributor

@PhilWun PhilWun left a comment

Choose a reason for hiding this comment

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

lgtm

@BenediktRiegel BenediktRiegel merged commit 43483db into main Oct 14, 2023
2 checks passed
@BenediktRiegel BenediktRiegel deleted the feature/extend_qknn branch October 14, 2023 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PhilWun PhilWun PhilWun approved these changes

@buehlefs buehlefs Awaiting requested review from buehlefs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BenediktRiegel @PhilWun