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

Move advanced design space handling in separate smt-design-space-ext module #651

Merged
merged 87 commits into from
Oct 2, 2024

Conversation

Paul-Saves
Copy link
Contributor

Create the connection between SMT and SMT-Design-Space.
Remove code related to SMT-Design-Space moved in a new repo

--- SMTOrg/SMT-Design-Space

  • Add the class ArchDesignSpaceGraph(DesignSpace) to DesignSpace.py
  • Improve seeding in DesignSpace.py
  • Fix a bug when rounding and the lower bound is not zero

  • Improve tests & notebook accordingly

@Paul-Saves Paul-Saves marked this pull request as ready for review September 22, 2024 07:37
@Paul-Saves
Copy link
Contributor Author

@relf Okay for me!

@Paul-Saves Paul-Saves force-pushed the smt-connexion-with-designspace branch from 0b343eb to 0b93b83 Compare October 1, 2024 08:24
@Paul-Saves
Copy link
Contributor Author

I fixed the initialization for Collab.

Copy link
Member

@relf relf left a comment

Choose a reason for hiding this comment

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

Actually the module name smt_design_space is confusing, I suggest to rename it in smt_design_space_ext for external or extension I mean smt has an API for design space and a standalone internal basic design space handling which can be enhanced using smt_design_space_ext module.
Sorry for this late suggestion. Naming is a well-know hard problem in computer science

smt/design_space/__init__.py Outdated Show resolved Hide resolved
@Paul-Saves Paul-Saves requested a review from relf October 1, 2024 15:17
@relf relf changed the title Smt connection with SMTOrg/designspace Move advanced design space handling in separate smt-design-space-ext module Oct 2, 2024
@Paul-Saves Paul-Saves requested a review from relf October 2, 2024 09:40
@relf relf merged commit 3646af2 into SMTorg:master Oct 2, 2024
11 checks passed
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.

2 participants