Skip to content

clp-package: Add Python __future__.annotations import to allow running with Python 3.8.#432

Merged
kirkrodrigues merged 1 commit into
y-scope:mainfrom
kirkrodrigues:add-futures-annotations
Jun 10, 2024
Merged

clp-package: Add Python __future__.annotations import to allow running with Python 3.8.#432
kirkrodrigues merged 1 commit into
y-scope:mainfrom
kirkrodrigues:add-futures-annotations

Conversation

@kirkrodrigues

@kirkrodrigues kirkrodrigues commented Jun 9, 2024

Copy link
Copy Markdown
Member

Description

#416 used syntax that isn't supported by Python 3.8 without the __future__.annotations import. The effect of this is that the search scheduler can't run with Python 3.8 (and all searches will seemingly freeze).

This PR adds the import.

Validation performed

  • Built and started the package in an Ubuntu Focal container
  • Compressed some data
  • Used sbin/search.sh to run a query and ensured that it didn't freeze.

@junhaoliao junhaoliao self-requested a review June 10, 2024 05:06

@junhaoliao junhaoliao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Validated on WSL Ubuntu 20.04.6 LTS (Python 3.8.10).

@kirkrodrigues kirkrodrigues changed the title clp-package: Add Python __future__.annotations to allow running with Python 3.8. clp-package: Add Python __future__.annotations import to allow running with Python 3.8. Jun 10, 2024
@junhaoliao

Copy link
Copy Markdown
Member

I agree with the new PR title to be used as the commit message.

@kirkrodrigues kirkrodrigues merged commit bf54480 into y-scope:main Jun 10, 2024
@kirkrodrigues kirkrodrigues deleted the add-futures-annotations branch June 10, 2024 05:16
jackluo923 pushed a commit to jackluo923/clp that referenced this pull request Dec 4, 2024
junhaoliao pushed a commit to junhaoliao/clp that referenced this pull request May 17, 2026
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