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

✨ Defect-aware on-the-fly SiDB circuit designer #317

Merged
merged 246 commits into from
Aug 17, 2024
Merged

✨ Defect-aware on-the-fly SiDB circuit designer #317

merged 246 commits into from
Aug 17, 2024

Conversation

Drewniok
Copy link
Collaborator

Description

This PR adds the defect-aware on-the-fly SiDB circuit designer and supports all 2 input functions.

Checklist:

  • The pull request only contains commits that are related to it.
  • I have added appropriate tests and documentation.
  • I have made sure that all CI jobs on GitHub pass.
  • The pull request introduces no new warnings and follows the project's style guidelines.

marcelwa and others added 30 commits September 6, 2023 14:36
# Conflicts:
#	include/fiction/utils/truth_table_utils.hpp
Signed-off-by: GitHub Actions <actions@github.com>
@marcelwa
Copy link
Collaborator

It seems not all references/calls to apply_sidb_on_the_fly_gate_library have been renamed. Thus, the build now fails

Copy link
Collaborator

@marcelwa marcelwa left a comment

Choose a reason for hiding this comment

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

Some minor changes plus missed renaming. Maybe there are more I overlooked. Please search your entire project. This can be done easily in your IDE.

@Drewniok Drewniok requested a review from marcelwa August 17, 2024 15:49
@Drewniok
Copy link
Collaborator Author

@marcelwa Many, many thanks for your tremendous work on this PR!

@Drewniok Drewniok merged commit 246bb8d into main Aug 17, 2024
61 checks passed
@Drewniok Drewniok deleted the 2-input-pr branch August 17, 2024 18:16
@marcelwa
Copy link
Collaborator

@marcelwa Many, many thanks for your tremendous work on this PR!

My pleasure. However, you definitely did all the heavy lifting here. Great addition to the framework. I'm glad it's finally merged 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants