Skip to content

Bump fms-extras version to avoid torch 2.3.0 issue#82

Merged
tdoublep merged 1 commit intomainfrom
tpa-fms-extras-bump
May 3, 2024
Merged

Bump fms-extras version to avoid torch 2.3.0 issue#82
tdoublep merged 1 commit intomainfrom
tpa-fms-extras-bump

Conversation

@tdoublep
Copy link
Copy Markdown
Member

@tdoublep tdoublep commented May 3, 2024

Motivation

Torch 2.3.0 was released last week and fms-extras was pulling it in as a build dependency, leading to symbol conflicts with torch 2.2.1 (what we are using currently). I've changed the versioning inside fms-extras to ensure this doesn't happen.

Modifications

Just bumping version for fms-extras

Result

Resolves ImportErrors that are present with current main:

 ImportError: /opt/tgis/lib/python3.11/site-packages/fms_extras/paged_c.cpython-311-x86_64-linux-gnu.so: undefined symbol: _ZN3c104cuda14ExchangeDeviceEa

Related Issues

n/a

@tdoublep tdoublep requested a review from njhill May 3, 2024 13:29
Signed-off-by: Thomas Parnell <tpa@zurich.ibm.com>
@tdoublep tdoublep force-pushed the tpa-fms-extras-bump branch from a7c68ce to 5d5b2d3 Compare May 3, 2024 13:30
@tdoublep tdoublep merged commit 1b801d7 into main May 3, 2024
@tdoublep tdoublep deleted the tpa-fms-extras-bump branch May 3, 2024 16:19
JRosenkranz pushed a commit that referenced this pull request May 6, 2024
#### Motivation

Torch 2.3.0 was released last week and fms-extras was pulling it in as a
build dependency, leading to symbol conflicts with torch 2.2.1 (what we
are using currently). I've changed the versioning inside fms-extras to
ensure this doesn't happen.

#### Modifications

Just bumping version for fms-extras

#### Result

Resolves ImportErrors that are present with current main:
```
 ImportError: /opt/tgis/lib/python3.11/site-packages/fms_extras/paged_c.cpython-311-x86_64-linux-gnu.so: undefined symbol: _ZN3c104cuda14ExchangeDeviceEa
 ```

#### Related Issues

n/a

Signed-off-by: Thomas Parnell <tpa@zurich.ibm.com>
Signed-off-by: Joshua Rosenkranz <jmrosenk@us.ibm.com>
Xaenalt pushed a commit to Xaenalt/text-generation-inference that referenced this pull request Jul 30, 2024
Torch 2.3.0 was released last week and fms-extras was pulling it in as a
build dependency, leading to symbol conflicts with torch 2.2.1 (what we
are using currently). I've changed the versioning inside fms-extras to
ensure this doesn't happen.

Just bumping version for fms-extras

Resolves ImportErrors that are present with current main:
```
 ImportError: /opt/tgis/lib/python3.11/site-packages/fms_extras/paged_c.cpython-311-x86_64-linux-gnu.so: undefined symbol: _ZN3c104cuda14ExchangeDeviceEa
 ```

n/a

Signed-off-by: Thomas Parnell <tpa@zurich.ibm.com>
Xaenalt pushed a commit to Xaenalt/text-generation-inference that referenced this pull request Sep 16, 2024
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