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

Design GOLEM facade API for simple running #177

Closed
gkirgizov opened this issue Aug 28, 2023 · 0 comments · Fixed by #251
Closed

Design GOLEM facade API for simple running #177

gkirgizov opened this issue Aug 28, 2023 · 0 comments · Fixed by #251
Assignees
Labels
architecture (re)design of existing or new framework subsystem

Comments

@gkirgizov
Copy link
Collaborator

gkirgizov commented Aug 28, 2023

Currently running the framework requires setting many parameters and understanding a lot of technical details. For example, look at current minimal simple_run.py -- it is too complicated.

GOLEM facade should be analogous to FEDOT API and require a minimal set of user parameters, if any.

Connected issues:

Some specific thoughts on parameters:

  • Adapter: possibly, by default can be used NetworkX adapter, as it's most likely familiar lib for new users.
  • Possibly, initial population, when not set, can be generated automatically
  • Tuner should be integrated into workflow, so that users can enable it with simple GOLEM.run(tuning=True)
  • If population_size and graph constraints (max graph size, graph breadth etc.) are applied automatically -- all important parameters of the search space should be Logged so that user is informed about what's happening.
@gkirgizov gkirgizov added the architecture (re)design of existing or new framework subsystem label Aug 28, 2023
@maypink maypink self-assigned this Dec 8, 2023
@maypink maypink mentioned this issue Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
architecture (re)design of existing or new framework subsystem
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants