Skip to content

add lazy import - #75

Open
Acquent0 wants to merge 8 commits into
mainfrom
lazy_import
Open

add lazy import#75
Acquent0 wants to merge 8 commits into
mainfrom
lazy_import

Conversation

@Acquent0

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@FeiLiu36 FeiLiu36 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  1. remove all the endpoints and keys
  2. add one [all] option to allow install all packages in one command

@QingL2000 QingL2000 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

tqdm is imported by several methods' resume.py (eoh / funsearch / hillclimb / lhns / mcts_ahd, etc.), but it's now missing from both install_requires and every extra, making it an uninstallable missing dependency. Suggest adding tqdm back to the core requirements.

@Acquent0

Copy link
Copy Markdown
Contributor Author

tqdm is imported by several methods' resume.py (eoh / funsearch / hillclimb / lhns / mcts_ahd, etc.), but it's now missing from both install_requires and every extra, making it an uninstallable missing dependency. Suggest adding tqdm back to the core requirements.

Checked for other similar issues and fixed.

@Acquent0

Copy link
Copy Markdown
Contributor Author
  1. remove all the endpoints and keys
  2. add one [all] option to allow install all packages in one command

Checked for other similar issues and fixed.

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.

4 participants